Skip to content

Add Playwright happy-path E2E#51

Merged
DavidMoritz merged 3 commits into
masterfrom
codex/playwright-e2e-happy-path
Jun 16, 2026
Merged

Add Playwright happy-path E2E#51
DavidMoritz merged 3 commits into
masterfrom
codex/playwright-e2e-happy-path

Conversation

@ejones23

@ejones23 ejones23 commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add Playwright config and scripts for a create -> vote -> results happy-path E2E
  • create and tear down an isolated MySQL rcv_e2e database for browser tests
  • add inert data-testid hooks to the AngularJS templates for stable E2E selectors
  • harden the harness for future CI use with run-specific DB names, ports, and cache directories
  • scope the test to active views/forms and assert against dedicated candidate-name and winner elements to reduce flake risk

Verification

  • npm run test:e2e
  • npm test
  • npm run build
  • ran the Playwright happy-path 10 times in two concurrent batches of 5; all 10 passed

@ejones23 ejones23 marked this pull request as ready for review June 16, 2026 03:49
@ejones23 ejones23 force-pushed the codex/playwright-e2e-happy-path branch from 9e92c57 to b38f739 Compare June 16, 2026 03:58

@DavidMeritas DavidMeritas left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work on this, ejones! Really well thought out — the run isolation with per-run DB names, ports, and cache dirs is exactly the kind of thing that saves headaches down the road in CI. The data-testid approach for selectors is solid, the SQL injection guards are a nice touch, and the whole create → vote → results happy path reads cleanly. Love the E2E_KEEP_DB=1 escape hatch for debugging too. Thanks for continuing to level up the test infrastructure — this project is in much better shape because of your contributions. 🎉

@DavidMoritz DavidMoritz merged commit 516aa84 into master Jun 16, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants