Changeset 61856 for trunk/tests/phpunit/tests/rest-api/rest-themes-controller.php

Timestamp:

03/06/2026 06:02:38 PM (2 months ago)

Author:

westonruter

Message:

REST API: Optimize themes collection response when querying active theme.

This updates WP_REST_Themes_Controller::get_items() to shortcut returning the current theme when the request explicitly queries for the active theme, avoiding expensive call to wp_get_themes().

Developed in ​https://github.com/WordPress/wordpress-develop/pull/11032

Follow up to r49925.

Props aduth, mukesh27, westonruter.
See #50152.
Fixes #64719.

File:

• trunk/tests/phpunit/tests/rest-api/rest-themes-controller.php (modified) (2 diffs)

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

@@ -164 +164 @@
      * @ticket 45016
      * @ticket 61021
-     * @ticket 62574.
+     * @ticket 62574
      */
     public function test_get_items() {
+        wp_set_current_user( self::$admin_id );
+        $request = new WP_REST_Request( 'GET', self::$themes_route );
+
+        $response = rest_get_server()->dispatch( $request );
+
+        $this->assertSame( 200, $response->get_status() );
+        $data = $response->get_data();
+
+        $fields = array(
+            '_links',
+            'author',
+            'author_uri',
+            'description',
+            'is_block_theme',
+            'name',
+            'requires_php',
+            'requires_wp',
+            'screenshot',
+            'status',
+            'stylesheet',
+            'stylesheet_uri',
+            'tags',
+            'template',
+            'template_uri',
+            'textdomain',
+            'theme_uri',
+            'version',
+        );
+        $this->assertIsArray( $data );
+        $this->assertNotEmpty( $data );
+        $this->assertSameSets( $fields, array_keys( $data[0] ) );
+
+        $this->assertContains( 'twentytwenty', wp_list_pluck( $data, 'stylesheet' ) );
+        $this->assertContains( get_stylesheet(), wp_list_pluck( $data, 'stylesheet' ) );
+    }
+
+    /**
+     * Test retrieving a collection of active themes.
+     *
+     * @ticket 64719
+     */
+    public function test_get_items_active() {
+        wp_set_current_user( self::$admin_id );
+
         $response = self::perform_active_theme_request();
 
@@ -197 +241 @@
         );
         $this->assertIsArray( $data );
-        $this->assertNotEmpty( $data );
+        $this->assertCount( 1, $data );
         $this->assertSameSets( $fields, array_keys( $data[0] ) );
+        $this->assertEquals( array( 'rest-api' ), wp_list_pluck( $data, 'stylesheet' ) );
     }