From 203f1135a298317d00fb2c5a19142eb6da7d17a2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marc-Andr=C3=A9=20Bombeck?= Date: Sun, 13 Sep 2026 23:24:02 +0200 Subject: [PATCH] test(e2e): wait for the capture picker to close before opening More The mobile smoke test pressed Escape on the capture picker and tapped the More button straight away. While the picker's close animation still covers the bottom nav, that tap lands on the overlay and the hub never opens, which failed the lucide-react update once. The test now waits until the picker is hidden. --- e2e/mobile-viewport.spec.ts | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/e2e/mobile-viewport.spec.ts b/e2e/mobile-viewport.spec.ts index ac6a0a8b3..91234b8bd 100644 --- a/e2e/mobile-viewport.spec.ts +++ b/e2e/mobile-viewport.spec.ts @@ -213,8 +213,11 @@ test.describe("mobile-viewport smoke", () => { for (const kind of ["measurement", "medication", "mood"]) { await expect(page.getByTestId(`capture-picker-${kind}`)).toBeVisible(); } - // Dismiss the picker before opening the hub. + // Dismiss the picker before opening the hub, and wait until it is gone: + // while its close animation runs, the overlay still covers the bottom + // nav and swallows the tap on "More", so the hub never opens. await page.keyboard.press("Escape"); + await expect(page.getByTestId("capture-picker-options")).toBeHidden(); // 2) The More hub keeps Mood + Measurements reachable (additive // middle-path — they left the strip but are NOT orphaned).