Changeset 51868 for trunk/package.json

Timestamp:

09/24/2021 04:54:42 PM (4 years ago)

Author:

hellofromTonya

Message:

Build/Test Tools: Remove the PHPUnit container from local Docker environment.

This commit:

• Removes the PHPUnit wordpressdevelop/phpunit container as a service to the Docker environment.
• Updates test:php (the default way to run tests) to run the Composer PHPUnit package default.
• Removes the test:php-composer script.

There is no longer a need for core to keep the wordpressdevelop/phpunit container as a service in docker-compose.yml. Removing it will reduce the overhead and bandwidth needed to set up WordPress locally and remove confusion about its use.

Follow-up to [45783-45784], [49099], [49362], [51545], [51736], [51685].

Props johnbillion, hellofromTonya.
Fixes #54112.

File:

• trunk/package.json (modified) (1 diff)

trunk/package.json

@@ -171 +171 @@
         "env:logs": "node ./tools/local-env/scripts/docker.js logs",
         "env:pull": "node ./tools/local-env/scripts/docker.js pull",
-        "test:php": "node ./tools/local-env/scripts/docker.js run -T php composer update -W && node ./tools/local-env/scripts/docker.js run --rm phpunit phpunit",
-        "test:php-composer": "node ./tools/local-env/scripts/docker.js run -T php composer update -W && node ./tools/local-env/scripts/docker.js run --rm phpunit php ./vendor/bin/phpunit",
+        "test:php": "node ./tools/local-env/scripts/docker.js run -T php composer update -W && node ./tools/local-env/scripts/docker.js run php ./vendor/bin/phpunit",
         "test:e2e": "node ./tests/e2e/run-tests.js",
         "wp-packages-update": "wp-scripts packages-update"