Changeset 38678

Timestamp:

09/30/2016 03:15:36 AM (10 years ago)

Author:

boonebgorges

Message:

Tests: Reset post-related globals after each test.

Globals like $pages were leaking between tests, resulting in various
bits of weirdness.

Globals will kill WordPress. Globals are killing WordPress.

See #38196.

Location:

trunk/tests/phpunit

Files:

• includes/testcase.php (modified) (2 diffs)
• tests/includes/helpers.php (modified) (1 diff)

trunk/tests/phpunit/includes/testcase.php

@@ -143 +143 @@
          */
         function tearDown() {
-                global $wpdb, $wp_query, $wp, $post;
+                global $wpdb, $wp_query, $wp;
                 $wpdb->query( 'ROLLBACK' );
                 if ( is_multisite() ) {
@@ -152 +152 @@
                 $wp_query = new WP_Query();
                 $wp = new WP();
-                $post = null;
+
+                // Reset globals related to the post loop and `setup_postdata()`.
+                $post_globals = array( 'post', 'id', 'authordata', 'currentday', 'currentmonth', 'page', 'pages', 'multipage', 'more', 'numpages' );
+                foreach ( $post_globals as $global ) {
+                        $GLOBALS[ $global ] = null;
+                }
+
                 remove_theme_support( 'html5' );
                 remove_filter( 'query', array( $this, '_create_temporary_tables' ) );

trunk/tests/phpunit/tests/includes/helpers.php

@@ -213 +213 @@
                 wp_die( new WP_Error( 'test', 'test' ) );
         }
+
+        /**
+         * This test is just a setup for the one that follows.
+         *
+         * @ticket 38196
+         */
+        public function test_setup_postdata_globals_should_be_reset_on_teardown__setup() {
+                $post = self::factory()->post->create_and_get();
+                $GLOBALS['wp_query'] = new WP_Query();
+                $GLOBALS['wp_query']->setup_postdata( $post );
+                $this->assertNotEmpty( $post );
+        }
+
+        /**
+         * @ticket 38196
+         */
+        public function test_setup_postdata_globals_should_be_reset_on_teardown() {
+                $globals = array( 'post', 'id', 'authordata', 'currentday', 'currentmonth', 'page', 'pages', 'multipage', 'more', 'numpages' );
+
+                foreach ( $globals as $global ) {
+                        $this->assertTrue( ! isset( $GLOBALS[ $global ] ), $global );
+                }
+        }
 }