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

Replace cypress tests with playwright tests #13571

Merged
merged 5 commits into from
Feb 22, 2024

Conversation

minkyngkm
Copy link
Contributor

@minkyngkm minkyngkm commented Feb 12, 2024

Done

  • Removed all cypress tests and cypress dependences
  • Wrote certified.spec.ts with playwright to replace certified_spec.js written with cypress test.
  • Wrote a11y.spect.ts with playwright to replace a11y_spec.js written with cypress test.
  • Created pro folder in /tests/playwright/tests, moved checkout.spec.ts and a11y.spec.ts to the pro folder, disabled tests in pro folder running.

Fixes https://warthogs.atlassian.net/browse/WD-8876

@webteam-app
Copy link

Demo starting at https://ubuntu-com-13571.demos.haus

Copy link

codecov bot commented Feb 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a9058fa) 74.41% compared to head (f051915) 74.41%.
Report is 38 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #13571   +/-   ##
=======================================
  Coverage   74.41%   74.41%           
=======================================
  Files         107      107           
  Lines        2838     2838           
  Branches      946      946           
=======================================
  Hits         2112     2112           
  Misses        702      702           
  Partials       24       24           

@minkyngkm minkyngkm changed the title Revert cypress tests Replace cypress tests with playwright tests Feb 19, 2024
@jpmartinspt jpmartinspt merged commit 87662a8 into canonical:main Feb 22, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants