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

Blocks E2E: Fix guest user handling in checkout tests #45624

Merged
merged 7 commits into from Mar 15, 2024

Conversation

WunderBart
Copy link
Contributor

@WunderBart WunderBart commented Mar 15, 2024

Changes proposed in this Pull Request:

Related: https://github.com/woocommerce/woocommerce/pull/44347/files#r1526255938

Fix flaky tests where the admin is logged out instead of using the guest storage state. Logging out via UI affects other tests as the user stays logged out.

Example error that should no longer happen:

Screenshot 2024-03-15 at 13 27 26

How to test the changes in this Pull Request:

All affected tests should pass.

@WunderBart WunderBart self-assigned this Mar 15, 2024
@WunderBart WunderBart requested a review from gigitux March 15, 2024 12:22
@woocommercebot woocommercebot requested a review from a team March 15, 2024 12:22
Copy link
Contributor

github-actions bot commented Mar 15, 2024

Hi @nielslange, @gigitux, @kmanijak, @woocommerce/rubik, @woocommerce/woo-fse

Apart from reviewing the code changes, please make sure to review the testing instructions as well.

You can follow this guide to find out what good testing instructions should look like:
https://github.com/woocommerce/woocommerce/wiki/Writing-high-quality-testing-instructions

Copy link
Contributor

github-actions bot commented Mar 15, 2024

Test Results Summary

Commit SHA: 692df11

Test 🧪Passed ✅Failed 🚨Broken 🚧Skipped ⏭️Unknown ❔Total 📊Duration ⏱️
API Tests25900202610m 39s
E2E Tests324002903537m 22s

To view the full API test report, click here.
To view the full E2E test report, click here.
To view all test reports, visit the WooCommerce Test Reports Dashboard.

@WunderBart WunderBart requested review from a team, nielslange and kmanijak and removed request for a team March 15, 2024 16:31
@WunderBart
Copy link
Contributor Author

WunderBart commented Mar 15, 2024

All tests have passed. 🎉

@WunderBart WunderBart requested a review from a team March 15, 2024 16:37
@github-actions github-actions bot added the plugin: woocommerce Issues related to the WooCommerce Core plugin. label Mar 15, 2024
Copy link
Member

@senadir senadir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change make sense, thank you for catching it and TIL

@WunderBart WunderBart enabled auto-merge (squash) March 15, 2024 17:13
@WunderBart WunderBart merged commit 1c1dc10 into trunk Mar 15, 2024
53 of 54 checks passed
@WunderBart WunderBart deleted the fix/e2e-logged-out-checkout branch March 15, 2024 17:23
@github-actions github-actions bot added this to the 8.8.0 milestone Mar 15, 2024
@github-actions github-actions bot added the needs: analysis Indicates if the PR requires a PR testing scrub session. label Mar 15, 2024
@alopezari alopezari added status: analysis complete Indicates if a PR has been analysed by Solaris and removed needs: analysis Indicates if the PR requires a PR testing scrub session. labels Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plugin: woocommerce Issues related to the WooCommerce Core plugin. status: analysis complete Indicates if a PR has been analysed by Solaris
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants