-
Notifications
You must be signed in to change notification settings - Fork 70
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
Fix related products E2Es for WC9.5 #481
Conversation
This run didn't pass. I got the same failure in a local run. |
Thanks. I forgot to migrate the adjusted selector. Now, the run is passing https://github.com/woocommerce/woocommerce-google-analytics-integration/actions/runs/12383008805 |
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.
Thanks for the compatibility fix! LGTM.
Hi @tomalec, I will address #481 (comment) and merge this PR to make the later release process a bit smoother. |
Changes proposed in this Pull Request:
This PR adjusts E2E tests for WC 9.5 changes.
In WC 9.5, by default, there is no related products section on a single product page - I made the test skip in such a case.
Also, I added a "related" tag to all simple products to make sure simple products will appear in the related products section instead of "Cat teasers" - without the "Add to cart" button.
Screenshots:
Detailed test instructions:
9.5.0-rc.1
- https://github.com/woocommerce/woocommerce-google-analytics-integration/actions/runs/123628678719.4.0
- https://github.com/woocommerce/woocommerce-google-analytics-integration/actions/runs/12383008805Additional details:
create*Page
sChangelog entry