Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: introduce playwright e2e tests #28178

Open
wants to merge 11 commits into
base: chore/measure-backend-on-depot
Choose a base branch
from

Conversation

pauldambra
Copy link
Member

@pauldambra pauldambra commented Feb 1, 2025

very basic copy from #23248

let's get the smallest set running that we can

(accidentally stacked on top of #28177 🫠)

@pauldambra pauldambra changed the title chore: measure backend on depot chore: introduce playwright e2e tests Feb 1, 2025
@pauldambra pauldambra changed the base branch from master to chore/measure-backend-on-depot February 1, 2025 14:13
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

This PR introduces comprehensive E2E testing infrastructure changes, migrating from Cypress to Playwright and moving backend CI to Depot runners for improved performance measurement.

  • Switches CI backend jobs to use depot-ubuntu-latest runners in .github/workflows/ci-backend.yml for performance measurement
  • Adds new Playwright E2E testing framework with authentication setup, page objects, and test specs in /playwright/ directory
  • Introduces race condition risk in bin/e2e-test-runner with duplicate code blocks and background process handling
  • Configures Playwright for CI environment with specific Chrome launch arguments and authentication state management in playwright.config.ts
  • Creates new CI workflow .github/workflows/ci-e2e-playwright.yml with container image caching and test execution on Depot

10 file(s) reviewed, 16 comment(s)
Edit PR Review Bot Settings | Greptile

playwright.config.ts Show resolved Hide resolved
bin/e2e-test-runner Show resolved Hide resolved
.github/workflows/ci-e2e-playwright.yml Outdated Show resolved Hide resolved
bin/e2e-test-runner Outdated Show resolved Hide resolved
playwright/auth.setup.ts Show resolved Hide resolved
playwright/utils.ts Show resolved Hide resolved
playwright/utils.ts Show resolved Hide resolved
playwright/shared/navigation.ts Show resolved Hide resolved
playwright/shared/navigation.ts Show resolved Hide resolved
playwright/shared/navigation.ts Show resolved Hide resolved
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