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: Paperclip <noreply@paperclip.ing>
fix(gro-558): add paginated mock for /api/invoices (#261 )
2026-04-11 16:43:45 +00:00 · 2026-04-11 16:38:51 +00:00
1 changed files with 5 additions and 1 deletions
@@ -44,7 +44,10 @@ test.beforeEach(async ({ page }) => {
        json: { newClients: [], activeInPeriodCount: 0, churnRisk: [], churnRiskTotal: 0 },
      });
    }
-    // Appointments, clients, services, staff, invoices, book, etc.
+    if (url.includes("/api/invoices")) {
+      return route.fulfill({ json: { data: [], total: 0 } });
+    }
+    // Appointments, clients, services, staff, book, etc.
    return route.fulfill({ json: [] });
  });
 });
@@ -82,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();
 });