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

Update ETK phpunit to work with new wp-env improvements #65653

Closed
wants to merge 6 commits into from

Conversation

noahtallen
Copy link
Contributor

Proposed Changes

In WordPress/gutenberg#41780 and WordPress/gutenberg#41852 @ObliviousHarmony made some improvements to wp-env we can take advantage of in Calypso:

  1. wp-env now installs the WP test files for the WordPress version installed, removing the need for wp-phpunit.
  2. the phpunit container was deprecated upstream. We can control the phpunit version ourselves with composer. Since this container was also broken on M1 machines, not using it will fix that issue.

This shouldn't be merged until the wp-env update is available on npm.

Testing Instructions

Run $ path/to/gutenberg/packages/env/bin/wp-env run tests-wordpress /var/www/html/wp-content/plugins/editing-toolkit-plugin/vendor/bin/phpunit -c /var/www/html/wp-content/plugins/editing-toolkit-plugin/phpunit.xml.dist --verbose and it should pass.

@github-actions
Copy link

github-actions bot commented Jul 15, 2022

@matticbot
Copy link
Contributor

This PR does not affect the size of JS and CSS bundles shipped to the user's browser.

Generated by performance advisor bot at iscalypsofastyet.com.

@noahtallen noahtallen self-assigned this Jul 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants