98939877db
* feat: comprehensive Playwright E2E test rewrite Rewrite all E2E tests with correct CSS selectors, add new spec files, and implement robust auth handling to work within backend rate limits. Changes: - Rewrite fixtures/index.ts with JWT-based /auth/me mock to avoid 10 req/min rate limit on /auth/me during test runs - Rewrite auth.setup.ts with offline JWT validity check to reuse existing auth state across runs (saves login rate-limit budget) - Rewrite auth.spec.ts (6 tests) - login page, fields, submit, redirect guard, invalid credentials, login/register toggle - Rewrite dashboard.spec.ts (8 tests) - header, nav tabs, navigation, overview/schedules sections, days selector, redirect - Rewrite medications.spec.ts (8 tests) - form fields, stock inventory, package type toggle, intake schedule, save/cancel, unsaved changes guard - Rewrite settings.spec.ts (12 tests) - language, notification matrix, thresholds, calculation mode, toggle switch, export/import, user menu navigation - Create planner.spec.ts (9 tests) - form, date inputs, calculate, reset, checkbox, submit, tab state, eyebrow heading - Create schedule.spec.ts (12 tests) - timeline, days selector, past/future toggles, day blocks, today highlight, collapse/expand, overview table, share button - Update playwright.config.ts: remove mobile projects, enable webServer section for CI - Add .github/workflows/e2e.yml CI workflow for Playwright tests Total: 57 E2E tests across 6 spec files, all passing consistently across 5+ consecutive runs without backend restart. Closes #154 * feat: add comprehensive E2E data tests with medication CRUD, dashboard, planner, schedule Add 48 new Playwright E2E tests covering real medication data scenarios: - medication-crud: 14 tests for create/edit/delete/list via UI form - dashboard-data: 13 tests for overview table, timeline, dose tracking - planner-data: 9 tests for demand calculator with results/status chips - schedule-data: 11 tests for timeline, collapse/expand, dose mark/undo Infrastructure improvements: - Add API helpers (createMedicationViaAPI, deleteMedicationViaAPI, deleteAllMedicationsViaAPI) with retry logic for rate-limit resilience - Configure chromium-data project for serial execution with retry:1 - Add /auth/me mock to avoid rate-limit exhaustion on auth endpoint - Increase navigateTo reliability with networkidle waits - Increase auth token validity threshold from 2 to 10 minutes - Make backend rate limit configurable via RATE_LIMIT_MAX env var - Set RATE_LIMIT_MAX=300 in dev docker-compose for E2E test support Total suite: 57 empty-state + 48 data tests = 105 tests (chromium) * test: add E2E tests for medication editing, stock status, and share schedule - medication-edit.spec.ts: 10 tests covering generic name, notes, taken-by add/remove, expiry date, refill, intake schedule editing, adding intake rows, reminder toggle, and package type changes - stock-status.spec.ts: 12 tests verifying dashboard shows correct status chips (High/Normal/Warning/Danger) for different stock levels, overview table, reorder card, detail modal, and planner integration - share-schedule.spec.ts: 10 tests for taken-by badges, share button, share dialog, link generation, shared schedule page navigation, dose tracking on shared page, and notes display - fixtures/index.ts: add createShareTokenViaAPI, updateSettingsViaAPI helpers; expand createMedicationViaAPI with takenBy, notes, expiryDate - playwright.config.ts: update testMatch/testIgnore for new test files - docker-compose.dev.yml: increase RATE_LIMIT_MAX to 1000 for E2E tests * docs: refine release-manager instructions for CLI safety and commit-linked release notes * fix: resolve PR155 CI failures for frontend lint and e2e proxy * fix: stabilize auth-related e2e checks in CI
143 lines
4.9 KiB
TypeScript
143 lines
4.9 KiB
TypeScript
import { expect } from "@playwright/test";
|
|
import { authFile, navigateTo, test } from "./fixtures";
|
|
|
|
/**
|
|
* Medications Page E2E Tests
|
|
*
|
|
* Verifies the medication list, add/edit form, CRUD operations,
|
|
* and form validation.
|
|
*/
|
|
test.describe("Medications Page", () => {
|
|
test.use({ storageState: authFile });
|
|
|
|
test("should display medications page", async ({ page }) => {
|
|
await navigateTo(page, "/medications");
|
|
|
|
// Medications tab should be active
|
|
await expect(page.locator('button.pill.primary:has-text("Medications")')).toBeVisible();
|
|
});
|
|
|
|
test("should show medication list or empty state", async ({ page }) => {
|
|
await navigateTo(page, "/medications");
|
|
|
|
// Should show either medication entries or the new medication form
|
|
const listTitle = page.locator("h2").filter({ hasText: /Medication list/i });
|
|
const formTitle = page.locator("h2").filter({ hasText: /New medication/i });
|
|
|
|
const hasList = await listTitle.isVisible().catch(() => false);
|
|
const hasForm = await formTitle.isVisible().catch(() => false);
|
|
|
|
expect(hasList || hasForm).toBeTruthy();
|
|
});
|
|
|
|
test("should display the medication form with required fields", async ({ page }) => {
|
|
await navigateTo(page, "/medications");
|
|
|
|
// The form should always be visible on the medications page
|
|
const commercialName = page.getByLabel(/Commercial Name/i);
|
|
await expect(commercialName).toBeVisible();
|
|
|
|
// Package type selector should exist
|
|
await expect(page.getByText(/Package Type/i)).toBeVisible();
|
|
|
|
// Intake schedule section should exist
|
|
await expect(page.getByText(/Intake schedule/i)).toBeVisible();
|
|
});
|
|
|
|
test("should fill in medication details", async ({ page }) => {
|
|
await navigateTo(page, "/medications");
|
|
|
|
const nameField = page.getByLabel(/Commercial Name/i);
|
|
await nameField.fill("Test Aspirin");
|
|
await expect(nameField).toHaveValue("Test Aspirin");
|
|
|
|
const genericField = page.getByLabel(/Generic Name/i);
|
|
await genericField.fill("Acetylsalicylic acid");
|
|
await expect(genericField).toHaveValue("Acetylsalicylic acid");
|
|
});
|
|
|
|
test("should have stock inventory fields", async ({ page }) => {
|
|
await navigateTo(page, "/medications");
|
|
|
|
// Stock fields should be visible
|
|
await expect(page.getByLabel(/^Packs$/i)).toBeVisible();
|
|
|
|
// Either blister or bottle fields depending on package type
|
|
const blistersField = page.getByLabel(/Blisters per pack/i);
|
|
const pillsField = page.getByLabel(/Pills per blister/i);
|
|
const capacityField = page.getByLabel(/Total Capacity/i);
|
|
|
|
const hasBlister = await blistersField.isVisible().catch(() => false);
|
|
const hasBottle = await capacityField.isVisible().catch(() => false);
|
|
|
|
expect(hasBlister || hasBottle).toBeTruthy();
|
|
});
|
|
|
|
test("should toggle package type between blister and bottle", async ({ page }) => {
|
|
await navigateTo(page, "/medications");
|
|
|
|
// Find the package type radio buttons or selector
|
|
const blisterOption = page.getByText(/Blister Pack/i);
|
|
const bottleOption = page.getByText(/Pill Bottle/i);
|
|
|
|
if (await blisterOption.isVisible().catch(() => false)) {
|
|
// Switch to bottle
|
|
await bottleOption.click();
|
|
// Bottle-specific fields should appear
|
|
await expect(page.getByLabel(/Total Capacity/i)).toBeVisible();
|
|
|
|
// Switch back to blister
|
|
await blisterOption.click();
|
|
await expect(page.getByLabel(/Blisters per pack/i)).toBeVisible();
|
|
}
|
|
});
|
|
|
|
test("should have intake schedule with add button", async ({ page }) => {
|
|
await navigateTo(page, "/medications");
|
|
|
|
// Intake schedule section
|
|
const scheduleSection = page.getByText(/Intake schedule/i);
|
|
await expect(scheduleSection).toBeVisible();
|
|
|
|
// Should have at least one intake entry
|
|
await expect(page.getByText(/Usage \(pills\)|Every \(days\)/i).first()).toBeVisible();
|
|
|
|
// Should have an add intake button
|
|
const addIntake = page.getByRole("button", { name: /Intake/i });
|
|
await expect(addIntake).toBeVisible();
|
|
});
|
|
|
|
test("should have save and cancel buttons", async ({ page }) => {
|
|
await navigateTo(page, "/medications");
|
|
|
|
// Fill in a name to make the form dirty
|
|
await page.getByLabel(/Commercial Name/i).fill("Test");
|
|
|
|
// Save button
|
|
const saveButton = page.getByRole("button", { name: /Save|Add Medication/i });
|
|
await expect(saveButton).toBeVisible();
|
|
});
|
|
|
|
test("should prevent navigation with unsaved changes", async ({ page }) => {
|
|
await navigateTo(page, "/medications");
|
|
|
|
// Fill in the form to create unsaved changes
|
|
await page.getByLabel(/Commercial Name/i).fill("Unsaved Medication");
|
|
|
|
// Try to navigate away
|
|
await page.locator('button.pill:has-text("Dashboard")').click();
|
|
|
|
// Should show unsaved changes warning modal
|
|
const modal = page.locator(".confirm-modal-overlay, .modal-overlay");
|
|
const hasWarning = await modal.isVisible().catch(() => false);
|
|
|
|
if (hasWarning) {
|
|
// Cancel to stay on page
|
|
const cancelBtn = page.getByRole("button", { name: /Cancel|Stay/i });
|
|
if (await cancelBtn.isVisible().catch(() => false)) {
|
|
await cancelBtn.click();
|
|
}
|
|
}
|
|
});
|
|
});
|