-
Notifications
You must be signed in to change notification settings - Fork 0
Flesh out E2E Playwright tests and run them in CI #35
Copy link
Copy link
Open
Labels
enhancementNew feature or requestNew feature or request
Description
Priority: P2
Source
Senior Architect Code Review (2026-04-03)
Problem
Four Playwright E2E spec files exist under apps/dashboard/tests/e2e/ but they are minimal stubs (284-420 bytes each). More critically, E2E tests are not run in CI — the CI workflow only runs bun run test:dashboard (Vitest unit tests), not bun run test:e2e (Playwright).
The Playwright config is set up but the specs need to be fleshed out to provide actual browser-level regression coverage.
Affected Files
apps/dashboard/tests/e2e/dashboard.spec.tsapps/dashboard/tests/e2e/detail.spec.tsapps/dashboard/tests/e2e/map.spec.tsapps/dashboard/tests/e2e/performance.spec.tsapps/dashboard/playwright.config.ts.github/workflows/ci.yml
Recommendation
-
Flesh out the E2E specs to cover critical user journeys:
- Wallet connect / disconnect flow
- Turret card selection and detail panel interaction
- Network node drawer open/close
- Filter application and clearing
- Demo mode navigation and rendering
-
Add a Playwright CI job that:
- Starts the dev server (or builds and serves)
- Runs E2E tests against the running app
- Uploads trace artifacts on failure
Acceptance Criteria
- E2E specs cover at least 5 critical user journeys
- CI has a dedicated E2E job that runs Playwright tests
- Test traces are uploaded as artifacts on failure
- E2E tests pass reliably (no flaky timing assertions)
- Playwright browsers are cached in CI
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request