Changeset 48592

Timestamp:

07/23/2020 11:26:50 PM (4 years ago)

Author:

SergeyBiryukov

Message:

Build/Test Tools: Check if all the required constants are defined before running the test suite.

Follow-up to [47904].

Props azaozz, TimothyBlynJacobs, SergeyBiryukov.
Fixes #50251.

File:

• trunk/tests/phpunit/includes/bootstrap.php (modified) (2 diffs)

trunk/tests/phpunit/includes/bootstrap.php

@@ -38 +38 @@
 require_once __DIR__ . '/functions.php';
 
-if ( version_compare( tests_get_phpunit_version(), '5.4', '<' ) || version_compare( tests_get_phpunit_version(), '8.0', '>=' ) ) {
+$phpunit_version = tests_get_phpunit_version();
+
+if ( version_compare( $phpunit_version, '5.4', '<' ) || version_compare( $phpunit_version, '8.0', '>=' ) ) {
     printf(
         "Error: Looks like you're using PHPUnit %s. WordPress requires at least PHPUnit 5.4 and is currently only compatible with PHPUnit up to 7.x.\n",
-        tests_get_phpunit_version()
+        $phpunit_version
     );
     echo "Please use the latest PHPUnit version from the 7.x branch.\n";
@@ -50 +52 @@
     echo "Error: The /build/ directory is missing! Please run `npm run build` prior to running PHPUnit.\n";
     exit( 1 );
+}
+
+$required_constants = array(
+    'WP_TESTS_DOMAIN',
+    'WP_TESTS_EMAIL',
+    'WP_TESTS_TITLE',
+    'WP_PHP_BINARY',
+);
+
+foreach ( $required_constants as $constant ) {
+    if ( ! defined( $constant ) ) {
+        printf(
+            "Error: The required %s constant is not defined. Check out `wp-tests-config-sample.php` for an example.\n",
+            $constant
+        );
+        exit( 1 );
+    }
 }