Remove workarounds for site-editing-templates tests #7151 #7568
Conversation
The release ZIP for this PR is accessible via:
|
remove workaround for site-editing-templates tests
55d6cdd
to
51a3f58
Compare
Size Change: 0 B Total Size: 974 kB ℹ️ View Unchanged
|
TypeScript Errors ReportFiles with errors: 430 🎉 🎉 This PR does not introduce new TS errors. |
TypeScript Errors ReportFiles with errors: 448 🎉 🎉 This PR does not introduce new TS errors. |
TypeScript Errors ReportFiles with errors: 448 🎉 🎉 This PR does not introduce new TS errors. |
TypeScript Errors ReportFiles with errors: 448 🎉 🎉 This PR does not introduce new TS errors. |
…ocks into fix/remove-comments-6-1-wp
72ea9ef
to
3d03dab
Compare
…mmerce/woocommerce-blocks into fix/remove-comments-6-1-wp
3d03dab
to
c288e7e
Compare
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.
@gigitux code looks good and the checks seem to all be passing on the PR.
When I run the tests locally, however, they aren't failing but they seem stuck in a pending
state:
I ran the following commands on my machine:
npm run wp-env destroy
npm run wp-env start
npm run test:e2e -- site-editing-templates
Let me know if I'm missing anything or if that's expected. Other than that, LGTM!
FYI:
|
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.
Based on @dinhtungdu's explanation (thanks Tung 🙌 ) I think everything is working as expected here!
LGTM! 🚀
This PR removes some workarounds that were introduced to avoid test failing. More context here #7151
Fixes #7151
Testing
User Facing Testing
WooCommerce Visibility