semana 8 #32
semana 8 #32
Conversation
mayaracsferreira
left a comment
mayaracsferreira
left a comment
There was a problem hiding this comment.
Parabéns pela entrega!!
Você atingiu o objetivo e criou testes de acordo!! Arrasou
O único feedback que tenho está no outro comentário mas não afeta sua nota de entrega!!
Mandou bem!
| #Act | ||
| self.biblioteca.emprestar_livro(nome_livro = 'O vendedor de sonhos') | ||
| #Assert | ||
| self.assertTrue(self.biblioteca.emprestar_livro) |
There was a problem hiding this comment.
Os testes mostram bastante coesão com a entrega!
Só tem um ponto de atenção nessa linha: no seu método emprestar_livro caso o livro esteja na lista biblioteca.livros vc não retorna nada (ou retorna None).
Somado a esse fato esse assertTrue está te retornando um falso positivo
Isso foi uma surpresa até para mim mas a documentação explica:
"Observe que isso é equivalente a bool(expr) is True e não a expr is True (use assertIs(expr, True) para o último)."
Portanto caso queira que seu método retorne True quando o livro é encontrado e emprestado (teria que modificar o metodo também) um assert sugerido para esse teste seria:
self.assertIs(self.biblioteca.emprestar_livro, True)
ou fazer um assert com o tipo None ( self.assertIsNone() )
2 participants
No description provided.