Changeset 45827

Timestamp:

08/19/2019 02:17:50 AM (4 years ago)

Author:

pento

Message:

Tests: Add an xdebug group for tests that require xdebug.

These tests are now also run as their own part of the Travis build, to ensure performance of other tests isn't affected by the presence of xdebug.

Fixes #40532.

Location:

trunk

Files:

• .travis.yml (modified) (1 diff)
• tests/phpunit/tests/ajax/Response.php (modified) (1 diff)
• tests/phpunit/tests/oembed/headers.php (modified) (1 diff)

trunk/.travis.yml

@@ -112 +112 @@
       npm run test:php -- --verbose -c tests/phpunit/multisite.xml --group ms-files &&
       npm run test:php -- --verbose -c phpunit.xml.dist --group external-http &&
-      npm run test:php -- --verbose -c phpunit.xml.dist --group restapi-jsclient
+      npm run test:php -- --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 npm run test:php -- -v --group xdebug --exclude-group __fakegroup__
     elif [[ "$WP_TRAVISCI" == "lint:php" ]]; then
       docker-compose run --rm php composer format &&

trunk/tests/phpunit/tests/ajax/Response.php

@@ -70 +70 @@
      * @runInSeparateProcess
      * @preserveGlobalState disabled
+     * @group xdebug
      */
     public function test_response_charset_in_header() {

trunk/tests/phpunit/tests/oembed/headers.php

@@ -6 +6 @@
  * @group oembed
  * @group oembed-headers
+ * @group xdebug
  */
 class Tests_oEmbed_HTTP_Headers extends WP_UnitTestCase {