Changeset 45811 for trunk/tests/phpunit/tests/rest-api/rest-users-controller.php

Timestamp:

08/15/2019 09:08:05 PM (5 years ago)

Author:

kadamwhite

Message:

REST API: Cache results of get_item_schema on controller instances for performance.

Caches the output of get_item_schema() to avoid redundant recomputation of translatable strings and other computed values. This method is called many times per item in each REST request, and the results of the method should not vary between calls.
Additional schema fields are not cached.

Props kadamwhite, joehoyle, TimothyBlynJacobs.
Fixes #47871.

File:

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

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

@@ -2523 +2523 @@
     public function test_get_item_schema_show_avatar() {
         update_option( 'show_avatars', false );
+
+        // Re-initialize the controller to cache-bust schemas from prior test runs.
+        $GLOBALS['wp_rest_server']->override_by_default = true;
+        $controller                                     = new WP_REST_Users_Controller();
+        $controller->register_routes();
+        $GLOBALS['wp_rest_server']->override_by_default = false;
+
         $request    = new WP_REST_Request( 'OPTIONS', '/wp/v2/users' );
         $response   = rest_get_server()->dispatch( $request );