-
Notifications
You must be signed in to change notification settings - Fork 179
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: Run against WordPress 5.9+ #12606
Conversation
@swissspidy @spacedmonkey --- seems like most of the remaining failures for this PR are for the WooCommerce tests any thoughts as to why? Bumped |
What about -
|
@spacedmonkey There's a separate ticket for changing the unit tests. |
@timarney Because WooCommerce is not being installed on the WP 5.9 tests. In |
FYI, looks like the WP latest e2e tests are not passing because the WP only ships three default themes, so with 6.1 it's To fix, we need to force-install the theme. So we need to change this line:
Into |
Context
Update E2E test to run 5.9 as min version
Summary
Updates workflow file
Relevant Technical Choices
To-do
User-facing changes
Testing Instructions
This PR can be tested by following these steps:
Reviews
Does this PR have a security-related impact?
No
Does this PR change what data or activity we track or use?
No
Does this PR have a legal-related impact?
No
Checklist
Type: XYZ
label to the PRFixes #12572