From f3b43c831f653c7f8c386512e79868103f849eb1 Mon Sep 17 00:00:00 2001 From: Viktor Vaczi Date: Sun, 19 Jul 2026 18:13:44 +0200 Subject: [PATCH] =?UTF-8?q?test(web):=20wait=20out=20the=20boot=20overlay?= =?UTF-8?q?=20before=20clicking=20Tools=20=E2=80=94=20assign-footprints=20?= =?UTF-8?q?flake?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The spec's boot gates (canvas, registry, title) are all wx-side and go live seconds before WasmTool drops its opaque boot overlay (ready only flips after the collab seed + waitForWxUi). The Tools click landed on the overlay, the popup never opened, and the menu-item wait timed out — flaky on CI (the 1s title poll sometimes samples after overlay unmount), 100% locally. Add the inset-0.z-30 count-0 guard the chrome-toggle / tool-switch / quit-* specs already use. Verified: web-chromium 5/5, web-firefox 8/9 (was 0/5, 0/1). The one remaining firefox failure is a distinct pre-existing wasm trap on the CvPcb-open path (EVT.MENU dispatch → "index out of bounds" in a worker), same signature as the eeschema-fp-selector CI trap — now known to reproduce on macOS real GL too. Left as a TODO(cvpcb-open-trap) in the spec; needs a debug-symbol repro. Co-Authored-By: Claude Fable 5 Claude-Session: https://claude.ai/code/session_01Rb9jsqtHsC3tHTaJ45244j --- tests/web/eeschema-assign-footprints.spec.ts | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/tests/web/eeschema-assign-footprints.spec.ts b/tests/web/eeschema-assign-footprints.spec.ts index 11bce2a..db3292d 100644 --- a/tests/web/eeschema-assign-footprints.spec.ts +++ b/tests/web/eeschema-assign-footprints.spec.ts @@ -91,6 +91,12 @@ test('Tools → Assign Footprints opens CvPcb (merged third kiface)', async ({ p await expect .poll(() => page.title(), { timeout: 150000, intervals: [1000] }) .toMatch(/Schematic Editor/i); + // The wx UI (canvas, registry, title) is live seconds BEFORE WasmTool drops + // its opaque boot overlay — `ready` only flips after the collab seed + + // waitForWxUi. A click in that window lands on the overlay, the Tools popup + // never opens, and the menu-item wait below times out. Same guard as + // chrome-toggle.spec.ts: boot + lib fat-load overlays are both `inset-0 z-30`. + await expect(page.locator('div.inset-0.z-30')).toHaveCount(0, { timeout: 150000 }); await page.screenshot({ path: SHOT('01-boot'), scale: 'css' }); expect(await clickMenuBarItem(page, 'Tools'), 'Tools menu opened').toBe(true); @@ -125,6 +131,15 @@ test('Tools → Assign Footprints opens CvPcb (merged third kiface)', async ({ p } }, item); + // TODO(cvpcb-open-trap): ~1/9 local web-firefox runs die right here — the + // EVT.MENU dispatch traps ("wx_dom_event(48,9) failed" + worker + // "RuntimeError: index out of bounds"), eeschema survives, CvPcb never + // appears, zero libs-bridge calls. Same trap signature as the + // eeschema-fp-selector CI trap (docs/features/web-e2e-rot/01) — which + // reproduced on macOS real GL, so it is NOT llvmpipe-only. Needs a + // debug-symbol repro; post-link binaryen rewriting makes the shipped + // module's DWARF useless for symbolizing the trap offset. + // // The frame construction + netlist mail runs off wxPostEvent'd follow-ups, // which the wx WASM port only flushes on input events — wiggle the mouse // while polling. (No auto-answering of prompts here: CvPcb itself shows an