-
Notifications
You must be signed in to change notification settings - Fork 10.8k
Add combined testing for tracking number parsers #59469
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
Add combined testing for tracking number parsers #59469
Conversation
…-global-testing-for-tracking-numbers
Testing GuidelinesHi @samnajian , Apart from reviewing the code changes, please make sure to review the testing instructions (Guide) and verify that relevant tests (E2E, Unit, Integration, etc.) have been added or updated as needed. Reminder: PR reviewers are required to document testing performed. This includes:
|
…-global-testing-for-tracking-numbers
Test using WordPress PlaygroundThe changes in this pull request can be previewed and tested using a WordPress Playground instance. Test this pull request with WordPress Playground. Note that this URL is valid for 30 days from when this comment was last updated. You can update it by closing/reopening the PR or pushing a new commit. |
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.
I tested with examples provided via the sheet and I'm getting around 24% mistakes or errors, please see the sheet under Column G
…-global-testing-for-tracking-numbers
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.
LGTM to for this phase, let's create a follow up issue to fine tune it for the next release.
1cabc7f
into
feature/57353-order-fulfillments-entity
Submission Review Guidelines:
Changes proposed in this Pull Request:
This PR adds combined testing for tracking number parsers across 10 major shipping providers with validation algorithms and comprehensive test coverage.
What was implemented:
Closes #59460 WOOPLUG-4914
How to test the changes in this Pull Request:
Using the WooCommerce Testing Instructions Guide, include your detailed testing instructions:
pnpm run watch:build
to build script assetswoocommerce_feature_fulfillments_enabled
option toyes
.Store country changer plugin
Order country changer patch
Testing that has already taken place:
Changelog entry
Changelog Entry Details
Significance
Type
Message
Changelog Entry Comment
Comment
This is a part of a feature branch, and will be added as a single changelog entry to trunk. Sub issues don't need changelog entries.