Changeset 59235 for trunk/tests/phpunit/tests/rest-api/rest-posts-controller.php

Timestamp:

10/14/2024 10:20:09 PM (7 weeks ago)

Author:

peterwilsoncc

Message:

Media: Account for post ID queries in update_post_thumbnail_cache().

Updates update_post_thumbnail_cache() to account for WP_Query objects that only contain the post ID field rather than the entire post object.

This changes passes the $post value to get_post_thumbnail_id() rather than assuming the presence of the ID property. Additionally, the posts to which the thumbnail is attached are now primed prior to calling the function to avoid numerous unnecessary database queries.

The test WP_Test_REST_Posts_Controller::test_get_items_primes_parent_post_caches() is modified to account for an order of operations change for the priming of post meta caches. The cache is no longer primed in the final call to update_meta_cache() so the tests need to account for the post meta to be primed in any call to the function.

Props antpb, jorbin, khokansardar, linsoftware, mukesh27, oglekler, rajinsharwar, sumitsingh, xendo.
Fixes #59521.

File:

• trunk/tests/phpunit/tests/rest-api/rest-posts-controller.php (modified) (1 diff)

trunk/tests/phpunit/tests/rest-api/rest-posts-controller.php

@@ -1713 +1713 @@
         rest_get_server()->dispatch( $request );
 
-        $args = $filter->get_args();
-        $last = end( $args );
-        $this->assertIsArray( $last, 'The last value is not an array' );
-        $this->assertSameSets( $parent_ids, $last[1] );
+        $events = $filter->get_events();
+        $args   = wp_list_pluck( $events, 'args' );
+        $primed = false;
+        sort( $parent_ids );
+        foreach ( $args as $arg ) {
+            sort( $arg[1] );
+            if ( $parent_ids === $arg[1] ) {
+                $primed = $arg;
+                break;
+            }
+        }
+
+        $this->assertIsArray( $primed, 'The last value is not an array' );
+        $this->assertSameSets( $parent_ids, $primed[1] );
     }