Skip to content

Fix E2E tests for remove label errors #314

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 8 commits into from
Mar 28, 2025
Merged

Fix E2E tests for remove label errors #314

merged 8 commits into from
Mar 28, 2025

Conversation

mukeshpanchal27
Copy link
Collaborator

All Submissions:

  • Does your code follow the WooCommerce Sniffs variant of WordPress coding standards?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully run tests with your changes locally?
  • Will this change require new documentation or changes to existing documentation?

Changes proposed in this Pull Request:

Closes #306.

Steps to test the changes in this Pull Request:

  1. Open the https://github.com/woocommerce/woocommerce-square/actions/runs/13812011592.
  2. Observe that all E2E test steps have completed successfully.
  3. Despite this, the summary page shows a failure due to the errors mentioned above.

Screenshots

Image

Changelog entry

Dev - Fix E2E tests remove label errors.

@mukeshpanchal27 mukeshpanchal27 self-assigned this Mar 19, 2025
@mukeshpanchal27 mukeshpanchal27 changed the title Fix E2E remove label errors Fix E2E tests for remove label errors Mar 19, 2025
@mukeshpanchal27 mukeshpanchal27 requested a review from dkotter March 19, 2025 11:45
@mukeshpanchal27 mukeshpanchal27 mentioned this pull request Mar 26, 2025
Open
@vikrampm1 vikrampm1 added this to the 4.8.8 milestone Mar 28, 2025
@vikrampm1 vikrampm1 marked this pull request as ready for review March 28, 2025 13:28
@vikrampm1 vikrampm1 merged commit 75cedfd into trunk Mar 28, 2025
9 of 10 checks passed
@vikrampm1 vikrampm1 deleted the fix/306 branch March 28, 2025 13:28
@vikrampm1 vikrampm1 mentioned this pull request Apr 4, 2025
Closed
22 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dkotter dkotter dkotter approved these changes

Assignees

@mukeshpanchal27 mukeshpanchal27

Labels
needs: e2e testing status: e2e tests failing status: e2e tests passing
Projects
None yet
Milestone
4.9.0
Development

Successfully merging this pull request may close these issues.

E2E tests passed, but summary page shows errors
3 participants
@mukeshpanchal27 @dkotter @vikrampm1