-
Notifications
You must be signed in to change notification settings - Fork 16
Replace Cypress with Playwright for e2e testing #62
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
Conversation
chore: add semantic-release
chore: ci build and release
9b422ea to
c550b35
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR migrates the project from Cypress to Playwright for end-to-end testing and adds semantic-release for automated versioning and publishing.
Key changes:
- Replaced Cypress with Playwright for E2E testing, including updated configuration, test scripts, and a sample test
- Integrated semantic-release with GitHub Actions for automated versioning and NPM publishing
- Updated CI/CD workflow to include build, test, and release steps with appropriate permissions
Reviewed Changes
Copilot reviewed 13 out of 15 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| playwright.config.js | Adds Playwright configuration targeting the showcase dev server on port 5000 |
| package.json | Replaces Cypress dependencies with Playwright and semantic-release packages; updates test scripts |
| e2e/showcase.spec.js | Creates new Playwright test file with basic showcase page load test |
| cypress/* | Removes all Cypress configuration and test files |
| .releaserc.json | Adds semantic-release configuration for main, next, and version branches |
| .gitignore | Replaces Cypress output directories with Playwright equivalents |
| .github/workflows/cicd.yml | Expands workflow to run on push to release branches, adds E2E tests and semantic-release steps |
| .eslintrc.js | Removes Cypress ESLint plugin and globals |
| README.md | Updates documentation with Playwright testing instructions and semantic-release process |
c9c1e23 to
90d43df
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 13 out of 15 changed files in this pull request and generated 2 comments.
90d43df to
a2f6c00
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 13 out of 15 changed files in this pull request and generated 1 comment.
Co-authored-by: dopry <[email protected]>
a2f6c00 to
cc0f96d
Compare
|
🎉 This PR is included in version 1.0.0-next.2 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 1.2.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Cypress cannot navigate across sites, blocking OIDC workflow testing. Playwright supports cross-site navigation.
Changes
Replaced testing framework
Test infrastructure
playwright.config.js- webServer config auto-starts showcase dev servere2e/showcase.spec.js- validates home page loads (initial test scope per requirements)npm test,npm run test:headed,npm run test:uiConfiguration updates
.eslintrc.js- removed Cypress globals and plugin.gitignore- Playwright artifacts instead of CypressREADME.md- added testing instructionsTest implementation
Playwright's webServer eliminates need for start-server-and-test wrapper.
Showcase page under test
Current test scope limited to home page load validation. Framework now supports cross-site navigation for future OIDC flow tests.
Warning
Firewall rules blocked me from connecting to one or more addresses (expand for details)
I tried to connect to the following addresses, but was blocked by firewall rules:
esm.ubuntu.com/usr/lib/apt/methods/https(dns block)If you need me to access, download, or install something from one of these locations, you can either:
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.