Marc-André Bombeck
594898d350
The suggestion anchored on the document's own date, which the DTO carries as a date only. The suggest endpoint takes a full instant, so every document asked the question as `2025-10-14` and got 422 back; the field then rendered nothing and no document ever offered the visit its date falls near. Lift the date to noon UTC before asking, the shape the two lab moments already send. The e2e that should have caught it never authenticated: the visits spec carried no storage state, so its `page` redirected to login and its `request` reads answered 401. It also asserts a global verdict — one visit in the document's window — so clear the account's visits before each test, run the file serially, and keep it to one project, since two projects mutating the shared account would leave two candidates in the window. The accessibility mock lagged the same feature: the detail sheet reads `encounterLinks` off the document and the mock omitted it, so the sheet threw before it painted. Add the field the server always sends.
e2e/
End-to-end browser tests. Playwright 1.60 drives a production build of the app against a real PostgreSQL, with @axe-core/playwright for accessibility assertions. This is the heaviest test tier; CI runs it on every PR, and locally it is worth running when a change touches a UI flow.
Running
pnpm e2e
The Playwright config (../playwright.config.ts) builds the app (next build) and brings up Postgres before the specs run; setup/global-setup.ts seeds the auth + fixture state. The suite is not part of the fast local gate (typecheck / lint / test) — it surfaces at PR CI, so expect responsive/wizard/nav reworks to flag here first.
Layout
setup/—test.ts(the suite'stest+expectentry point),global-setup.ts(auth + DB seeding),test-helpers.ts(shared login + navigation helpers).utils/— fixtures such asmock-dashboard-snapshot.ts.*.spec.ts— one spec per flow: auth/login/redirect, dashboard, charts, the medication-wizard cadences (medications-wizard-*), insights generation + scroll restoration, doctor report, settings, mobile-viewport consistency, locale switch, accessibility (a11y.spec.ts), and thev1427-*regression set.
Conventions
Import test and expect from ./setup/test, never from @playwright/test — types (Page, Route, …) still come from the package. The entry point extends Playwright's test with an automatic fixture that drops route handlers and page listeners after every test, so a handler left in flight cannot fail the test it belonged to once the context closes. src/__tests__/e2e-route-cleanup-guard.test.ts enforces the import boundary.
Assert against stable data-* attributes (data-slot, data-display-step, root display attrs), not viewport-dependent visible text. Responsive sm:hidden UI changes break getByText on desktop while the underlying element is still present — anchoring on data attributes keeps the specs stable across breakpoints. See ../CLAUDE.md.