Conversation
|
Do the tests pass already? |
|
Yes, they do ;-) |
benel
left a comment
benel
left a comment
There was a problem hiding this comment.
J'ai indiqué plusieurs écarts à ce que je considère comme les "best practices". Après... Tu es le responsable de l'entrepôt. Tu peux aussi dire que ça te convient tel que c'est ;)
features/diary_create.feature
Outdated
|
|
||
| Scénario: créer un journal | ||
|
|
||
| Soit l'analyse est connecté |
features/diary_create.feature
Outdated
| Scénario: créer un journal | ||
|
|
||
| Soit l'analyse est connecté | ||
| Quand l'analyse crée un journal |
features/diary_create.feature
Outdated
|
|
||
| Soit l'analyse est connecté | ||
| Quand l'analyse crée un journal | ||
| Alors le journal est créé |
There was a problem hiding this comment.
Comment sais-tu que c'est le nouveau journal (et pas un autre créé précédemment) ?
There was a problem hiding this comment.
Je n'ai pas l'impression que ce scénario soit implémenté, si ?
features/step_definitions/context.rb
Outdated
| visit "/memo/" | ||
| end | ||
|
|
||
| Soit("l'analyse est connecté") do |
| @@ -0,0 +1,12 @@ | |||
| #language: fr | |||
|
|
|||
| Fonctionnalité: Entrer | |||
There was a problem hiding this comment.
C'était probablement un bon entraînement pour la suite... mais je pense qu'à terme ça n'a pas forcément beaucoup de sens de le garder car ce n'est pas une "fonctionnalité" au sens propre. Je ne me dis pas avant de me coucher : "Demain, il faut que je pense à entrer à partir de la page d'accueil".
|
|
||
| Soit un navigateur sur la page d'accueil | ||
| Quand l'analyste clique sur "Journal pour l'analyse qualitative" | ||
| Alors le titre "Journaux de bord" est affiché |
There was a problem hiding this comment.
Ce qui est entre guillemets correspond normalement à des données. C'est un passage de paramètres.
| Soit un navigateur sur la page d'accueil | ||
| Quand l'analyste clique sur "Journal pour l'analyse qualitative" | ||
| Alors le titre "Journaux de bord" est affiché | ||
| Et le bouton "S'identifier" est affiché |
| Quand l'analyste clique sur "Journal pour l'analyse qualitative" | ||
| Alors le titre "Journaux de bord" est affiché | ||
| Et le bouton "S'identifier" est affiché | ||
| Et le bouton "Créer un compte" est affiché |
|
|
||
| Fonctionnalité: Créer un compte | ||
|
|
||
| Scénario: se rendre au bon endroit |
There was a problem hiding this comment.
Normalement les différents scénarios correspondent à différents cas et non à différentes étapes du même cas.
| Scénario: se rendre au bon endroit | ||
|
|
||
| Soit la page des journaux de bord | ||
| Quand l'analyste clique sur "Créer un compte" |
There was a problem hiding this comment.
Là encore les guillemets ne servent pas à cela. C'est par exemple pour indiquer un nom et le vérifier plus loins dans le scénario.
christophe-lejeune
force-pushed
the
tests
branch
2 times, most recently
from
acbd7e0 to
7be27b9
Compare
* A user can register herself * A registered user can connect to the platform * A connected user can create a diary
christophe-lejeune
force-pushed
the
tests
branch
from
7be27b9 to
54b5420
Compare
christophe-lejeune
force-pushed
the
master
branch
3 times, most recently
from
27690bc to
aba8ed2
Compare
2 participants
The associated commit contains a draft for a few test to be reinstated.