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

Projeto 2 - Aline Prado #17

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

Conversation

linieprado
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!!
Seu código está muito elegante, e fez até os exercícios extras!!
Mandou muito bem


# Assert
self.assertTrue(emprestado) # Verifica se o livro foi emprestado com sucesso
self.assertTrue(livro.emprestado) # Verifica se o atributo emprestado do livro está como True
Copy link
Collaborator

Choose a reason for hiding this comment

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

O teste está muito coerente, deu para notar como vc compreendeu o conteúdo e aplicou o TDD na implementação.
Só nesse ponto é um erro muito sutil que todas nós que iniciamos em Python passamos.

Vc está comparando o objeto livro no seu teste ao invés do seu alvo de teste, que é o livro de dentro da lista self.biblioteca.livros

O teste funcionou pq o python faz uma cópia superficial do objeto auxiliar (livro) e o objeto que vc atribuiu esse valor (self.biblioteca.livros[x].livro), então quando altera um o outro também é alterado.

Não se preocupe, é apenas um detalhe de como a linguagem funciona, do ponto de vista da entrega está ótimo, é apenas para informar que o python funciona dessa forma. E para se aprofundar aqui está um artigo: Python - A diferença entre “Deep Copy e Shallow Copy”

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