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
Only under full-suite load. Failed 1 of 3 full npm run test:e2e runs; reload-resume.spec.js on its own passed 5/5.
It is a timeout, not a wrong value. The failing run took 30.1 s; passing runs take 2.0–2.4 s. No assertion diff was produced — the whole test times out.
What is not known
The cause. Unlike #126, there is no diagnosed race here: that one compares a HUD snapshot against a later pagehide save, and buoyancy momentum accounts for the delta. This one gives no assertion failure to reason from, only a wall-clock blowout an order of magnitude past the normal run.
Worth establishing first:
Which step stalls — the reload, the resume, or the free-flow state settling.
Whether it is contention (the suite runs workers: 1, but the static server and browser are shared) or a genuine deadlock in the resume path that only a loaded machine exposes.
Whether the 30 s is Playwright's default timeout or an internal wait.
A trace or video from a failing full-suite run would settle most of that; --trace on-first-retry would capture it cheaply.
Split from #126 on review. Both are e2e races and may share timing pressure under full-suite load, but they need different fixes and different closure criteria:
Between them, these two make a green verify weaker evidence than it appears. That is worth fixing on its own terms — CI has been used repeatedly as the basis for merge decisions.
tests/reload-resume.spec.js:150intermittently fails under full-suite load:What is known
mainwith the fix: give the diver a body instead of colliding on its centre #129 branch stashed and its new spec moved aside.npm run test:e2eruns;reload-resume.spec.json its own passed 5/5.What is not known
The cause. Unlike #126, there is no diagnosed race here: that one compares a HUD snapshot against a later
pagehidesave, and buoyancy momentum accounts for the delta. This one gives no assertion failure to reason from, only a wall-clock blowout an order of magnitude past the normal run.Worth establishing first:
workers: 1, but the static server and browser are shared) or a genuine deadlock in the resume path that only a loaded machine exposes.A trace or video from a failing full-suite run would settle most of that;
--trace on-first-retrywould capture it cheaply.Relationship to #126
Split from #126 on review. Both are e2e races and may share timing pressure under full-suite load, but they need different fixes and different closure criteria:
Fixing #126 would not close this, and vice versa.
Why it matters
Between them, these two make a green
verifyweaker evidence than it appears. That is worth fixing on its own terms — CI has been used repeatedly as the basis for merge decisions.