Skip to content

test(playwright): complément d'acceptance utilisateur — parcours secrets - #2589

Open
shikanime wants to merge 3 commits into
mainfrom
test/uat-playwright
Open

test(playwright): complément d'acceptance utilisateur — parcours secrets#2589
shikanime wants to merge 3 commits into
mainfrom
test/uat-playwright

Conversation

@shikanime

@shikanime shikanime commented Aug 25, 2026

Copy link
Copy Markdown
Member

Issues liées

#2574


Quel est le comportement actuel ?

Le parcours UI des secrets de projet n'avait aucun spec Playwright dédié ; le cahier de tests fonctionnels documente la bascule « Afficher/Cacher les secrets des services ».

Quel est le nouveau comportement ?

playwright/e2e-tests/secrets.spec.ts — un seul test :

  • bascule du libellé Afficher ↔ Cacher sur un projet neuf (le retour à l'état caché clôt le parcours) ;
  • la visibilité initiale du bouton reste couverte par « Should show project secrets » (dashboard.spec.ts) ;
  • le cas projet verrouillé (bouton indisponible) n'est pas couvert — la stack CI ne provisionne pas de projet verrouillé.

playwright/integration-tests/user-flow.spec.ts — « Check project secrets » :

  • sur un environnement d'intégration, la modale expose la section GITLAB avec les clés GIT_MIRROR_PROJECT_ID / GIT_MIRROR_TOKEN (valeur non vide) et la section VAULT — clés seulement, les valeurs tournent avec le provisionnement.

Suite e2e : 176 tests listés (chromium+firefox), lint ESLint vert. Le parcours a été exécuté avec succès dans le run merge-queue #32905282108 (4 shards Playwright success) sur une base proche.

Cette PR introduit-elle un breaking change ?

Non.

@yorha-operator yorha-operator left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Verdict : Approuvé — la PR reste en draft dans l'attente de la décision d'atterrissage.

Complément d'acceptance SECR autonome : un seul fichier, deux commits dans le périmètre, aucun conflit prévisible avec les autres files. ✨ Éloge — le parcours secrets en Playwright referme l'écart customer-facing identifié dans #2574 entre unité/e2e et expérience réelle.

@yorha-operator yorha-operator left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Verdict : Approuvé — la PR reste en draft dans l'attente de la décision d'atterrissage.

Complément d'acceptance SECR autonome : un seul fichier, deux commits dans le périmètre, aucun conflit prévisible avec les autres files. ✨ Éloge — le parcours secrets en Playwright referme l'écart customer-facing identifié dans #2574 entre unité/e2e et expérience réelle.

@shikanime
shikanime force-pushed the test/uat-playwright branch 3 times, most recently from 37d0132 to 35cf87d Compare September 10, 2026 15:15
@shikanime shikanime changed the title test(playwright): complément d'acceptance utilisateur — parcours secrets (SECR) test(playwright): complément d'acceptance utilisateur — parcours secrets Sep 10, 2026
@shikanime
shikanime force-pushed the test/uat-playwright branch 2 times, most recently from 09b4a0f to 8be7be5 Compare September 10, 2026 15:35
@shikanime
shikanime marked this pull request as ready for review September 10, 2026 15:40
@shikanime
shikanime requested a review from a team as a code owner September 10, 2026 15:40
shikanime and others added 3 commits September 10, 2026 17:41
Co-authored-by: Automata <automata@shikanime.studio>
Retire le test redondant (visibilité du toggle, déjà couvert par
'Should show project secrets' dans dashboard.spec.ts), le skip verrouillé
(0 assertion) et le guard afterEach superflu (deleteProject gère sa
navigation). Un seul test conserve le contrat : libellé Afficher/Cacher.

Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr>
Change-Id: I0e1156816ab0c44aebbbdfbf7be968576a6a6964
Le parcours d'intégration vérifie désormais l'existence des secrets du
projet via la modale : section GITLAB avec les clés
GIT_MIRROR_PROJECT_ID / GIT_MIRROR_TOKEN (valeur non vide), section
VAULT présente. Clés seulement — les valeurs tournent avec le
provisionnement du projet.

Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr>
Change-Id: If822346c7f6fbf9df8d1ee3b0a44a13c6a6a6964
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants