Changeset 38974 for trunk/tests/phpunit/tests/rest-api/rest-tags-controller.php

Timestamp:

10/27/2016 01:41:08 AM (8 years ago)

Author:

boonebgorges

Message:

REST API: Use wp_get_object_terms() when fetching terms for a post object.

The WP-API plugin originally used a custom method for fetching object
terms in a way that supported the object cache and also accepted all
parameters for get_terms(). In [38667], the internals of
wp_get_object_terms() were modified to use WP_Term_Query, thus
delivering in a native fashion the features that the API had
previously achieved bespokely.

Fixes #38504.

File:

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

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

@@ -771 +771 @@
     }
 
+    /**
+     * @ticket 38504
+     */
+    public function test_object_term_queries_are_cached() {
+        global $wpdb;
+
+        $tags = $this->factory->tag->create_many( 2 );
+        $p = $this->factory->post->create();
+        wp_set_object_terms( $p, $tags[0], 'post_tag' );
+
+        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
+        $request->set_param( 'post', $p );
+        $response = $this->server->dispatch( $request );
+        $found_1 = wp_list_pluck( $response->data, 'id' );
+
+        unset( $request, $response );
+
+        $num_queries = $wpdb->num_queries;
+
+        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
+        $request->set_param( 'post', $p );
+        $response = $this->server->dispatch( $request );
+        $found_2 = wp_list_pluck( $response->data, 'id' );
+
+        $this->assertEqualSets( $found_1, $found_2 );
+        $this->assertSame( $num_queries, $wpdb->num_queries );
+    }
+
     public function additional_field_get_callback( $object, $request ) {
         return 123;