Fix flaky ingredient-row visibility in recipe-form.feature #1

New issue

Closed

opened 2026-08-27 12:47:56 +02:00 by kyuno · 1 comment

kyuno commented

2026-08-27 12:47:56 +02:00

Owner

Erreur observée :
AssertionError: Timed out retrying after 4000ms: expected '<span.ingredient-row__name>' to be 'visible'
This element <span.ingredient-row__name> is not visible because its content is being clipped by one of its parent elements, which has a CSS property of overflow: hidden, scroll or auto

Reproduction : cd apps/web puis (PowerShell) :
$env:ELECTRON_EXTRA_LAUNCH_ARGS="--disable-gpu --disable-software-rasterizer --disable-gpu-compositing --no-sandbox"
npx cypress run --e2e --browser chrome --spec "cypress/e2e/recipe-form.feature"

Merci d'investiguer : soit un vrai bug de layout CSS (overflow clippant .ingredient-row__name dans le formulaire d'édition de recette), soit un souci de timing du test (l'élément apparaît après le timeout par défaut). Voir apps/web/cypress/e2e/recipe-form.feature et son step-definitions colocalisé pour le scénario exact, et le composant RecipeForm/ingredient-row

Le scénario Cypress "Preloads an existing recipe's ingredients when editing, and lets you add another" dans apps/web/cypress/e2e/recipe-form.feature échoue de façon reproductible (2 runs consécutifs, pas un flake ponctuel) sur la branche main actuelle, avant tout changement lié à la fonctionnalité de pagination/scroll infini des sources externes (aucun fichier touché par cette session n'a de rapport avec RecipeForm/ingredient-row). Erreur observée : AssertionError: Timed out retrying after 4000ms: expected '<span.ingredient-row__name>' to be 'visible' This element <span.ingredient-row__name> is not visible because its content is being clipped by one of its parent elements, which has a CSS property of overflow: hidden, scroll or auto Reproduction : cd apps/web puis (PowerShell) : $env:ELECTRON_EXTRA_LAUNCH_ARGS="--disable-gpu --disable-software-rasterizer --disable-gpu-compositing --no-sandbox" npx cypress run --e2e --browser chrome --spec "cypress/e2e/recipe-form.feature" Merci d'investiguer : soit un vrai bug de layout CSS (overflow clippant .ingredient-row__name dans le formulaire d'édition de recette), soit un souci de timing du test (l'élément apparaît après le timeout par défaut). Voir apps/web/cypress/e2e/recipe-form.feature et son step-definitions colocalisé pour le scénario exact, et le composant RecipeForm/ingredient-row

kyuno added the

bug

label 2026-08-27 17:38:03 +02:00

kyuno added the

planned

label 2026-09-05 15:48:36 +02:00

kyuno added the

Low

label 2026-09-05 18:52:07 +02:00

kyuno referenced this issue from a commit

2026-09-05 20:46:04 +02:00

test(recipes): fiabilise le scénario 3 pages (scroll réel + log de curseurs)

kyuno commented

2026-09-05 21:04:22 +02:00

Author

Owner

Ne se reproduit plus sur main : le scénario "Preloads an existing recipe's ingredients when editing" passe en CI (job e2e, recipe-form.feature 5/5).

C'était bien un souci de timing/scroll du test, pas un bug de layout CSS. Le step partagé Then("the recipe should include the ingredient {string}") dans cypress/support/step_definitions/common.steps.ts fait maintenant cy.contains(".ingredient-row__name", name).scrollIntoView().should("be.visible") — la liste d'ingrédients peut être sous la ligne de flottaison du scroll de .app-content, et .should("be.visible") ne défile pas tout seul.

Fermé comme corrigé. Traité pendant la passe autonome sur les tickets planned, sans nouvelle PR.

Ne se reproduit plus sur `main` : le scénario "Preloads an existing recipe's ingredients when editing" passe en CI (job e2e, `recipe-form.feature` 5/5). C'était bien un souci de timing/scroll du test, pas un bug de layout CSS. Le step partagé `Then("the recipe should include the ingredient {string}")` dans `cypress/support/step_definitions/common.steps.ts` fait maintenant `cy.contains(".ingredient-row__name", name).scrollIntoView().should("be.visible")` — la liste d'ingrédients peut être sous la ligne de flottaison du scroll de `.app-content`, et `.should("be.visible")` ne défile pas tout seul. Fermé comme corrigé. Traité pendant la passe autonome sur les tickets `planned`, sans nouvelle PR.

kyuno closed this issue

2026-09-05 22:56:09 +02:00