woocommerce / woocommerce-square

Square POS and Payments Integration
https://woo.com/products/square
Other
11 stars 7 forks source link

[E2E] Fix test c3.woo-sor.spec-with-inventory > OnePlus 8 pushed to Square with inventory #244

Closed MiguelAxcar closed 4 days ago

MiguelAxcar commented 1 week ago

All Submissions:


Changes proposed in this Pull Request:

This PR intends to fix the failing E2E test c3.woo-sor.spec-with-inventory.spec.js:53:5 › OnePlus 8 pushed to Square with inventory.

The failure was due to an issue in the test setup or assertions, and the changes ensure the test now passes successfully.

Steps to test the changes in this Pull Request:

Prerequisites:

  1. Ensure you have the appropriate environment for E2E testing with Square.

Testing steps

  1. Checkout the branch associated with this PR.
  2. Run the specific test using the following command:
    npx playwright test tests/e2e/specs/c3.woo-sor.spec-with-inventory.spec.js --config=tests/e2e/config/playwright.config.js --grep "OnePlus 8 pushed to Square with inventory"
  3. You can also try running all the tests in the file using the following command:
    npx playwright test tests/e2e/specs/c3.woo-sor.spec-with-inventory.spec.js --config=tests/e2e/config/playwright.config.js

Expected outcome

The specific E2E test c3.woo-sor.spec-with-inventory.spec.js:53:5 › OnePlus 8 pushed to Square with inventory should pass successfully, confirming that the issue has been resolved.

Changelog entry

Dev - Resolved the failing E2E test for c3.woo-sor.spec-with-inventory.spec.js:53:5 › OnePlus 8 pushed to Square with inventory.

MiguelAxcar commented 1 week ago

@iamdharmesh here's one of the PRs I plan to add to fix failing E2E tests. I'm planning to tackle d6 now, which is similar to c3, and I let you to know. Thank you!

iamdharmesh commented 5 days ago

This looks good to me. Any thoughts on things here @iamdharmesh?

LGTM @dkotter. Thanks for fixing this @MiguelAxcar