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 guiado II - Natalia Rosa #36

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

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