Repository navigation
Conversation
yorha-operator
left a comment
yorha-operator
left a comment
There was a problem hiding this comment.
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
left a comment
yorha-operator
left a comment
There was a problem hiding this comment.
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.
37d0132 to
35cf87d
Compare
shikanime
changed the title
09b4a0f to
8be7be5
Compare
8be7be5 to
fd35bba
Compare
|
🤖 Hey ! A preview of the application is available at : https://console-pr-2589.dso.cpin-hp.numerique-interieur.fr Please be patient, deployment may take a few minutes. |
fd35bba to
b5ab8be
Compare
b5ab8be to
7df372e
Compare
StephaneTrebel
left a comment
StephaneTrebel
left a comment
There was a problem hiding this comment.
Le parcours Playwright vérifie utilement la bascule Afficher/Cacher et le test d’intégration cible les clés stables, sans exposer les valeurs tournantes. La branche est à 46 commits de main : rebaser et relancer les contrôles avant approbation.
7df372e to
c390e5d
Compare
|
Relecture de la tête actuelle : le parcours Playwright vérifie la bascule Afficher/Cacher et le scénario d’intégration vérifie les secrets attendus sans figer leurs valeurs. La PR étant en brouillon, aucun verdict formel n’est publié. |
c390e5d to
30a83fb
Compare
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 Co-authored-by: Automata <automata@shikanime.studio>
Co-authored-by: Automata <automata@shikanime.studio>
30a83fb to
48f61fa
Compare
|
CC #2796 |
4 participants
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 :dashboard.spec.ts) ;playwright/integration-tests/user-flow.spec.ts— « Check project secrets » :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.