-
Notifications
You must be signed in to change notification settings - Fork 68
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
Convert shopper checkout save card and purchase test to Playwright #10208
Convert shopper checkout save card and purchase test to Playwright #10208
Conversation
Test the buildOption 1. Jetpack Beta
Option 2. Jurassic Ninja - available for logged-in A12s🚀 Launch a JN site with this branch 🚀 ℹ️ Install this Tampermonkey script to get more options. Build info:
Note: the build is updated when a new commit is pushed to this PR. |
Size Change: 0 B Total Size: 1.36 MB ℹ️ View Unchanged
|
…eckout-save-card-and-purchase
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.
Thanks for working on this one, Taha. The code looks great. Tests are passing locally and in GH actions. The new Playwright spec is covering all the tests that the old Puppeteer spect covered.
I found a typo in one of the test name strings. I mentioned it inline. Once that is updated, it will be good to ship.
Pre-approving.
Fixes #10066
Changes proposed in this Pull Request
This PR converts the shopper checkout save card and purchase E2E test to Playwright and deletes the Puppeteer one.
Testing instructions
npm run test:e2e-pw save-card-and-purchase
and expect the tests to succeed.npm run changelog
to add a changelog file, choosepatch
to leave it empty if the change is not significant. You can add multiple changelog files in one PR by running this command a few times.Post merge