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

Port Cypress tests to Playwright #1879

Closed
tijlleenders opened this issue Apr 7, 2024 · 2 comments · Fixed by #1942
Closed

Port Cypress tests to Playwright #1879

tijlleenders opened this issue Apr 7, 2024 · 2 comments · Fixed by #1942

Comments

@tijlleenders
Copy link
Owner

Is your feature request related to a problem? Please describe.
It's very annoying when we have two frameworks for tests when we can have one.

Describe the solution you'd like
Port all the Cypress to Playwright and delete Cypress dependency.

Describe alternatives you've considered
None.

Additional context
None.

@tijlleenders tijlleenders moved this to temp to drag faster in ZinZen® Apr 7, 2024
@tijlleenders tijlleenders moved this from temp to drag faster to Parking lot in ZinZen® Apr 26, 2024
@tijlleenders
Copy link
Owner Author

@vinaybadgujar102 I think we don't have any more tests in Cypress, right?
Then we only need a PR to remove all Cypress mentions and dependencies to close this issue.

@vinaybadgujar102
Copy link
Collaborator

@tijlleenders Some tests are remaining. In #1929, only the failing test was migrated. Migrating all the tests in that PR would have made it unrelated and hard to review.

@vinaybadgujar102 vinaybadgujar102 self-assigned this Jun 26, 2024
@vinaybadgujar102 vinaybadgujar102 moved this from Parking lot to In progress in ZinZen® Jun 26, 2024
@github-project-automation github-project-automation bot moved this from In progress to Done in ZinZen® Jul 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants