Skip to content

Conversation

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

Olá, Natalia.
Parabéns pela entrega! Essa semana foi correria mas espero que esse projeto te ajude a continuar progredindo em sua jornada.
No código tem alguns ajustes a serem feitos mas coisas bem simples.
Mandou bem!

Copy link
Collaborator

Choose a reason for hiding this comment

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

Acho que esse arquivo foi copiado para cá por engano. :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Acho que esse arquivo foi copiado para cá por engano. :)

raise TypeError(f"Esperado Livro obtido valor {livro} do tipo {type(livro)}")
self.livros.append(livro)

def exibir_livros(self):
Copy link
Collaborator

Choose a reason for hiding this comment

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

Boa! simples e objetivo

def exibir_livros(self):
return self.livros

def emprestar_livro(self): No newline at end of file
Copy link
Collaborator

Choose a reason for hiding this comment

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

Aqui ficou faltando o corpo do método, a lógica mesmo. E se deixar apenas a assinatura dessa forma dá erro e não executa

with self.assertRaises(TypeError):
biblioteca.adicionar_livro(livro)

def test_exibir_livros(self):
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ficou faltando nesse teste chamar o método exibir_livros e fazer a validação (assert)

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