Skip to content

fix(e2e): la fiche produit attend sa création avant de relire la liste ; traces conservées sur échec (refs #498) - #500

Merged
guycorbaz merged 2 commits into
mainfrom
fix/e2e-course-fiche-produit
Oct 3, 2026
Merged

guycorbaz merged 2 commits into
mainfrom
fix/e2e-course-fiche-produit

Conversation

@guycorbaz

Copy link
Copy Markdown
Owner

Refs #498 (KF-054) — ne la ferme pas : sa cause n'est pas établie.

Ce qui change

  • product-revenue-account.spec.ts:133 : une course du test, pas une pollution d'état. Le test cliquait sur Créer puis relisait aussitôt la liste par l'API, sans attendre le POST : sous la charge de la suite complète, la relecture partait la première et revenait vide (Expected: 6, Received: undefined). fillProductForm attend désormais la réponse du serveur.
    • Preuve : délai de 1,5 s injecté sur le POST — l'ancien test échoue avec le symptôme exact, le corrigé passe (avec et sans délai).
  • trace: 'retain-on-failure' dans playwright.config.ts : la cause de products.spec.ts:166 (KF-054, [KF-054] products.spec.ts:166 — « Créer » reste désactivé 30 s, par intermittence, dans la suite complète #498) n'a pas pu être établie, parce que le rejeu isolé avait effacé test-results/. La prochaine occurrence laissera sa trace.
  • docs/testing.md : les deux cas décrits, et la consigne de mettre frontend/test-results/ à l'abri avant tout rejeu.

Vérifications

🤖 Generated with Claude Code

guycorbaz and others added 2 commits October 3, 2026 12:44
…e ; traces conservées sur échec

product-revenue-account.spec.ts:133 relisait la liste par l'API sans
attendre le POST qu'il venait de déclencher : sous la charge de la suite
complète, la liste revenait vide. Prouvé par un délai injecté (l'ancien test
échoue avec le symptôme exact, le corrigé passe). Classé à tort en
pollution d'état.

products.spec.ts:166 n'a pas de cause établie (non reproduit) : la trace est
désormais conservée sur échec, et docs/testing.md dit de mettre
test-results/ à l'abri avant tout rejeu.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…e (refs #498)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@guycorbaz
guycorbaz merged commit 912bef3 into main Oct 3, 2026
3 checks passed
@guycorbaz
guycorbaz deleted the fix/e2e-course-fiche-produit branch October 3, 2026 13:18
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.

1 participant