Changeset 55745 for trunk/tests/phpunit/tests/user/query.php

Timestamp:

05/11/2023 10:05:51 AM (19 months ago)

Author:

spacedmonkey

Message:

Tests: Use the function get_num_queries across all unit tests.

Replace use of $wpdb->num_queries with a function call to get_num_queries. This improves readability and consistency between tests.

Props SergeyBiryukov, peterwilsoncc, spacedmonkey.
See #57841.

File:

• trunk/tests/phpunit/tests/user/query.php (modified) (2 diffs)

trunk/tests/phpunit/tests/user/query.php

@@ -1721 +1721 @@
      */
     public function test_users_pre_query_filter_should_bypass_database_query() {
-        global $wpdb;
-
         add_filter( 'users_pre_query', array( __CLASS__, 'filter_users_pre_query' ), 10, 2 );
 
-        $num_queries = $wpdb->num_queries;
+        $num_queries = get_num_queries();
         $q           = new WP_User_Query(
             array(
@@ -1735 +1733 @@
 
         // Make sure no queries were executed.
-        $this->assertSame( $num_queries, $wpdb->num_queries );
+        $this->assertSame( $num_queries, get_num_queries() );
 
         // We manually inserted a non-existing user and overrode the results with it.