From 1d76c631370c7eff5dca25dac824b161a72f3f0b Mon Sep 17 00:00:00 2001 From: "groombook-qa[bot]" <269744346+groombook-qa[bot]@users.noreply.github.com> Date: Sat, 11 Apr 2026 16:50:35 +0000 Subject: [PATCH] fix(e2e): use domcontentloaded instead of networkidle in admin invoices test The networkidle wait causes flakiness in CI due to slow external resource loading. Use domcontentloaded which fires earlier and is sufficient for SPA navigation checks. Co-authored-by: Pawla Abdul (Bot) Co-authored-by: Paperclip --- apps/e2e/tests/navigation.spec.ts | 1 + 1 file changed, 1 insertion(+) diff --git a/apps/e2e/tests/navigation.spec.ts b/apps/e2e/tests/navigation.spec.ts index 8ba228c..29a7060 100644 --- a/apps/e2e/tests/navigation.spec.ts +++ b/apps/e2e/tests/navigation.spec.ts @@ -85,6 +85,7 @@ test("admin staff page loads", async ({ page }) => { test("admin invoices page loads", async ({ page }) => { await page.goto("/admin/invoices"); + await page.waitForLoadState("domcontentloaded"); await expect(page.getByText("GroomBook")).toBeVisible(); await expect(page.getByRole("link", { name: "Invoices" })).toBeVisible(); });