Changeset 59899 for trunk/tests/phpunit/tests/rest-api/wpRestTemplateAutosavesController.php

Timestamp:

03/02/2025 10:05:08 PM (3 months ago)

Author:

TimothyBlynJacobs

Message:

REST API: Improve performance for HEAD requests.

By default, the REST API responds to HEAD rqeuests by calling the GET handler and omitting the body from the response. While convenient, this ends up performing needless work that slows down the API response time.

This commit adjusts the Core controllers to specifically handle HEAD requests by not preparing the response body.

Fixes #56481.
Props antonvlasenko, janusdev, ironprogrammer, swissspidy, spacedmonkey, mukesh27, mamaduka, timothyblynjacobs.

File:

• trunk/tests/phpunit/tests/rest-api/wpRestTemplateAutosavesController.php (modified) (3 diffs)

trunk/tests/phpunit/tests/rest-api/wpRestTemplateAutosavesController.php

@@ -34 +34 @@
      */
     const TEMPLATE_PART_POST_TYPE = 'wp_template_part';
+
+    /**
+     * @var string
+     */
+    const PARENT_POST_TYPE = 'wp_template';
 
     /**
@@ -294 +299 @@
 
     /**
+     * @ticket 56481
+     */
+    public function test_get_items_should_return_no_response_body_for_head_requests() {
+        wp_set_current_user( self::$admin_id );
+        $autosave_post_id = wp_create_post_autosave(
+            array(
+                'post_content' => 'Autosave content.',
+                'post_ID'      => self::$template_post->ID,
+                'post_type'    => self::PARENT_POST_TYPE,
+            )
+        );
+
+        $request  = new WP_REST_Request(
+            'HEAD',
+            '/wp/v2/templates/' . self::TEST_THEME . '/' . self::TEMPLATE_NAME . '/autosaves'
+        );
+        $response = rest_get_server()->dispatch( $request );
+        $this->assertSame( 200, $response->get_status(), 'Response status is 200.' );
+        $this->assertNull( $response->get_data(), 'The server should not generate a body in response to a HEAD request.' );
+    }
+
+    /**
      * Data provider for test_get_items_with_data_provider.
      *
@@ -425 +452 @@
             )
         );
+    }
+
+    /**
+     * @ticket 56481
+     */
+    public function test_get_item_should_return_no_response_body_for_head_requests() {
+        wp_set_current_user( self::$admin_id );
+
+        $autosave_post_id = wp_create_post_autosave(
+            array(
+                'post_content' => 'Autosave content.',
+                'post_ID'      => self::$template_post->ID,
+                'post_type'    => self::PARENT_POST_TYPE,
+            )
+        );
+
+        $request  = new WP_REST_Request( 'HEAD', '/wp/v2/templates/' . self::TEST_THEME . '/' . self::TEMPLATE_NAME . '/autosaves/' . $autosave_post_id );
+        $response = rest_get_server()->dispatch( $request );
+        $this->assertSame( 200, $response->get_status(), 'Response status is 200.' );
+        $this->assertNull( $response->get_data(), 'The server should not generate a body in response to a HEAD request.' );
     }