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
Unskip Mini Cart i18n test #45371
Unskip Mini Cart i18n test #45371
Conversation
Test Results SummaryCommit SHA: 219bded
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. |
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: |
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 your work! LGTM! 🙇
I'm pre-approving the PR, but I left a comment.
npm run wp-env run tests-cli -- wp language plugin update woocommerce
Is this command necessary?
...sts/e2e/tests/cart/cart-checkout-block-translations.shopper.block_theme.side_effects.spec.ts
Outdated
Show resolved
Hide resolved
...ce-blocks/tests/e2e/tests/mini-cart/mini-cart-block.shopper.block_theme.side_effects.spec.ts
Outdated
Show resolved
Hide resolved
Thanks for reviewing and pre-approving this PR, @gigitux. I've updated the PR based on your feedback and as all tests are passing, I'm merging it now. |
* Unskip Mini Cart i18n test * Install language packs before each i18n-related test suite * Delete translations.sh * Add changelog * Replace deprecated WP-CLI command * Remove obsolete WP-CLI command
Submission Review Guidelines:
Changes proposed in this Pull Request:
Closes #45346.
This PR contains the following changes:
User can see translation in empty Mini-Cart
.plugins/woocommerce-blocks/tests/e2e/tests/cart/cart-checkout-block-translations.shopper.block_theme.side_effects.spec.ts
.plugins/woocommerce-blocks/tests/e2e/tests/mini-cart/mini-cart-block.shopper.block_theme.side_effects.spec.ts
.plugins/woocommerce-blocks/tests/e2e/bin/scripts/parallel/translations.sh
as language pcks will be installed directly in the previous mentioned files.How to test the changes in this Pull Request:
Using the WooCommerce Testing Instructions Guide, include your detailed testing instructions:
Changelog entry
Significance
Type
Message
Comment