Unit tests

[comteharbour]

Recap of our workflow

Prerequisites

• [ ] You have configured the triggering of the pre-commit hook on your local repository's clone.
• [ ] You have covered your code with unit and acceptance tests
• [ ] The feature you have implemented has no tag @wip any more
• [ ] None of the feature files have a tag @current or @focus

Changes

Please describe here what has changed between this feature branch and the dev branch

How to use the feature

Please describe here how to use the feature, possibly with animated GIFs

Additional notes

Merci @zadigus de regarder le fichier Signup.spec.js pour me faire un retour sur les tests Est-ce que tu attends que tous les champs soient remplis ? Il me semble que je fais la pull request trop tôt si c'est le cas.

[zadigus]

Pour les prochaines PRs, merci de remplir le template ci-dessus avec les détails nécessaires

Qui que ce soit qui fait la review n'est pas nécessairement très au courant de ce code. Pour cette raison, mettre un contexte clair à ta PR est vraiment plus que recommandé pour gagner du temps.

[zadigus]

ok to test

[zadigus]

@comteharbour les tests d'acceptance ne passent pas, mais ça n'a pas l'air dû aux modifications de cette branche. On dirait que la branche dev a un problème.

[zadigus]

je fais des tests sur jenkins et te tiens au courant

[comteharbour]

@zadigus Ma faute, apparemment. J'ai oublié de vérifier que les tests d'acceptance passent. Comme j'ai refactoré la page signup, il y a des références qui ne collent plus.

Je sais comment corriger ces erreurs, mais je ne sais pas comment les envoyer dans la pull request. Tu peux me donner des infos ?