Changeset 34719 for trunk/tests/phpunit/includes/testcase.php

Timestamp:

09/30/2015 01:08:49 PM (9 years ago)

Author:

boonebgorges

Message:

Prevent Multisite term tests from hitting database for 'db_version'.

[34718] introduced a 'db_version' check to term meta functions, to ensure that
they don't run when the term meta schema is not yet in place. This call to
get_option() causes a database hit during Multisite tests, due to the
presence of the WP_INSTALLING constant. See #31130. The extra database
queries are causing cache tests to fail.

In similar cases, we have markTestSkipped() when is_multisite(). Because
the term meta API is so extensive - term meta caches can be primed anywhere a
WP_Query loop is fired up - we implement a more generous workaround in this
case. To prevent get_option( 'db_version' ) from hitting the database during
multisite unit tests, we use a 'pre_option_' filter.

Heaven help us.

See #34091.

File:

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

trunk/tests/phpunit/includes/testcase.php

@@ -56 +56 @@
 
         add_filter( 'wp_mail', array( $this, 'set_wp_mail_globals' ) );
+
+        /*
+         * During multisite tests, WP_INSTALLING forces `get_option()` to miss the cache, which causes problems
+         * with our query-counting cache tests. As a workaround in the case of tests that require checking
+         * 'db_version' (such as any test that uses the Term Meta API), we filter 'pre_option_db_version' and
+         * avoid hitting the database.
+         *
+         * See #31130.
+         */
+        if ( is_multisite() ) {
+            $this->db_version = get_option( 'db_version' );
+            add_filter( 'pre_option_db_version', array( $this, 'db_version' ) );
+        }
     }
 
@@ -618 +631 @@
         }
     }
+
+    /**
+     * Return the current database version without hitting the database.
+     *
+     * This is used to bypass cache problems with some multisite tests. See #31130.
+     *
+     * @todo Don't do this anymore once #31130 is fixed.
+     *
+     * @since 4.4.0
+     */
+    public function db_version() {
+        return $this->db_version;
+    }
 }