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

Teste para usuário não logado #131

Merged
merged 8 commits into from Aug 5, 2018

Conversation

amandaclarck
Copy link
Contributor

  • Foi criado um novo teste para um usuário que não está logado, ou seja, usando o mesmo form que já existia, porém agora o teste cobre que esse usuário se registrando com e-mail e senha não quebre mais o fluxo de registro
  • Veio um adicional quando fiz o pull retirando o default: true de mail_notification (tive que commitar)

@alexanmtz
Copy link
Member

@amandaclarck , depois de alguns ajustes o seu Pull request será integrado. Por favor, dê uma olhada nas mudanças que fiz para que você entenda melhor. Havia muitos passos desnecessários no cenário de usuário não logado, pois ele pode se cadastrar apenas preenchendo o formulário de guia uma vez.

Parabéns pela solução. Tive que fazer alguns ajustes também para que os testes rodassem adequadamente sem depender do Mailchimp, que introduzimos neste tempo.

E sobre o schema com o mail_notification, eu deixei como está no master.

Obrigado pela contribuição e espero que tenha aprendido bastante sobre esta ferramenta do Rails (Cucumber)

🚀 🚀 🚀 🚀 🚀 🚀

@alexanmtz alexanmtz merged commit 68843a6 into worknenjoy:master Aug 5, 2018
alexanmtz pushed a commit that referenced this pull request Sep 17, 2018
* Teste para novo guia

* Adiciona teste para novo guia registrado e ajuste no teste de ver meu perfil

* Retira default:true de mail_notification

* Teste para usuário não logado

* updating schmea

* check if theres mailchimp key

* calling mailchimp api only if the api keys are defined
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

2 participants