fix: update E2E tests for branding changes

- Update navigation test to expect "GroomBook" (default branding) instead
  of hardcoded "Paws & Reflect" since CustomerPortal now uses dynamic branding
- Add /api/branding mock to shared E2E fixtures so BrandingProvider resolves
  immediately in all tests, preventing unhandled fetch interference

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
GroomBook CTO
2026-03-19 11:04:40 +00:00
parent ae9afab5a5
commit 7d030f20db
2 changed files with 13 additions and 1 deletions
+12
View File
@@ -16,6 +16,18 @@ export const test = base.extend({
await page.route("**/api/dev/config", (route) =>
route.fulfill({ json: { authDisabled: false } })
);
// Mock the branding endpoint so BrandingProvider resolves immediately
await page.route("**/api/branding", (route) =>
route.fulfill({
json: {
businessName: "GroomBook",
primaryColor: "#4f8a6f",
accentColor: "#8b7355",
logoBase64: null,
logoMimeType: null,
},
})
);
// Seed localStorage as a fallback in case the mock is bypassed
await page.addInitScript(() => {
localStorage.setItem(
+1 -1
View File
@@ -42,7 +42,7 @@ test.beforeEach(async ({ page }) => {
test("customer portal loads at root", async ({ page }) => {
await page.goto("/");
await expect(page.getByRole("navigation").getByText("Paws & Reflect")).toBeVisible();
await expect(page.getByRole("navigation").getByText("GroomBook")).toBeVisible();
await expect(page.locator("nav")).toBeVisible();
});