Changeset 61984

Timestamp:

03/12/2026 08:03:36 AM (7 months ago)

Author:

gziolo

Message:

REST API: Fix finalize endpoint tests in Attachments controller.

Enable client-side media processing in the three test_finalize_item*
tests so the /finalize route is registered before dispatching requests.
Without this, the route does not exist and all assertions fail with
rest_no_route.

Extracts the repeated filter-and-reinitialize pattern into a shared
enable_client_side_media_processing() helper, used by both the
sideload and finalize tests.

Follow-up to [61982].

File:

• trunk/tests/phpunit/tests/rest-api/rest-attachments-controller.php (modified) (9 diffs)

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

@@ -193 +193 @@
 
                 parent::tear_down();
+        }
+
+        /**
+         * Enables client-side media processing and reinitializes the REST server
+         * so that the sideload and finalize routes are registered.
+         */
+        private function enable_client_side_media_processing(): void {
+                add_filter( 'wp_client_side_media_processing_enabled', '__return_true' );
+
+                global $wp_rest_server;
+                $wp_rest_server = new Spy_REST_Server();
+                do_action( 'rest_api_init', $wp_rest_server );
         }
 
@@ -3237 +3249 @@
          */
         public function test_sideload_scaled_image() {
-                add_filter( 'wp_client_side_media_processing_enabled', '__return_true' );
-                // Reinitialize REST server so the sideload route is registered.
-                global $wp_rest_server;
-                $wp_rest_server = new Spy_REST_Server();
-                do_action( 'rest_api_init', $wp_rest_server );
+                $this->enable_client_side_media_processing();
 
                 wp_set_current_user( self::$author_id );
@@ -3296 +3304 @@
          */
         public function test_sideload_scaled_image_requires_auth() {
-                add_filter( 'wp_client_side_media_processing_enabled', '__return_true' );
-                // Reinitialize REST server so the sideload route is registered.
-                global $wp_rest_server;
-                $wp_rest_server = new Spy_REST_Server();
-                do_action( 'rest_api_init', $wp_rest_server );
+                $this->enable_client_side_media_processing();
 
                 wp_set_current_user( self::$author_id );
@@ -3331 +3335 @@
          */
         public function test_sideload_route_includes_scaled_enum() {
-                add_filter( 'wp_client_side_media_processing_enabled', '__return_true' );
-                // Reinitialize REST server so the sideload route is registered.
-                global $wp_rest_server;
-                $wp_rest_server = new Spy_REST_Server();
-                do_action( 'rest_api_init', $wp_rest_server );
+                $this->enable_client_side_media_processing();
 
                 $server = rest_get_server();
@@ -3359 +3359 @@
          */
         public function test_sideload_scaled_unique_filename() {
-                add_filter( 'wp_client_side_media_processing_enabled', '__return_true' );
-                // Reinitialize REST server so the sideload route is registered.
-                global $wp_rest_server;
-                $wp_rest_server = new Spy_REST_Server();
-                do_action( 'rest_api_init', $wp_rest_server );
+                $this->enable_client_side_media_processing();
 
                 wp_set_current_user( self::$author_id );
@@ -3399 +3395 @@
          */
         public function test_sideload_scaled_unique_filename_conflict() {
-                add_filter( 'wp_client_side_media_processing_enabled', '__return_true' );
-                // Reinitialize REST server so the sideload route is registered.
-                global $wp_rest_server;
-                $wp_rest_server = new Spy_REST_Server();
-                do_action( 'rest_api_init', $wp_rest_server );
+                $this->enable_client_side_media_processing();
 
                 wp_set_current_user( self::$author_id );
@@ -3457 +3449 @@
          */
         public function test_finalize_item(): void {
+                $this->enable_client_side_media_processing();
+
                 wp_set_current_user( self::$author_id );
 
@@ -3509 +3503 @@
          */
         public function test_finalize_item_requires_auth(): void {
+                $this->enable_client_side_media_processing();
+
                 wp_set_current_user( self::$author_id );
 
@@ -3535 +3531 @@
          */
         public function test_finalize_item_invalid_id(): void {
+                $this->enable_client_side_media_processing();
+
                 wp_set_current_user( self::$author_id );