Skip to content

Fix Playwright tests and add CI validation#9

Draft
kertal wants to merge 1 commit into
mainfrom
claude/fix-playwright-tests-ci-9RU0p
Draft

Fix Playwright tests and add CI validation#9
kertal wants to merge 1 commit into
mainfrom
claude/fix-playwright-tests-ci-9RU0p

Conversation

@kertal

@kertal kertal commented Jan 1, 2026

Copy link
Copy Markdown
Owner

…tion

  • Create new test.yml workflow that runs on PRs and main branch pushes

  • Install Playwright browsers and dependencies in CI

  • Run unit tests (Vitest) with --run flag for CI mode

  • Run E2E tests (Playwright) after build step

  • Upload Playwright test reports as artifacts for debugging

  • Optimize Playwright config for CI:

    • Run tests only on Chromium in CI for speed and reliability
    • Keep all browsers (Chromium, Firefox, WebKit, Mobile) for local testing
    • Existing CI settings preserved (workers: 1, retries: 2, forbidOnly)
  • Add test status badge to README for visibility

  • Create branch protection setup documentation

    • Guide for configuring required status checks
    • Ensures tests must pass before PR merge

This ensures all Playwright tests run automatically on PRs before they can be merged, improving code quality and catching regressions early.

…tion

- Create new test.yml workflow that runs on PRs and main branch pushes
- Install Playwright browsers and dependencies in CI
- Run unit tests (Vitest) with --run flag for CI mode
- Run E2E tests (Playwright) after build step
- Upload Playwright test reports as artifacts for debugging

- Optimize Playwright config for CI:
  - Run tests only on Chromium in CI for speed and reliability
  - Keep all browsers (Chromium, Firefox, WebKit, Mobile) for local testing
  - Existing CI settings preserved (workers: 1, retries: 2, forbidOnly)

- Add test status badge to README for visibility
- Create branch protection setup documentation
  - Guide for configuring required status checks
  - Ensures tests must pass before PR merge

This ensures all Playwright tests run automatically on PRs before they can be merged,
improving code quality and catching regressions early.
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.

2 participants