Add 404 page unit test #142
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Obrigada por criares este teste @juliamendesc
Vou adicionar outros maintainers aqui, com mais conhecimento para ajudar a rever.
Podias acrescentar o comando que usaste para correr o teste, no README, na lista de comandos.
|
Cheguei! Só umas coisas antes: Como a @toothlesspanda disse, há outras tasks para configs e setup do framework. Por isso não fiz nessa, mas se preferirem podemos tentar já fazer. Só que pra isso, penso que preciso do vosso input sobre como preferem organizar (página tests ou arquivos junto dos componentes, etc) e sobre que nome preferem. Além disso, relativamente à #125, apenas usei a testing-library porque já estava no package.json, por isso não me aventurei por outras sugestões (também pq não tenho vasta experiência com testes). Vou resolver os conflitos e ver os comentários do @gustavocr19-developer pra dar update nessa branch :) |
|
SORRY, did shit! ja repus! |
|
Por mim, só resolver os últimos comentários está ok :) |
|
Gente, obrigada pela ajuda em todos os PRs. Foi muito divertido contribuir com vcs. :) |
| } | ||
| }, | ||
|
|
||
| resources: { pt: { translation: translationsPT }, en: { translation: translationsEN } } |
There was a problem hiding this comment.
Dei conta agora que as traducoes estao a falhar na pagina de Preview com esta PR. Nao sei bem o que é 🤔
Não há pressas, mas achei que devia deixar aqui o comentário.
There was a problem hiding this comment.
Já vi e corrigi! No preview parece estar bem agora :)
There was a problem hiding this comment.
Acabei de ver! Já corrigi e parece estar ok no preview! 👯
|
bora dar merge a isto? |
There was a problem hiding this comment.
Obrigada pela tua contribuição!
cc @toothlesspanda se tás okay com isto, força ai no merge 👍🏾 Eu usaria o 'Squash & Merge' option, porque há aqui uns commits que vêm de outros branches, isso ia ajudar a ter so um commit para a PR toda
Descrição
Adiciona teste unitário para verificar se página 404 é renderizada.
Fixes #118
Como testar esta modificação?
Correr o comando npm run test
Checklist: