Changeset 48957 for trunk/.travis.yml

Timestamp:

09/07/2020 07:21:26 PM (5 years ago)

Author:

SergeyBiryukov

Message:

Build/Test Tools: Allow unit tests to run on PHP 8 in full.

PHP 8 introduces a new control structure called match, which makes match a reserved keyword in PHP 8.

One of the PHPUnit dependencies declares a class named Match, which triggered a fatal error before PHPUnit could even start.

To be able to use PHPUnit 7.x on PHP 8 and run the tests, core needs a new version of that dependency, which is now installed using Composer.

This is the simplest way to get things working again and start addressing the individual test failures.

Additionally, various test runs on PHP 8 on Travis are now performed individually instead of being chained, so that failures outside of WP scope don't block further execution.

Props jrf, jorbin, pento.
See #50902.

File:

• trunk/.travis.yml (modified) (3 diffs)

trunk/.travis.yml

@@ -52 +52 @@
   - env: LOCAL_PHP=5.6-fpm WP_TRAVISCI=test:php
     name: "PHPUnit Tests: PHP 5.6"
-  - env: LOCAL_PHP=8.0-fpm WP_TRAVISCI=test:php
+  - env: LOCAL_PHP=8.0-fpm COMPOSER_INSTALL=true WP_TRAVISCI=test:php
     name: "PHPUnit Tests: PHP 8.0"
+    script:
+      # The PHPUnit 7.x phar is not compatible with PHP 8 and won't be updated,
+      # as PHPUnit 7 is no longer supported, so run the Composer-installed PHPUnit instead.
+      - docker-compose run --rm phpunit php ./vendor/bin/phpunit --verbose -c phpunit.xml.dist
+      - docker-compose run --rm phpunit php ./vendor/bin/phpunit --verbose -c phpunit.xml.dist --group ajax
+      - docker-compose run --rm phpunit php ./vendor/bin/phpunit --verbose -c tests/phpunit/multisite.xml
+      - docker-compose run --rm phpunit php ./vendor/bin/phpunit --verbose -c tests/phpunit/multisite.xml --group ms-files
+      - docker-compose run --rm phpunit php ./vendor/bin/phpunit --verbose -c phpunit.xml.dist --group external-http
+      - docker-compose run --rm phpunit php ./vendor/bin/phpunit --verbose -c phpunit.xml.dist --group restapi-jsclient
+      # __fakegroup__ is excluded to force PHPUnit to ignore the <exclude> settings in phpunit.xml.dist.
+      - LOCAL_PHP_XDEBUG=true docker-compose run --rm phpunit phpunit -v --group xdebug --exclude-group __fakegroup__
+
   allow_failures:
-  - env: LOCAL_PHP=8.0-fpm WP_TRAVISCI=test:php
+  - env: LOCAL_PHP=8.0-fpm COMPOSER_INSTALL=true WP_TRAVISCI=test:php
   fast_finish: true
 
@@ -71 +83 @@
 before_script:
 - |
-  if [[ "$COMPOSER_INSTALL" == "true" ]]; then
+  if [[ "$COMPOSER_INSTALL" == "true" && ${LOCAL_PHP:0:3} != "8.0" ]]; then
     docker-compose run --rm php composer --version
     docker-compose run --rm php composer install
+  elif [[ "$COMPOSER_INSTALL" == "true" && ${LOCAL_PHP:0:3} == "8.0" ]]; then
+    docker-compose run --rm php composer --version
+    docker-compose run --rm php composer install --ignore-platform-reqs
   fi
 - npm --version
@@ -89 +104 @@
     docker-compose run --rm php php --version
     docker-compose run --rm php php -m
+  fi
+- |
+  if [[ ${LOCAL_PHP:0:3} == "8.0" ]]; then
+    docker-compose run --rm phpunit php ./vendor/bin/phpunit --version
+  elif [[ "$WP_TRAVISCI" == "test:e2e" ]] || [[ "$WP_TRAVISCI" == "test:php" ]]; then
     docker-compose run --rm phpunit phpunit --version
   fi