-
Notifications
You must be signed in to change notification settings - Fork 42
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
Aula 05 - Testes para criar sessão no banco #143
Comments
@lbmendes tem uma menção a isso no próprio bloco. Mas não sei se é suficiente |
Ah verdade @dunossauro, vacilo meu. Obrigado pelo retorno! Li e fui fazendo a aula em 2 dias separados e acabei me passando nessa informação. Uma sugestão que pode ajudar a deixar a info mais em evidência seria já deixar adicionado o comentário Fique a vontade para fechar a issue. Valeu! |
Ah vi agora que o comentário é só um e ele já aplica a todo bloco, não precisa colocar em cada linha, show de bola. Sucesso: with Session(engine) as session: # pragma: no cover
yield session |
Acho que vou adicionar uma nota sobre isso na aula em que refatorarmos o projeto. Mas vou fechar por agora. Obrigado @lbmendes |
Acabei de finalizar a aula 05 e os respectivos exercícios e após realizar a
task test
a parte de coverage mostra que ainda faltam testes no arquivo fast_zero/database.pyPelo HTML gerado mostra que o consta pendente de testes neste arquivo são as duas últimas linhas, as quais seguem abaixo:
Essas duas linhas em específico devem ser testadas explicitamente? Ou devem ser ignoradas? Acho que pode ser interessante acrescentar uma menção a essas linhas na aula 05.
No mais a aula está excelente, estou adorando o curso. Abraço!
The text was updated successfully, but these errors were encountered: