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

dev: add Playwright as a testing tool #2589

Closed
mastercactapus opened this issue Aug 24, 2022 · 0 comments · Fixed by #2608
Closed

dev: add Playwright as a testing tool #2589

mastercactapus opened this issue Aug 24, 2022 · 0 comments · Fixed by #2608
Labels
enhancement New feature or request

Comments

@mastercactapus
Copy link
Member

What problem would you like to solve? Please describe:

  • Flaky Cypress selector chains
  • Inability to test popup links & separate servers/hosts (e.g., oauth flows)
  • Cypress upgrades frequently break tests & require manual updates
  • Layout of tests & locations isn't documented
  • Cypress test code needs to be built/bundled to run in the browser
  • Inability to use things like async/await

Describe the solution you'd like:

  • Move smoketests to tests/smoketest
  • Create tests/integration for playwright tests
  • Add test-integration and start-integration make targets (prod-mode only)
  • Add EMAIL contact method as first test
  • Add test-integration target to build-all script in CI

Describe alternatives you've considered:
The current setup works for what it can do, and there are workarounds like setting up reverse proxy routes to get around some of Cypress's limitations; but it all adds complexity, time, and maintenance cost.

Playwright was evaluated and has a preferable setup, better integration with existing IDEs, and more flexibility to test multiple tabs/windows & concurrently run tests.

Additional context:
Playwright

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant