Changeset 42343 for trunk/tests/phpunit/includes/bootstrap.php

Timestamp:

11/30/2017 11:09:33 PM (7 years ago)

Author:

pento

Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:

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

trunk/tests/phpunit/includes/bootstrap.php

@@ -14 +14 @@
 if ( ! file_exists( $config_file_path . '/wp-tests-config.php' ) ) {
     // Support the config file from the root of the develop repository.
-    if ( basename( $config_file_path ) === 'phpunit' && basename( dirname( $config_file_path ) ) === 'tests' )
+    if ( basename( $config_file_path ) === 'phpunit' && basename( dirname( $config_file_path ) ) === 'tests' ) {
         $config_file_path = dirname( dirname( $config_file_path ) );
+    }
 }
 $config_file_path .= '/wp-tests-config.php';
@@ -39 +40 @@
 define( 'WP_LANG_DIR', DIR_TESTDATA . '/languages' );
 
-if ( ! defined( 'WP_TESTS_FORCE_KNOWN_BUGS' ) )
+if ( ! defined( 'WP_TESTS_FORCE_KNOWN_BUGS' ) ) {
     define( 'WP_TESTS_FORCE_KNOWN_BUGS', false );
+}
 
 // Cron tries to make an HTTP request to the blog, which always fails, because tests are run in CLI mode only
@@ -54 +56 @@
 // Should we run in multisite mode?
 $multisite = '1' == getenv( 'WP_MULTISITE' );
-$multisite = $multisite || ( defined( 'WP_TESTS_MULTISITE') && WP_TESTS_MULTISITE );
+$multisite = $multisite || ( defined( 'WP_TESTS_MULTISITE' ) && WP_TESTS_MULTISITE );
 $multisite = $multisite || ( defined( 'MULTISITE' ) && MULTISITE );
 
@@ -72 +74 @@
 
 if ( $multisite ) {
-    echo "Running as multisite..." . PHP_EOL;
+    echo 'Running as multisite...' . PHP_EOL;
     defined( 'MULTISITE' ) or define( 'MULTISITE', true );
     defined( 'SUBDOMAIN_INSTALL' ) or define( 'SUBDOMAIN_INSTALL', false );
     $GLOBALS['base'] = '/';
 } else {
-    echo "Running as single site... To run multisite, use -c tests/phpunit/multisite.xml" . PHP_EOL;
+    echo 'Running as single site... To run multisite, use -c tests/phpunit/multisite.xml' . PHP_EOL;
 }
 unset( $multisite );
@@ -87 +89 @@
 // Preset WordPress options defined in bootstrap file.
 // Used to activate themes, plugins, as well as  other settings.
-if(isset($GLOBALS['wp_tests_options'])) {
+if ( isset( $GLOBALS['wp_tests_options'] ) ) {
     function wp_tests_options( $value ) {
         $key = substr( current_filter(), strlen( 'pre_option_' ) );
-        return $GLOBALS['wp_tests_options'][$key];
+        return $GLOBALS['wp_tests_options'][ $key ];
     }
 
     foreach ( array_keys( $GLOBALS['wp_tests_options'] ) as $key ) {
-        tests_add_filter( 'pre_option_'.$key, 'wp_tests_options' );
+        tests_add_filter( 'pre_option_' . $key, 'wp_tests_options' );
     }
 }
@@ -143 +145 @@
                     PHPUnit_Util_Getopt::parseLongOption( substr( $arg, 2 ), $this->longOptions, $options, $argv );
                 }
-            }
-            catch ( PHPUnit_Framework_Exception $e ) {
+            } catch ( PHPUnit_Framework_Exception $e ) {
                 // Enforcing recognized arguments or correctly formed arguments is
                 // not really the concern here.
@@ -152 +153 @@
 
         $skipped_groups = array(
-            'ajax' => true,
-            'ms-files' => true,
+            'ajax'          => true,
+            'ms-files'      => true,
             'external-http' => true,
         );
@@ -159 +160 @@
         foreach ( $options as $option ) {
             switch ( $option[0] ) {
-                case '--exclude-group' :
+                case '--exclude-group':
                     foreach ( $skipped_groups as $group_name => $skipped ) {
                         $skipped_groups[ $group_name ] = false;
                     }
                     continue 2;
-                case '--group' :
+                case '--group':
                     $groups = explode( ',', $option[1] );
                     foreach ( $groups as $group ) {
@@ -192 +193 @@
             echo PHP_EOL;
         }
-    }
+    }
 }
 new WP_PHPUnit_Util_Getopt( $_SERVER['argv'] );