Playwright smoke tests across critical authenticated pages

[bokelley]

Context

PR #1895 added Playwright smoke tests for the profile edit page, catching a class of bug (DOM restructure dropping elements) that unit tests alone missed. The same pattern — dev login, verify rendering, test key interactions, check for console errors — should cover other pages with complex client-side JS.

Pages to cover (priority order)

Page	Route	Why	Dev user
Membership hub	/membership/hub	Billing data integration, subscription status, multiple account types	member, personal, admin
Organization dashboard	/organization	Team management, seat allocation, org-scoped views	admin, member
Certification dashboard	/certification	Progress tracking, credential display, LinkedIn sharing	personal, admin
Brand viewer	/brand/:domain	Registry data rendering, identity display	admin
Community hub	/community	Connection management, activity feed, gamification	personal, member
Admin digest	/admin/digest	Newsletter editor, section management	admin

Pattern

Each test follows the same structure (see tests/e2e/profile-edit.smoke.js):

1. Login via /dev-login.html as the relevant dev user
2. Navigate to the page, wait for content to load
3. Verify critical DOM elements exist
4. Test key interactions (save, toggle, submit)
5. Check for console errors (filtering out expected noise like PostHog/network)
6. Screenshot on failure

Shared infrastructure to build

• Test helper: Extract login, error collection, and assertion utilities from profile-edit.smoke.js into a shared tests/e2e/helpers.js
• Runner script: npm run test:e2e that starts docker, waits for health, runs all smoke tests, tears down
• CI integration (optional): GitHub Action that spins up the dev container and runs smoke tests on PRs that touch server/public/

How to run today

docker compose up --build -d
npx playwright install chromium  # first time only
node tests/e2e/profile-edit.smoke.js

References

• fix: prevent profile save crash and portrait widget error #1869 — Profile save crash (the bug that motivated this)
• feat: extract profile-edit inline JS into testable module #1891 — JS extraction with jsdom unit tests
• feat: add Playwright smoke tests for profile edit page #1895 — First Playwright smoke test (profile edit)

[bokelley]

Triage

Classification: Feature request
Bucket(s): admin/ops tools
Status: drafting-pr

What the experts said:

• adtech-product-expert: Execute — pattern proven by feat: add Playwright smoke tests for profile edit page #1895, all 6 pages confirmed in server/public/, no protocol impact, changeset --empty
• dx-expert: Execute with two pre-flight clarifications (runner model matches existing node pattern; /admin/digest is a 301 redirect to /admin/newsletters/the_prompt — PR uses canonical URL)
• code-reviewer: two blockers found and fixed — resetFailures() inside loops was silently discarding failures; unused resp variable in admin-newsletter test

My take: Draft PR #3335 opened. Note: npm run test:e2e runner script (per the issue) requires a package.json entry that cannot be added in this PR — tests run as node tests/e2e/<file>.smoke.js today, matching the existing profile-edit pattern. CI integration is a deliberate follow-on.

---

Triaged by Claude Code. Session: https://claude.ai/code/session_016dJ4nMXCbcLEmCsCNrBK6d

---

Generated by Claude Code