Repair UI-core E2E baseline after shell overhaul #30

Closed

nsaspy wants to merge 7 commits from fix/ui-core-e2e-baseline into main

nsaspy commented

2026-08-27 19:28:31 +00:00

Owner

Summary

Repairs the nine stale Playwright expectations left red by merged PR #20. The failures reproduce identically on #20's own head and on unrelated graph-control PR #21, so this is baseline contract repair rather than a graph regression workaround.

test the unified runtime StatusCenter instead of removed .sync-badge markup;
assert the persistent shared sidebar/topbar on desktop graph routes;
keep graph geometry assertions inside the global shell instead of requiring route-wide fullscreen takeover;
use the visible desktop dataset filter instead of removed fullscreen dataset button/dialog chrome;
account for the mobile topbar when validating graph/editor viewport bounds;
update the mobile workspace label and include the first-class Datasets route in the shared route-model expectations.

No production application code is changed.

Evidence

The exact same nine E2E cases failed on:

#20 head 078342aa946983fee3dc57239f3542abd4918faf (CI run 173), before it was merged;
#21 head 5a07911dbaebda44e554799183930ad20f6c0ea7 (CI run 179).

Both failure-artifact sets are identical by test case.

Merge gate

Merge only when this exact head is green. Afterward, rebase #21 and other current branches on the repaired main baseline.

## Summary Repairs the nine stale Playwright expectations left red by merged PR #20. The failures reproduce identically on #20's own head and on unrelated graph-control PR #21, so this is baseline contract repair rather than a graph regression workaround. - test the unified runtime StatusCenter instead of removed `.sync-badge` markup; - assert the persistent shared sidebar/topbar on desktop graph routes; - keep graph geometry assertions inside the global shell instead of requiring route-wide fullscreen takeover; - use the visible desktop dataset filter instead of removed fullscreen dataset button/dialog chrome; - account for the mobile topbar when validating graph/editor viewport bounds; - update the mobile workspace label and include the first-class Datasets route in the shared route-model expectations. No production application code is changed. ## Evidence The exact same nine E2E cases failed on: - #20 head `078342aa946983fee3dc57239f3542abd4918faf` (CI run 173), before it was merged; - #21 head `5a07911dbaebda44e554799183930ad20f6c0ea7` (CI run 179). Both failure-artifact sets are identical by test case. ## Merge gate Merge only when this exact head is green. Afterward, rebase #21 and other current branches on the repaired main baseline.

nsaspy added 7 commits

2026-08-27 19:28:31 +00:00