You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Playwright ladder-config.spec fails on main: 'Dismiss notification' strict-mode violation #802
The Frontend E2E check "Playwright Chromium" fails on main in frontend/tests-e2e/ladder-config.spec.ts ("ladder editor preserves selections and resets new configs at 1280px"):
locator.click: strict mode violation: getByRole('button', { name: 'Dismiss notification' }) resolved to 2 elements
Two toast "Dismiss notification" buttons are present when the spec clicks one; the locator needs scoping (or the duplicate toast needs explaining). "Playwright Chromium" is not a required check, so merges are not blocked, but the failure hides real regressions in this suite.
Closing as already fixed by #868 (merged 2026-10-14, after this issue was filed). It changed frontend/tests-e2e/ladder-config.spec.ts to wait for the previous notification to disappear before the next save. On main a14dc8f the spec passes in the full Playwright run, and 20/20 on repeated runs (10 per viewport width).
Activity
IAmJSD commentedon Oct 24, 2026
Closing as already fixed by #868 (merged 2026-10-14, after this issue was filed). It changed
frontend/tests-e2e/ladder-config.spec.tsto wait for the previous notification to disappear before the next save. On main a14dc8f the spec passes in the full Playwright run, and 20/20 on repeated runs (10 per viewport width).