Skip to content

Conversation

@danisnegrao
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.

Dani
parabéns por ter concluído a entrega. O método de exibir livros está coerente, o teste bem escrito verificando as saídas do metodo.
Porém o método de emprestar livro está fazendo algo diferente do esperado para esse método, e o teste não tem um assert.
Por outro lado fez a entrega no prazo, se esforçou para entregar o exercício e mostrou bastante participação, tirando dúvidas.

Vc está indo bem, continue curiosa e aqui são ajustes bem simples a serem feitos. Parabéns

def emprestar_livro(self, nome_livro):
for livro in self.livros:
if livro.nome == nome_livro:
self.livros.remove(livro)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Acredito que aqui tenha tido um equivoco, uma troca da implementação do método emprestar_livro com o remover_livro.

O esperado para o método emprestar_livro é:
"O método deve marcar o valor de esta_emprestado como True"


# Act/Assert
self.biblioteca.emprestar_livro(nome_livro)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Aqui parece estar faltando o assert para o método de emprestar_livro

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