-
Notifications
You must be signed in to change notification settings - Fork 72
[E2E][QIT] Migrate WooCommerce Blocks checkout specs #11146
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
Migrates 3 WooCommerce Blocks checkout test specs to QIT: - Blocks checkout failures - Blocks checkout purchase - Blocks saved card checkout and usage Tests cover WooCommerce Blocks-based checkout flows including the new checkout experience and saved payment methods.
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: 876 kB ℹ️ View Unchanged
|
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 WooCommerce Blocks checkout end-to-end tests to the QIT (Quality Insights Toolkit) framework, part of a larger migration effort.
Key Changes:
- Added three new WC Blocks checkout test specs with comprehensive test coverage
- Tests cover successful purchases (basic and 3DS cards), checkout failures with various card types, and saved card functionality
- Added corresponding changelog entry
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| tests/qit/e2e/specs/woopayments/shopper/shopper-wc-blocks-checkout-failures.spec.ts | Adds comprehensive failure scenario tests for WC Blocks checkout with various declined card types |
| tests/qit/e2e/specs/woopayments/shopper/shopper-wc-blocks-checkout-purchase.spec.ts | Adds successful purchase tests for WC Blocks checkout using basic and 3DS cards |
| tests/qit/e2e/specs/woopayments/shopper/shopper-wc-blocks-saved-card-checkout-and-usage.spec.ts | Adds tests for saving, using, and deleting payment methods in WC Blocks checkout |
| changelog/dev-qit-e2e-shopper-blocks | Documents the test migration as a development change with no production impact |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
dmvrtx
left a comment
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.
Specs work as intended! Great work!
|
@mgascam , there are odd CI failures. Is it related to the version compatibility? |
|
On a separate note, if I run |
Thanks for testing @dmvrtx. In theory, the order in which the tests are run should not affect since QIT spins up a new docker environment and the test site is recreated every time the tests run. I tried on my machine:
All tests passed for me. Would you mind to try again? Maybe is the same issue we saw that was solved refreshing the Jetpack tokens? In any case, taking a look to the qit report would be great if it keeps failing. |
@dmvrtx Looks like there was a one time error, I retried the QIT Security and Malware Tests and it worked fine this time. |
|
Thanks for checking, @mgascam. Can confirm it works now for me as well! |
Part 2 of 5: WC Blocks Checkout
Test Coverage
Testing
The site that QIT spins needs an onboarded account. Follow this steps to get the JetPack tokens:
tests/qit/config/local.envwith those values. You can usetests/qit/config/default.envas a templateE2E_JP_SITE_IDE2E_JP_BLOG_TOKENE2E_JP_USER_TOKENRun the tests
All tests should pass ✅
Next Steps
Subsequent PRs will add more test specs: