Skip to content

chore: add Playwright E2E testing infrastructure#673

Merged
cjpais merged 6 commits intocjpais:mainfrom
joshribakoff:feat/playwright-testing
Feb 6, 2026
Merged

chore: add Playwright E2E testing infrastructure#673
cjpais merged 6 commits intocjpais:mainfrom
joshribakoff:feat/playwright-testing

Conversation

@joshribakoff
Copy link
Contributor

@joshribakoff joshribakoff commented Jan 26, 2026

Before Submitting This PR

Please confirm you have done the following:

  • I have searched existing issues and pull requests to ensure this isn't a duplicate
  • I have read CONTRIBUTING.md

Human Written Description

Adding Playwright for verifying the UI works correctly (integration style test).

This is the infrastructure with a hello world test.

I have follow up PRs ready to go that backfill tests.

Related Issues/Discussions

Foundation for better test coverage. No specific issue - proactive improvement.

Testing

  • bun run test:e2e runs tests headless
  • bun run test:e2e:ui opens Playwright UI for debugging
  • CI workflow runs on every PR

AI Assistance

  • AI was used (please describe below)

If AI was used:

  • Tools used: Claude Code
  • How extensively: Wrote the Playwright config, test file, and CI workflow

- Add @playwright/test dependency
- Add playwright.config.ts with Chromium setup
- Add test scripts (test, test:ui, test:headed)
- Add hello world test to verify setup works

This is the foundation for adding comprehensive E2E tests.
Tests run against the Vite dev server on port 1420.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@joshribakoff joshribakoff force-pushed the feat/playwright-testing branch from 786bb43 to 88a3e7f Compare January 26, 2026 02:03
@joshribakoff joshribakoff changed the title feat: add Playwright E2E testing infrastructure WIP: add Playwright E2E testing infrastructure Jan 26, 2026
joshribakoff and others added 2 commits January 25, 2026 18:13
Runs Playwright tests on pull requests using Chromium.
Uploads test artifacts on failure.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Rename workflow from "E2E Tests" to "Playwright"
- Rename scripts from test:e2e to test:playwright
- Fix test to not require Tauri APIs (just check server responds)
- Run prettier

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@joshribakoff joshribakoff changed the title WIP: add Playwright E2E testing infrastructure chore: add Playwright E2E testing infrastructure Jan 26, 2026
@joshribakoff joshribakoff marked this pull request as ready for review January 26, 2026 02:52
@cjpais
Copy link
Owner

cjpais commented Feb 1, 2026

Thank you, I will get this in soon

@cjpais cjpais merged commit 9ef7104 into cjpais:main Feb 6, 2026
4 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.

2 participants