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

Timestamp:

03/06/2026 06:02:38 PM (7 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' ) );
         }