test(e2e): give the default-view redirect a realistic budget

[rubenvdlinde]

user-settings.spec.ts — "default view Meetings redirects the app root to the meetings list" — failed at 20.9s against a 20s budget, on both attempts of one run (retries: 1), then passed on a re-run of the identical sha. Nothing about the app changed between those runs.

Why it is under-budgeted. The test does three full app navigations — settings panel, app root, deep link — each blocking on the pre-mount initializeStores() round trip. The restore comment in the test already recorded the arithmetic from an earlier flake fix: the redirect landed at 10.6s and the deep link held at 17.6s. That is roughly 2 seconds of headroom, so a merely busy runner tips it over.

⚠️ The 20s does not come from the config you find first. The root playwright.config.ts says timeout: 30_000; the governing value is timeout: 20_000 in tests/e2e/playwright.config.ts, because CI resolves the config through playwright-test-path. I read the wrong file first and had to chase the discrepancy — worth stating so the next person does not.

60s deliberately, not the 120s the facet specs use: enough that load cannot fail it, small enough that a genuine hang still fails this test rather than burning the job's 45-minute cap.

Why it is worth fixing rather than re-running. This flake made decidesk's development read red while the code was fine — it is the expensive kind, because it trains people to re-run rather than read. It also cost real time here: it appeared on the run for a formatter sweep, so the first hypothesis was that the sweep had changed behaviour. Re-running the same sha is what ruled that out.

[rubenvdlinde]

Cross-reference: #821 (fix/user-settings-default-view-budget, +16/-0) targets the same failing spec — the default-view redirect timeout that shows as decidesk's single e2e failure (1 failed / 131 passed). Both are open and mergeable; worth closing one before they conflict.

[github-actions]

Quality Report — ConductionNL/decidesk @ b54d2d3

Check	PHP	Vue	Security	License	Tests
lint	✅
phpcs	✅
phpmd	✅
psalm	✅
phpstan	✅
phpmetrics	✅
eslint		✅
stylelint		✅
build		✅
check-manifest		✅
check-nav-ceiling		✅
test-l10n		✅
format		✅
composer			✅	✅ 104/104
npm			✅	✅ 538/538
app:check-code	⏭️
info.xml	✅
REUSE	❌
PHPUnit					✅
Newman					✅
Playwright					✅
Hydra gates					✅

---

Quality workflow — 2026-08-21 09:01 UTC

Download the full PDF report from the workflow artifacts.

[rubenvdlinde]

Closing as a duplicate — #821 landed the same fix first and did it better.

Both PRs diagnosed the same defect independently and reached the same arithmetic: the test needs the settings panel plus three full app navigations, ~16s of a 20s budget, so it fails on load rather than on defect (it passed at 05:54 and timed out at 07:08 on unchanged code).

#821 uses test.slow(), which triples the configured budget for this test only. That is better than my hardcoded test.setTimeout(60_000): it stays correct if the project cap in tests/e2e/playwright.config.ts is ever retuned, whereas my literal would silently drift away from it.

The one thing worth carrying over from this PR's description, since it cost me time to establish: the 20s comes from tests/e2e/playwright.config.ts, not the root playwright.config.ts, which says 30s. CI resolves the config through playwright-test-path, so the file you find first is not the one that governs the run.