Changeset 55035 for trunk/tests/phpunit/tests/post/nav-menu.php

Timestamp:

01/06/2023 10:34:00 AM (2 years ago)

Author:

spacedmonkey

Message:

Query: Stop priming posts twice in WP_Query.

Part revert of [54352]. In cases where queries are not split, then it results in priming posts that are already loaded resulting in a superfluous database query.

Props spacedmonkey, flixos90, peterwilsoncc.
Fixes #57373.

File:

• trunk/tests/phpunit/tests/post/nav-menu.php (modified) (2 diffs)

trunk/tests/phpunit/tests/post/nav-menu.php

@@ -299 +299 @@
         wp_get_nav_menu_items( $this->menu_id );
         $queries_made = get_num_queries() - $start_num_queries;
-        $this->assertSame( 7, $queries_made, 'Only does 7 database queries when running wp_get_nav_menu_items.' );
+        $this->assertSame( 6, $queries_made, 'Only does 6 database queries when running wp_get_nav_menu_items.' );
 
         $args = $action->get_args();
@@ -340 +340 @@
         wp_get_nav_menu_items( $this->menu_id );
         $queries_made = get_num_queries() - $start_num_queries;
-        $this->assertSame( 7, $queries_made, 'Only does 7 database queries when running wp_get_nav_menu_items.' );
+        $this->assertSame( 6, $queries_made, 'Only does 6 database queries when running wp_get_nav_menu_items.' );
 
         $args = $action_terms->get_args();