Changeset 64283

Timestamp:

10/10/2026 02:23:34 PM (4 days ago)

Author:

lancewillett

Message:

Tests: Assert unsupported sidebar REST requests.

Replace empty create and delete test stubs with assertions that unsupported requests return rest_no_route. Verify the sidebar item route accepts GET before checking DELETE.

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

Props shailu25.
See #66073.

File:

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

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

@@ -720 +720 @@
 
         /**
-         * The create_item() method does not exist for sidebar.
+         * Sidebars collection is read-only; create requests should not match a route.
          *
-         * @doesNotPerformAssertions
+         * @ticket 66073
          */
         public function test_create_item() {
-                // Controller does not implement create_item().
+                wp_set_current_user( self::$admin_id );
+
+                $request  = new WP_REST_Request( 'POST', '/wp/v2/sidebars' );
+                $response = rest_get_server()->dispatch( $request );
+
+                $this->assertErrorResponse( 'rest_no_route', $response, 404 );
         }
 
@@ -1107 +1112 @@
 
         /**
-         * The delete_item() method does not exist for sidebar.
+         * Sidebars item route is not deletable; delete requests should not match a route.
          *
-         * @doesNotPerformAssertions
+         * @ticket 66073
          */
         public function test_delete_item() {
-                // Controller does not implement delete_item().
+                wp_set_current_user( self::$admin_id );
+
+                $this->setup_sidebar(
+                        'sidebar-1',
+                        array(
+                                'name' => 'Test sidebar',
+                        )
+                );
+
+                $route = '/wp/v2/sidebars/sidebar-1';
+
+                $response = rest_get_server()->dispatch( new WP_REST_Request( 'GET', $route ) );
+                $this->assertSame( 200, $response->get_status() );
+
+                $request  = new WP_REST_Request( 'DELETE', $route );
+                $response = rest_get_server()->dispatch( $request );
+
+                $this->assertErrorResponse( 'rest_no_route', $response, 404 );
         }