[E2E][QIT] Add remaining merchant E2E test specs to QIT test-package#11221
Merged
[E2E][QIT] Add remaining merchant E2E test specs to QIT test-package#11221
Conversation
Migrates 8 shopper test specs to QIT: My Account: - My account payment methods add failures - My account saved cards management Multicurrency & Pay for Order: - Multi-currency checkout - Multi-currency widget - Pay for order flow Alternative Payment Methods: - Alipay checkout purchase - Klarna checkout purchase - BNPLs (Buy Now Pay Later) checkout Tests cover shopper flows for account management, currency switching, and alternative payment methods.
- Alipay: Disable multi-currency before tests (like Klarna does) - Alipay: Use URL check for redirect instead of page text - Pay for order: Handle both 'card was declined' and 'payment was not processed' errors
Alipay payments require specific Stripe account configuration that is not available in the QIT test environment. The payment method can be enabled in settings but checkout fails with 'Invalid or missing payment details'.
The QIT devtools implementation uses WP-CLI to set options directly, but the card testing protection feature relies on filters/hooks from the Dev Tools plugin (option_wcpay_account_data, woocommerce_payments_account_refreshed) that aren't available in the QIT environment. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
The dev-trunk version of qit-cli requires PHP 7.4+, which conflicts with the config.platform.php setting of 7.3. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Contributor
Author
|
Hi 👋 @Automattic/gamma @Automattic/masamune gentle ping on this PR in case it got buried. |
frosso
reviewed
Jan 8, 2026
Contributor
frosso
left a comment
frosso
left a comment
There was a problem hiding this comment.
No major concerns.
However, the tests/woopayments/merchant/non-admin-wp-admin-access.spec.ts test consistently fails, even when run singularly.
tests/qit/test-package/tests/woopayments/merchant/merchant-admin-deposits.spec.ts
Outdated
Show resolved
Hide resolved
Contributor
Author
Thanks for the review @frosso!. This is now fixed. QIT report: https://qit.woo.com/?qit_results=4951235.UMK8Hbnq7yeVLWeZ4LEZOvDsq0VCpEgEvCyomNtHe4GXSGnl4XEMWPxTUxRq24rX FYI It was also failing in the non-QIT E2E test for the same reason: a recent server-side change, you can find more details here #11241 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follows up on #11209
Changes proposed in this Pull Request
This PR completes the merchant E2E test migration by adding the remaining 12 specs to the QIT test-package infrastructure.
Why is this needed?
PR #11209 added the first batch of 7 merchant specs (order-related tests). This PR adds the remaining 12 specs as mentioned in that PR: disputes, multi-currency, analytics, and woopay tests.
Specs included in this PR:
merchant-admin-analytics.spec.ts- Analytics page testsmerchant-admin-deposits.spec.ts- Deposits page testsmerchant-admin-disputes.spec.ts- Disputes listing testsmerchant-admin-transactions.spec.ts- Transactions page testsmerchant-disputes-respond.spec.ts- Dispute response workflow testsmerchant-disputes-view-details-via-order-notice.spec.ts- Dispute details via order noticemerchant-multi-currency-widget.spec.ts- Multi-currency widget testsmulti-currency-on-boarding.spec.ts- Multi-currency onboarding flowmulti-currency-setup.spec.ts- Multi-currency setup testsmulti-currency.spec.ts- General multi-currency testsnon-admin-wp-admin-access.spec.ts- Non-admin user access testswoopay-setup.spec.ts- WooPay setup testsAdditional improvements:
utils/merchant.tsfor reliabilityTesting instructions
Refer to tests/qit/README.md for setup instructions.
All tests should pass ✅
npm run changelogto add a changelog file, choosepatchto 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
🤖 Generated with Claude Code