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

Fix: Corrigindo testes E2E #76

Merged
merged 2 commits into from
Apr 10, 2024
Merged

Fix: Corrigindo testes E2E #76

merged 2 commits into from
Apr 10, 2024

Conversation

GuilhermeAureliano
Copy link
Contributor

Basicamente, foram realizadas correções nas URLs que anteriormente consideravam os deploys da aplicação, passando agora a considerar o localhost. Isso se deve ao fato de que o localhost representa um ambiente local e controlado, afetado pelas alterações locais.

No arquivo de teste login.cy.ts, agora não há simulação da funcionalidade de login; ele realiza efetivamente o login, levando em consideração o nome de usuário e a senha descritos nos testes. Portanto, para aqueles que forem realizar os testes em sua máquina localmente, é necessário criar esse usuário.

No arquivo de teste signup.cy.ts, a funcionalidade de cadastro é simulada, pois a cada execução do teste, um usuário seria de fato criado na aplicação. Portanto, talvez fosse ideal haver um endpoint no frontend para excluir o usuário após o cadastro, o que atualmente não está disponível. Por esse motivo, a simulação é preferível.

Além disso, há um teste específico que falha (veja a imagem em anexo), possivelmente devido a um erro recebido do backend. No caso, o teste espera a mensagem "Número de telefone inválido.", e não a mensagem estranha mostrada na imagem.

Captura de tela 2024-04-09 235049

@davigps davigps merged commit 2574098 into main Apr 10, 2024
1 check passed
@davigps davigps deleted the fix/tests-E2E branch April 10, 2024 09:38
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