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

[e2e tests] Disable woocommerce_coming_soon during test environment setup #47024

Merged
merged 3 commits into from Apr 29, 2024

Conversation

adimoldovan
Copy link
Contributor

@adimoldovan adimoldovan commented Apr 29, 2024

Submission Review Guidelines:

Changes proposed in this Pull Request:

#46842 removed the setting of woocommerce_coming_soon in the global setup in favour of setting it in the test-env-setup.sh, but the latter was missed somehow.

How to test the changes in this Pull Request:

Using the WooCommerce Testing Instructions Guide, include your detailed testing instructions:

Run all tests without sharding. There should be no failures caused by the site not being available.

@adimoldovan adimoldovan self-assigned this Apr 29, 2024
@github-actions github-actions bot added focus: e2e tests Issues related to e2e tests plugin: woocommerce Issues related to the WooCommerce Core plugin. labels Apr 29, 2024
@adimoldovan adimoldovan marked this pull request as ready for review April 29, 2024 15:37
@woocommercebot woocommercebot requested a review from a team April 29, 2024 15:37
Copy link
Contributor

Hi , @woocommerce/vortex

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

@lanej0 lanej0 merged commit e36baab into trunk Apr 29, 2024
25 checks passed
@lanej0 lanej0 deleted the e2e/disable-coming-soon-in-env-setup branch April 29, 2024 17:27
@github-actions github-actions bot added this to the 9.0.0 milestone Apr 29, 2024
@github-actions github-actions bot added the needs: analysis Indicates if the PR requires a PR testing scrub session. label Apr 29, 2024
@rodelgc rodelgc 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 May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
focus: e2e tests Issues related to e2e tests 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