Changeset 51868 for trunk/.github/workflows/test-coverage.yml

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/.github/workflows/test-coverage.yml (modified) (2 diffs)

trunk/.github/workflows/test-coverage.yml

@@ -144 +144 @@
       - name: Run tests as a single site
         if: ${{ ! matrix.multisite }}
-        run: npm run test:php-composer -- --verbose -c phpunit.xml.dist --coverage-clover wp-code-coverage-single-clover-${{ github.sha }}.xml
+        run: npm run test:php -- --verbose -c phpunit.xml.dist --coverage-clover wp-code-coverage-single-clover-${{ github.sha }}.xml
 
       - name: Ensure version-controlled files are not modified during the tests
@@ -158 +158 @@
       - name: Run tests as a multisite install
         if: ${{ matrix.multisite }}
-        run: npm run test:php-composer -- --verbose -c tests/phpunit/multisite.xml --coverage-clover wp-code-coverage-multisite-clover-${{ github.sha }}.xml
+        run: npm run test:php -- --verbose -c tests/phpunit/multisite.xml --coverage-clover wp-code-coverage-multisite-clover-${{ github.sha }}.xml
 
       - name: Ensure version-controlled files are not modified during the tests