Changeset 42427 for branches/4.9/tests/phpunit/tests/rest-api/rest-settings-controller.php

Timestamp:

01/06/2018 07:28:44 PM (7 years ago)

Author:

rachelbaker

Message:

EST API: Adjust unit testes to expect a 401 status code in error responses from permission callbacks when user is not authenticated.

Missed in [42421].

Merges [42423] to the 4.9 branch.
Fixes #42828.

Location:

branches/4.9

Files:

• . (modified) (1 prop)
• tests/phpunit/tests/rest-api/rest-settings-controller.php (modified) (3 diffs)

branches/4.9/tests/phpunit/tests/rest-api/rest-settings-controller.php

@@ -11 +11 @@
  */
 class WP_Test_REST_Settings_Controller extends WP_Test_REST_Controller_Testcase {
+    
     protected static $administrator;
+    protected static $author;
 
     public static function wpSetUpBeforeClass( $factory ) {
@@ -17 +19 @@
             'role' => 'administrator',
         ) );
+
+        self::$author        = $factory->user->create(
+            array(
+                'role' => 'author',
+            )
+        );
     }
 
     public static function wpTearDownAfterClass() {
         self::delete_user( self::$administrator );
+        self::delete_user( self::$author );
     }
 
@@ -44 +53 @@
     }
 
-    public function test_get_item_is_not_public() {
-        $request = new WP_REST_Request( 'GET', '/wp/v2/settings' );
+    public function test_get_item_is_not_public_not_authenticated() {
+        $request = new WP_REST_Request( 'GET', '/wp/v2/settings' );
+        $response = $this->server->dispatch( $request );
+        $this->assertEquals( 401, $response->get_status() );
+    }
+
+    public function test_get_item_is_not_public_no_permission() {
+        wp_set_current_user( self::$author );
+        $request  = new WP_REST_Request( 'GET', '/wp/v2/settings' );
         $response = $this->server->dispatch( $request );
         $this->assertEquals( 403, $response->get_status() );