Skip to content

chore: add E2E testing, CI pipeline, and improve coverage#14

Merged
johnpc merged 1 commit into
mainfrom
chore/testing-infra
May 24, 2026
Merged

chore: add E2E testing, CI pipeline, and improve coverage#14
johnpc merged 1 commit into
mainfrom
chore/testing-infra

Conversation

@johnpc
Copy link
Copy Markdown
Owner

@johnpc johnpc commented May 24, 2026

Summary

  • Add Playwright + playwright-bdd for Gherkin-based E2E tests
  • Add GitHub Actions CI workflow that blocks PRs on test/build/e2e failure
  • Add AttendeeNamesContext test suite (9 tests)
  • Add 5 new EditNameModal tests (keyboard, 64-char limit, overlay dismiss, state reset)
  • Pre-commit hook now runs tests + build (not just lint-staged)
  • test:coverage includes CRAP score analysis
  • Coverage thresholds set to 80%

Results

  • 113 unit tests passing
  • All 82 functions have CRAP score ≤ 15
  • Coverage above 80% thresholds
  • Build clean

Test plan

  • npm run test:coverage passes
  • npm run build passes
  • CI workflow triggers on PR

🤖 Generated with Claude Code

- Add Playwright + playwright-bdd for Gherkin E2E tests
- Add landing page feature file and step definitions
- Add GitHub Actions CI workflow (test + coverage + build + e2e)
- Add tests for AttendeeNamesContext (9 tests)
- Add tests for EditNameModal keyboard/validation (5 new tests)
- Update pre-commit hook to run tests + build
- Set coverage thresholds to 80% (matches project standard)
- All 82 functions pass CRAP score ≤ 15

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@johnpc johnpc enabled auto-merge (squash) May 24, 2026 19:15
@johnpc johnpc merged commit 346555d into main May 24, 2026
1 check 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.

1 participant