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

docs: add e2e test development process #264

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

clarani
Copy link
Contributor

@clarani clarani commented Feb 3, 2025

No description provided.

@@ -31,6 +31,15 @@ Cela se fait par des appels API en typescript avant de lancer le test à proprem

Les données testées sont les mêmes en local ou via l'intégration continue.

#### Process de développement des tests e2e
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
#### Process de développement des tests e2e
#### Processus de développement des tests e2e

- certaines parties du test devront être mockées le temps que la fonctionnalité soit entièrement développée mais à la fin des développements, le test e2e devra être complet et les données mockées devront avoir disparu. Les modifications finales à faire dans le test pour retirer le mocking seront normalement minimes (uniquement changer les expected values).
- les cas de test et les parcours utilisateur devront être définis en amont, au moment du refinement du ticket, avant le PIP. Ils pourront être proposés par Aymen ou un.e PO, et devront être validés par Aymen, le.la PO compétent.e et des devs front.
- si un test e2e touche à la config des tests e2e, à l’architecture du projet (typiquement le snapshoting) ou présente un risque de ralentir la CI, un atelier de refinement devra être organisé pour consulter les personnes en charge de l'architecture du projet et de la CI, notamment la team devops.

#### Atomicité d'un test
Chaque test doit être **atomique** : il se suffit à lui même et ne peut pas être divisé.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Chaque test doit être **atomique** : il se suffit à lui même et ne peut pas être divisé.
Chaque test doit être **atomique** : il se suffit à lui-même et ne peut pas être divisé.

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.

2 participants