Changeset 49784

Timestamp:

12/10/2020 03:28:09 PM (4 years ago)

Author:

desrosj

Message:

Build/Test Tools: Inline documentation updates for the PHPUnit workflow.

See #50401.

File:

• trunk/.github/workflows/phpunit-tests.yml (modified) (5 diffs)

trunk/.github/workflows/phpunit-tests.yml

@@ -27 +27 @@
   # - Checks out the WordPress Importer plugin (needed for the Core PHPUnit tests).
   # - Logs debug information about the runner container.
-  # - Installs NodeJS 12 (todo: install the version of NPM specified in the `.nvmrc` file to support older branches).
+  # - Installs NodeJS 14.
   # - Sets up caching for NPM.
   # _ Installs NPM dependencies using install-changed to hash the `package.json` file.
@@ -106 +106 @@
   # - Downloads the built WordPress artifact from the previous job.
   # - Unzips the artifact.
-  # - Installs NodeJS 12 (todo: install the version of NPM specified in the `nvmrc` file to support older branches)
+  # - Installs NodeJS 14.
   # - Sets up caching for NPM.
   # _ Installs NPM dependencies using install-changed to hash the `package.json` file.
@@ -120 +120 @@
   # - Install WordPress within the Docker container.
   # - Run the PHPUnit tests.
-  # - Reports test results to the Distributed Hosting Tests.
+  # - Checks out the WordPress Test reporter repository.
+  # - Submit the test results to the WordPress.org host test results.
   # - todo: Configure Slack notifications for failing tests.
   test-php:
@@ -272 +273 @@
         run: LOCAL_PHP_XDEBUG=true npm run test:php -- -v --group xdebug --exclude-group __fakegroup__
 
-      - name: WordPress Test Reporter
+      - name: Checkout the WordPress Test Reporter
         if: ${{ github.repository == 'WordPress/wordpress-develop' && github.ref == 'refs/heads/master' && matrix.report }}
         uses: actions/checkout@v2
@@ -279 +280 @@
           path: 'test-runner'
 
-      - name: WordPress Test Reporter
+      - name: Submit test results to the WordPress.org host test results
         if: ${{ github.repository == 'WordPress/wordpress-develop' && github.ref == 'refs/heads/master' && matrix.report }}
         env:
-          WPT_REPORT_API_KEY: ${{ secrets.WPT_REPORT_API_KEY }}
+          WPT_REPORT_API_KEY: "${{ secrets.WPT_REPORT_API_KEY }}"
         run: docker-compose run --rm -e "$WPT_REPORT_API_KEY" -e WPT_PREPARE_DIR=/var/www -e WPT_TEST_DIR=/var/www php php test-runner/report.php