-
Notifications
You must be signed in to change notification settings - Fork 1
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
7 feat login de usuário #6
The head ref may contain hidden characters: "7-feat-login-de-usu\u00E1rio"
Conversation
37f838c
to
c6a8a5b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aprovado, muito bom
Nos testes de body incompleto vocês fizeram uma escolha interessante para se implementar, apesar de que quando fazemos dessa forma estamos meio q "omitindo informações de teste" pq não estamos dizendo logo no cabeçalho do teste, nesse caso 'Status 400: Deve dar erro se o body estiver incompleto' qual parte do corpo da requisição está faltando. EU, por exemplo não faria assim, colocaria separado por instâncias de testes distintas se tivesse faltando algo, mas depende tbm da quantidade de campos faltantes pra se testar se a validação do body está funcionando, se não acaba se tornando um caso de testes exaustivos kkk. Bom código irmão 👍👍 parte do código citado ⬇️⬇️⬇️
|
o fato de vcs estarem modularizando a implementação das funções do controller é uma escolha boa, só tem q manter a atenção que para projetos diminutos, como o caso que vocês estão trabalhando, onde não vai ocorrer a reutilização das funções que vcs modularizaram para outros propósitos, seria mais "facil" ou "rápido" vcs implementarem direto no controller. |
PR para merge da #4