Skip to content

Conversation

@rafaellafiel
Copy link

No description provided.

Copy link
Collaborator

@mayaracsferreira mayaracsferreira left a comment

Choose a reason for hiding this comment

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

Rafaella, arrasou!!
parabéns pela entrega, código muito legível, bem organizado, tem um pequeno ajuste a ser feito no método emprestar_livro e no teste correspondente, mas são coisas simples
Muito bem!

return self.livros

def emprestar_livros(self, livro: Livro):
livro.esta_emprestado = True No newline at end of file
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

#autor_livro = "bell hooks"
livro = Livro("Tudo sobre o amor", "bell hooks")
self.biblioteca.emprestar_livros(livro)
self.assertTrue = () No newline at end of file
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)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants