-
Notifications
You must be signed in to change notification settings - Fork 7
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
TEST: Reintroducing tests #118
base: master
Are you sure you want to change the base?
Conversation
Do the tests pass already? |
Yes, they do ;-) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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é |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
l'analyse ou l'analyste ?
features/diary_create.feature
Outdated
Scénario: créer un journal | ||
|
||
Soit l'analyse est connecté | ||
Quand l'analyse crée un journal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
l'analyse ou l'analyste ?
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comment sais-tu que c'est le nouveau journal (et pas un autre créé précédemment) ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
l'analyse ou l'analyste ?
@@ -0,0 +1,12 @@ | |||
#language: fr | |||
|
|||
Fonctionnalité: Entrer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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é |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
idem.
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é |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
idem
|
||
Fonctionnalité: Créer un compte | ||
|
||
Scénario: se rendre au bon endroit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
acbd7e0
to
7be27b9
Compare
* A user can register herself * A registered user can connect to the platform * A connected user can create a diary
7be27b9
to
54b5420
Compare
27690bc
to
aba8ed2
Compare
The associated commit contains a draft for a few test to be reinstated.