mirror of
https://github.com/MBombeck/HealthLog.git
synced 2026-11-07 04:12:04 +00:00
Marc-André Bombeck
038f8b10cd
Every instrument now runs the same wizard: the form's own question stem above each item (the PHQ-9 and GAD-7 stems were missing), the item's exact answer options as one radio group in source order, progress, Back, number-key answers, and a review step that lists every answer, lets any of them be changed and is the only step that submits. The result leads with the crisis card when it applies, then shows the score on the instrument's scale, the band, what the band means, the change since the last check-in against the meaningful-change threshold, and the next step. Scoring is untouched. The wording was read against the published versions. German follows the PHQ-D and Löwe's GAD-7 verbatim, including the follow-up question in its published formal address. Spanish, Italian and Polish carry the official PHQ-9 and GAD-7 translations, French the official GAD-7, and the GAD-7 gets its own answer set because the Italian forms word it differently. Where a version could not be matched to a published form (the French PHQ-9, every Korean form) the language now shows the English original. Each locale list names its sources, and spot checks pin the corrected wording.
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.fixtures/— synthetic payloads a journey uploads, built in code rather than committed as blobs so the content stays reviewable (apple-health-export.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.