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

feat: adicionado cypress e criado alguns casos de testes #4 #11

Closed
wants to merge 1 commit into from

Conversation

kevenleone
Copy link

Olá !!

Conforme sugerido na #4 eu criei uma configuração e alguns casos de testes com Cypress (https://www.cypress.io/) essa ferramenta incrível para realização de testes E2E.

Caso vejam utilidade nesta implementação agregaria muito a ideia do projeto.

Abraços

@vercel
Copy link

vercel bot commented Feb 10, 2020

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/filipedeschamps/brasilapi/eu8pittfh
✅ Preview: https://brasilapi-git-fork-kevenleone-master.filipedeschamps.now.sh

@filipedeschamps
Copy link
Member

Sensacional @kevenleone !! Eu quero analisar com calma ao longo da semana, até porque nunca usei o Cypress, mas de qualquer forma, vou lhe adicionar na lista dos contribuidores porque sua postura foi fantástica, entrou num repositório super novo, deu sua sugestão, implementou o PR rapidamente para ver como isso fica aplicado na parte prática, e pra mim isso vale tudo 👍

@kevenleone
Copy link
Author

Opa @filipedeschamps , muito obrigado pelo feedback 🤙

Pode analisar com calma a viabilidade do Cypress no projeto, como eu utilizo bastante no meu dia-a-dia posso garantir que é uma ferramenta excelente. Mas se tiver meios mais ágeis para realizar os testes da aplicação com certeza deve ser levado em consideração.

Espero ajudar em outras questões também, valeuu!!

@vinicioslc
Copy link

Muito prático e eficiente esse tal de cypress irei conferir @kevenleone

@filipedeschamps
Copy link
Member

@kevenleone vou fechar esse PR em favor do https://github.com/filipedeschamps/BrasilAPI/pull/29 mas de qualquer forma, MUITO obrigado por essa contribuição, inclusive você me fez testar um Cypress ao ponto de eu querer usar em um projeto privado meu 👍

@kevenleone
Copy link
Author

Opa @filipedeschamps , tudo bem sem problemas, fico feliz por ter contribuído e que você conheceu uma ferramenta nova que possa te ajudar em projetos futuros, abraços 🤗

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.

None yet

3 participants