Changeset 52374 for trunk/tests/phpunit/tests/rest-api/rest-global-styles-controller.php

Timestamp:

12/14/2021 04:29:03 PM (3 years ago)

Author:

spacedmonkey

Message:

REST API: Ensure that the get_theme_item method should respect fields param.

Fix the get_theme_item method in the WP_REST_Global_Styles_Controller class to respect the fields param context filtering and to return links.

Props spacedmonkey, hellofromtonya, peterwilsoncc, costdev.
Fixes #54595.

File:

• trunk/tests/phpunit/tests/rest-api/rest-global-styles-controller.php (modified) (3 diffs)

trunk/tests/phpunit/tests/rest-api/rest-global-styles-controller.php

@@ -153 +153 @@
         $response = rest_get_server()->dispatch( $request );
         $data     = $response->get_data();
-        unset( $data['_links'] );
-
+        $links    = $response->get_links();
         $this->assertArrayHasKey( 'settings', $data );
         $this->assertArrayHasKey( 'styles', $data );
+        $this->assertArrayHasKey( 'self', $links );
+        $this->assertStringContainsString( '/wp/v2/global-styles/themes/tt1-blocks', $links['self'][0]['href'] );
+    }
+
+    /**
+     * @covers WP_REST_Global_Styles_Controller::get_theme_item
+     * @ticket 54595
+     */
+    public function test_get_theme_item_fields() {
+        wp_set_current_user( self::$admin_id );
+        $request = new WP_REST_Request( 'GET', '/wp/v2/global-styles/themes/tt1-blocks' );
+        $request->set_param( '_fields', 'settings' );
+        $response = rest_get_server()->dispatch( $request );
+        $data     = $response->get_data();
+        $this->assertArrayHasKey( 'settings', $data );
+        $this->assertArrayNotHasKey( 'styles', $data );
     }
 
@@ -224 +239 @@
         $response = rest_get_server()->dispatch( $request );
         $data     = $response->get_data();
-        unset( $data['_links'] );
+        $links    = $response->get_links();
 
         $this->assertEquals(
@@ -238 +253 @@
             $data
         );
+
+        $this->assertArrayHasKey( 'self', $links );
+        $this->assertStringContainsString( '/wp/v2/global-styles/' . self::$global_styles_id, $links['self'][0]['href'] );
     }