Changeset 50309 for branches/4.8/.github/workflows/phpunit-tests.yml

Timestamp:

02/12/2021 07:34:39 PM (4 years ago)

Author:

desrosj

Message:

Build/Test Tools: Restore automated testing in the 4.8 branch.

This commit merges the workflow files required to run automated testing on GitHub Actions.

In addition, [49836] and [50285] have been included in order to keep the local Docker environment consistent across all branches.

Merges [49162,49168-49169,49175,49204,49227-49228,49244,49369,49371,49548,49781-49784,49786,49836,49938,50268,50285,50298] to the 4.8 branch.
See #50401.

Location:

branches/4.8

Files:

• . (modified) (1 prop)
• .github (added)
• .github/workflows (copied) (copied from trunk/.github/workflows)
• .github/workflows/phpunit-tests.yml (modified) (16 diffs)

branches/4.8/.github/workflows/phpunit-tests.yml

@@ -5 +5 @@
     branches:
       - master
-      - '*.*'
+      - '3.[7-9]'
+      - '[4-9].[0-9]'
+    tags:
+      - '3.[7-9]*'
+      - '[4-9].[0-9]*'
   pull_request:
   # Once weekly On Sundays at 00:00 UTC.
@@ -23 +27 @@
   #
   # Performs the following steps:
-  # - Cancels all previous workflow runs that have not completed.
+  # - Cancels all previous workflow runs for pull requests that have not completed.
   # - Checks out the repository.
-  # - 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.
+  # _ Installs NPM dependencies using install-changed to hash the `package.json` file.
   # - Builds WordPress to run from the `build` directory.
-  # - Creates a ZIP file of compiled WordPress
+  # - Creates a ZIP file of compiled WordPress.
   # - Uploads ZIP file as an artifact.
   setup-wordpress:
     name: Setup WordPress
     runs-on: ubuntu-latest
+    if: ${{ github.repository == 'WordPress/wordpress-develop' || github.event_name == 'pull_request' }}
 
     steps:
-      - name: Cancel previous runs of this workflow
+      - name: Cancel previous runs of this workflow (pull requests only)
+        if: ${{ github.event_name == 'pull_request' }}
         uses: styfle/cancel-workflow-action@0.5.0
         with:
@@ -46 +51 @@
         uses: actions/checkout@v2
 
-      - name: Checkout the WordPress Importer plugin
-        run: svn checkout -r 2387243 https://plugins.svn.wordpress.org/wordpress-importer/trunk/ tests/phpunit/data/plugins/wordpress-importer
-
       - name: Log debug information
         run: |
+          echo "$GITHUB_REF"
+          echo "$GITHUB_EVENT_NAME"
           npm --version
           node --version
@@ -63 +67 @@
         uses: actions/setup-node@v1
         with:
-          node-version: 12
+          node-version: 14
 
       - name: Cache NodeJS modules
@@ -77 +81 @@
 
       - name: Install Dependencies
-        run: npm ci
+        run: npx install-changed --install-command="npm ci"
 
       - name: Build WordPress
@@ -86 +90 @@
         with:
           filename: built-wp-${{ github.sha }}.zip
-          exclusions: '/*node_modules/*'
+          exclusions: '*.git* /*node_modules/* packagehash.txt'
 
       - name: Upload build artifact
@@ -102 +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.
+  # _ Installs NPM dependencies using install-changed to hash the `package.json` file.
   # - Configures caching for Composer.
-  # _ Installs Composer dependencies (if desired)
-  # - Logs Docker debug information (about both the Docker installation within the runner)
+  # _ Installs Composer dependencies (if desired).
+  # - Logs Docker debug information (about both the Docker installation within the runner).
   # - Starts the WordPress Docker container.
   # - Starts the memcached server after the Docker network has been created (if desired).
@@ -113 +117 @@
   # - Logs debug general information.
   # - Logs the running Docker containers.
-  # - Logs Docker debug information (about both the Docker installation within the runner and the WordPress container)
+  # - Logs debug information about what's installed within the WordPress Docker containers.
   # - 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.
+  # - Reconnect the directory to the Git repository.
+  # - Submit the test results to the WordPress.org host test results.
   # - todo: Configure Slack notifications for failing tests.
   test-php:
-    name: ${{ matrix.php_versions }} on ${{ matrix.os }}
+    name: ${{ matrix.php }}${{ matrix.memcached && ' with memcached' || '' }} on ${{ matrix.os }}
     needs: setup-wordpress
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
-        php_versions: [ '8.0', 7.4, '7.4 with memcached', 7.3, 7.2, 7.1, '7.0', 5.6.20 ]
+        php: [ '7.1' ]
+        phpunit: [ '6-php-7.1' ]
         os: [ ubuntu-latest ]
+        memcached: [ false ]
+        include:
+          # Include job for PHP 7.1 with memcached.
+          - php: '7.1'
+            phpunit: '6-php-7.1'
+            os: ubuntu-latest
+            memcached: true
+          - php: '7.0'
+            phpunit: '6-php-7.0'
+            os: ubuntu-latest
+            memcached: false
+          - php: '5.6'
+            phpunit: '4-php-5.6'
+            os: ubuntu-latest
+            memcached: false
+          - php: '5.5'
+            phpunit: '5.5'
+            os: ubuntu-latest
+            memcached: false
+          - php: '5.4'
+            phpunit: '5.4'
+            os: ubuntu-latest
+            memcached: false
+          - php: '5.3'
+            phpunit: '5.3'
+            os: ubuntu-latest
+            memcached: false
     env:
-      LOCAL_PHP: ${{ matrix.php_versions }}-fpm
+      LOCAL_PHP: ${{ matrix.php }}-fpm
+      LOCAL_PHPUNIT: ${{ matrix.phpunit }}-fpm
+      LOCAL_PHP_MEMCACHED: ${{ matrix.memcached }}
 
     steps:
@@ -135 +171 @@
           echo "PHP_FPM_GID=$(id -g)" >> $GITHUB_ENV
 
-      - name: Configure memcached
-        if: ${{ contains( matrix.php_versions, 'memcached' ) }}
-        run: |
-          echo "LOCAL_PHP=$(echo ${{ matrix.php_versions }} | cut -c1-3)-fpm" >> $GITHUB_ENV
-          echo "LOCAL_PHP_MEMCACHED=true" >> $GITHUB_ENV
-
       - name: Download the built WordPress artifact
         uses: actions/download-artifact@v2
@@ -152 +182 @@
         uses: actions/setup-node@v1
         with:
-          node-version: 12
+          node-version: 14
 
       - name: Use cached Node modules
@@ -166 +196 @@
 
       - name: Install Dependencies
-        run: npm ci
-
-      - name: Get composer cache directory
-        id: composer-cache
-        if: ${{ env.COMPOSER_INSTALL == true || env.LOCAL_PHP == '8.0-fpm' }}
-        run: echo "::set-output name=dir::$(composer config cache-files-dir)"
+        run: npx install-changed --install-command="npm ci"
 
       - name: Cache Composer dependencies
-        if: ${{ env.COMPOSER_INSTALL == true || env.LOCAL_PHP == '8.0-fpm' }}
+        if: ${{ env.COMPOSER_INSTALL == true }}
         uses: actions/cache@v2
         env:
@@ -180 +205 @@
         with:
           path: ${{ steps.composer-cache.outputs.dir }}
-          key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
+          key: ${{ runner.os }}-php-${{ matrix.php }}-composer-${{ hashFiles('**/composer.lock') }}
           restore-keys: |
-            ${{ runner.os }}-composer-
+            ${{ runner.os }}-php-${{ matrix.php }}-composer-
 
       - name: Install Composer dependencies
-        if: ${{ env.COMPOSER_INSTALL == true || env.LOCAL_PHP == '8.0-fpm' }}
+        if: ${{ env.COMPOSER_INSTALL == true }}
         run: |
           docker-compose run --rm php composer --version
-
-          # The PHPUnit 7.x phar is not compatible with PHP 8 and won't be updated,
-          # as PHPUnit 7 is no longer supported. The Composer-installed PHPUnit should be
-          # used for PHP 8 testing instead.
-          if [ ${{ env.LOCAL_PHP }} == '8.0-fpm' ]; then
-            docker-compose run --rm php composer install --ignore-platform-reqs
-            echo "PHPUNIT_SCRIPT=php-composer" >> $GITHUB_ENV
-          else
-            docker-compose run --rm php composer install
-          fi
+          docker-compose run --rm php composer install
 
       - name: Docker debug information
@@ -210 +226 @@
       # The memcached server needs to start after the Docker network has been set up with `npm run env:start`.
       - name: Start the Memcached server.
-        if: ${{ contains( matrix.php_versions, 'memcached' ) }}
+        if: ${{ matrix.memcached }}
         run: |
           cp tests/phpunit/includes/object-cache.php build/wp-content/object-cache.php
@@ -228 +244 @@
       - name: WordPress Docker container debug information
         run: |
-          docker -v
-          docker-compose -v
           docker-compose run --rm mysql mysql --version
           docker-compose run --rm php php --version
@@ -248 +262 @@
         run: npm run test:${{ env.PHPUNIT_SCRIPT }} -- --verbose -c tests/phpunit/multisite.xml
 
-      - name: Run mutlisite file tests
+      - name: Run ms-files tests as a multisite install
         run: npm run test:${{ env.PHPUNIT_SCRIPT }} -- --verbose -c tests/phpunit/multisite.xml --group ms-files
 
@@ -257 +271 @@
         run: npm run test:${{ env.PHPUNIT_SCRIPT }} -- --verbose -c phpunit.xml.dist --group restapi-jsclient
 
-      # Xdebug supports PHP 8 only from version 3.0, which is not released yet.
-      # Once Xdebug 3.0 is released and included in the Docker image, the IF condition should be removed.
       # __fakegroup__ is excluded to force PHPUnit to ignore the <exclude> settings in phpunit.xml.dist.
       - name: Run (xDebug) tests
-        if: ${{ env.LOCAL_PHP != '8.0-fpm' }}
         run: LOCAL_PHP_XDEBUG=true npm run test:php -- -v --group xdebug --exclude-group __fakegroup__
 
-      - name: WordPress Test Reporter
-        if: ${{ matrix.php_versions == '7.4' }}
+      - name: Checkout the WordPress Test Reporter
+        if: ${{ github.repository == 'WordPress/wordpress-develop' && github.ref == 'refs/heads/master' && matrix.report }}
         uses: actions/checkout@v2
         with:
           repository: 'WordPress/phpunit-test-runner'
           path: 'test-runner'
-        # TODO: Configure hidden keys to successfully report test results.
-        # 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
+
+      - name: Set up the Git repository
+        if: ${{ github.repository == 'WordPress/wordpress-develop' && github.ref == 'refs/heads/master' && matrix.report }}
+        run: |
+          git init
+          git remote add origin https://github.com/WordPress/wordpress-develop.git
+          git fetch
+          git reset origin/master
+
+      - 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 }}"
+        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