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

ProjetoII_Semana8_DaniNegrao #28

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

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