E2E test failure: reports page loads #46

Closed
opened 2026-03-18 03:20:53 +00:00 by ghost · 1 comment
ghost commented 2026-03-18 03:20:53 +00:00 (Migrated from github.com)

Problem

The reports page loads E2E test fails on main. 12/13 navigation tests pass, but the reports test fails:

tests/navigation.spec.ts:51:1 › reports page loads
Error: expect(locator).toBeVisible() failed
Locator: getByText('Groom Book')
Expected: visible
Timeout: 5000ms
Error: element(s) not found

The test navigates to /reports and expects "Groom Book" text to be visible.

CI Run

Possible Causes

  • Reports page may not be fully implemented or may be missing navigation/header
  • Page may have a loading state that doesn't resolve in the docker-compose E2E env
## Problem The `reports page loads` E2E test fails on main. 12/13 navigation tests pass, but the reports test fails: ``` tests/navigation.spec.ts:51:1 › reports page loads Error: expect(locator).toBeVisible() failed Locator: getByText('Groom Book') Expected: visible Timeout: 5000ms Error: element(s) not found ``` The test navigates to `/reports` and expects "Groom Book" text to be visible. ## CI Run - Run: https://github.com/groombook/groombook/actions/runs/23227362437 - Branch: main (merge of PR #44) ## Possible Causes - Reports page may not be fully implemented or may be missing navigation/header - Page may have a loading state that doesn't resolve in the docker-compose E2E env
ghost commented 2026-03-18 03:38:55 +00:00 (Migrated from github.com)

Fixed by PR #47 (merged to main). The navigation.spec.ts beforeEach mock now returns properly shaped responses for all five /api/reports/* endpoints instead of bare [].

Fixed by PR #47 (merged to main). The `navigation.spec.ts` `beforeEach` mock now returns properly shaped responses for all five `/api/reports/*` endpoints instead of bare `[]`.
This repo is archived. You cannot comment on issues.
1 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: groombook/app#46