Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Projeto Guiado 2 - Rafaella Fiel #24

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 19 additions & 0 deletions exercicios/para-casa/Biblioteca.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
# Sistema de biblioteca
# Adicionar, listar e emprestar livros
from Livro import Livro

class Biblioteca:
"""Classe biblioteca"""
def __init__(self):
self.livros = []

def adicionar_livro(self, livro: Livro):
if (not isinstance(livro, Livro)):
raise TypeError(f"Esperado do valor {livro} do tipo {type(livro)}")
self.livros.append(livro)

def exibir_livros(self):
return self.livros

def emprestar_livros(self, livro: Livro):
livro.esta_emprestado = True
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Olá Rafaella.
Arrasou entregando toda a atividade. Aqui nesse método acho que só a lógica pode ser aprimorada.
Aqui vc está definindo o objeto livro que recebeu de parametro como emprestado
Mas seria mais interessante a gente alterar o status do livro que está na nossa biblioteca, ou seja dentro de self.livros

6 changes: 6 additions & 0 deletions exercicios/para-casa/Livro.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
class Livro:
"""Classe livro"""
def __init__(self, nome, autor):
self.nome = nome
self.autor = autor
self.esta_emprestado = False
Empty file.
44 changes: 44 additions & 0 deletions exercicios/para-casa/testeBiblioteca.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,44 @@
from unittest import TestCase
from Biblioteca import Biblioteca
from Livro import Livro

class TestBiblioteca(TestCase):
def setUp(self):
self.biblioteca = Biblioteca()

def test_init_deve_passar(self):
# arrange e act
#biblioteca = Biblioteca()
# assert
self.assertIsInstance(self.biblioteca.livros, list)

def test_adicionar_livro_deve_passar(self):
# arrange
#biblioteca = Biblioteca()
nome_livro = "Tudo sobre o amor"
autor_livro = "bell hooks"
livro = Livro(nome = nome_livro, autor = autor_livro)
# act
self.biblioteca.adicionar_livro(livro)
# assert
self.assertEqual(1, len(self.biblioteca.livros))

def test_adicionar_livro_nao_deve_inserir_numero(self):
# arrange
#biblioteca = Biblioteca()
livro = 1988
# act e assert
with self.assertRaises(TypeError):
self.biblioteca.adicionar_livro(Livro)

def test_exibir_livros_deve_passar(self):
livro = Livro("Tudo sobre o amor", "bell hooks")
self.biblioteca.adicionar_livro(livro)
self.assertEqual([livro], self.biblioteca.exibir_livros())

def test_emprestar_livro(self):
#nome_livro = "Tudo sobre o amor"
#autor_livro = "bell hooks"
livro = Livro("Tudo sobre o amor", "bell hooks")
self.biblioteca.emprestar_livros(livro)
self.assertTrue = ()
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aqui nesse teste acho que ele está retornando um falso positivo. Quando executamos só self.assertTrue ele executa sem erros mas também não valida o que queremos :T
será que vc não está querendo validar o assertTrue com o seu alvo do teste? (o livro emprestado)

14 changes: 14 additions & 0 deletions exercicios/para-casa/testeLivro.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
from unittest import TestCase
from Livro import Livro

class TestLivro(TestCase):
def test_init_deve_passar(self):
# arrange
nome = "pachinko"
autor = "Min Jin Lee"
# act
livro = Livro(nome, autor)
# assert
self.assertEqual(nome, livro.nome)
self.assertEqual(autor, livro.autor)
self.assertEqual(False, livro.esta_emprestado)