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

semana 8 #32

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

semana 8 #32

wants to merge 2 commits into from

Conversation

DamarisSantos
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.

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)
Copy link
Collaborator

Choose a reason for hiding this comment

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

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() )

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