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

[E2E] WooCommerce Blocks tests fail with WP Nightly #10345

Closed
mgascam opened this issue Feb 10, 2025 · 2 comments
Closed

[E2E] WooCommerce Blocks tests fail with WP Nightly #10345

mgascam opened this issue Feb 10, 2025 · 2 comments
Labels
category: e2e Issues and PRs related to e2e tests. e2e: broken environment Change in e2e environment caused test failure. focus: checkout payments needs triage Manually put issue into triage process.

Comments

@mgascam
Copy link
Contributor

mgascam commented Feb 10, 2025

Description

There are shopper test suites failing in the E2E - All workflow, with WP Nightly version:

  • tests/e2e-pw/specs/wcpay/shopper/shopper-wc-blocks-checkout-failures.spec.ts:106:8 › WooCommerce Blocks > Checkout failures › Should show error – Your card was declined.
  • tests/e2e-pw/specs/wcpay/shopper/shopper-wc-blocks-checkout-purchase.spec.ts:44:7 › WooCommerce Blocks > Successful purchase › using a basic card
  • tests/e2e-pw/specs/wcpay/shopper/shopper-wc-blocks-saved-card-checkout-and-usage.spec.ts:58:7 › WooCommerce Blocks > Saved cards › should be able to save basic card on Blocks checkout

Output from the test failure:

    "beforeAll" hook timeout of 120000ms exceeded.

      70 | 		let shopperPage: Page;
      71 |
    > 72 | 		test.beforeAll( async ( { browser }, { project } ) => {
         | 		     ^
      73 | 			shopperPage = ( await getShopper( browser ) ).shopperPage;
      74 |
      75 | 			if (
        at /home/runner/work/woocommerce-payments/woocommerce-payments/tests/e2e-pw/specs/wcpay/shopper/shopper-wc-blocks-checkout-failures.spec.ts:72:8

    Error: locator.fill: Test ended.
    Call log:
      - waiting for getByPlaceholder('Search')

Additional context

Priority

I'm assigning a medium priority because only the WP Nightly tests are failing.

Reason why this e2e test is broken

The tests are failing due to missing elements in the DOM. Given it's only failing for the WP Nightly tests this could be related to some upstream changes in the structure of the DOM (needs to be confirmed). For this reason I'm adding the e2e: broken environment label

Important

Please, ensure when closing this issue (PR fix) that only one e2e: broken label is added and it is accurate.

  • I confirmed there's only one e2e: broken label in this issue and it is accurate.
@mgascam mgascam added category: e2e Issues and PRs related to e2e tests. e2e: broken environment Change in e2e environment caused test failure. needs triage Manually put issue into triage process. priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability labels Feb 10, 2025
@c-shultz c-shultz added focus: checkout payments and removed priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability labels Feb 10, 2025
@c-shultz
Copy link
Collaborator

I'm pulling off the priority and routing to the checkout payments focus area so the team can double-check on priority during their triage.

@mgascam
Copy link
Contributor Author

mgascam commented Feb 11, 2025

Tests are passing after #10340 was merged

@mgascam mgascam closed this as not planned Won't fix, can't repro, duplicate, stale Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: e2e Issues and PRs related to e2e tests. e2e: broken environment Change in e2e environment caused test failure. focus: checkout payments needs triage Manually put issue into triage process.
Projects
None yet
Development

No branches or pull requests

2 participants