Preparando revisão do back-end do T1#11
Preparando revisão do back-end do T1#11gugabs wants to merge 1 commit intoissitarual:mainfrom gugabs:main
Conversation
gugabs
left a comment
gugabs
left a comment
There was a problem hiding this comment.
Fala Ana, Lau e Yann! Tudo bem com vocês? Aqui é o Guga :)
Parabéns pela entrega de mais um projetão! A aplicação de vocês ficou simplesmente maravilhosa, mandaram bem demais.
Só queria atentar vocês a respeito da independência dos testes, evitem usar rotas diferentes daquelas que estão sendo testadas pois isso pode invalidar um teste que na teoria está correto.
Nos repositórios da entrega de vocês eu deixei alguns comentários que podem ajudá-los daqui para frente.
Saudades de vocês! 💕
| confirmPassword:'1' | ||
| }; | ||
|
|
||
| const signUpResult = await supertest(app).post("/signUp").send(signUp); |
There was a problem hiding this comment.
Não utilizem outras rotas nos testes de uma rota específica, isso pode gerar um erro no teste a partir daquilo que não está sendo testado.
| const errors = PasswordSchema.validate(req.body).error; | ||
|
|
||
| if(errors) { | ||
| console.log(errors) | ||
| return res.sendStatus(400); | ||
| } | ||
| try{ | ||
| const result = await connection.query(` |
There was a problem hiding this comment.
Código com alguns problemas de indentação.
1 participant
No description provided.