Changeset 39008 for trunk/tests/phpunit/tests/user/capabilities.php

Timestamp:

10/30/2016 12:32:03 AM (10 years ago)

Author:

johnbillion

Message:

Role/Capability: Reuse a fixture in a couple more unit tests.

See #30017

File:

• trunk/tests/phpunit/tests/user/capabilities.php (modified) (3 diffs)

trunk/tests/phpunit/tests/user/capabilities.php

@@ -1383 +1383 @@
                 }
 
-                $user = new WP_User( self::factory()->user->create( array( 'role' => 'administrator' ) ) );
+                $user = self::$users['administrator'];
                 $user->add_cap( 'manage_network_users' );
                 $other_user = self::$users['subscriber'];
@@ -1389 +1389 @@
                 wp_set_current_user( $user->ID );
 
-                $this->assertTrue( current_user_can( 'edit_user', $other_user->ID ) );
+                $can_edit_user = current_user_can( 'edit_user', $other_user->ID );
+
+                $user->remove_cap( 'manage_network_users' );
+
+                $this->assertTrue( $can_edit_user );
         }
 
@@ -1398 +1402 @@
                 }
 
-                $user = new WP_User( self::factory()->user->create( array( 'role' => 'administrator' ) ) );
+                $user = self::$users['administrator'];
                 $user->add_cap( 'manage_network_users' );
-                $super_admin = new WP_User( self::factory()->user->create( array( 'role' => 'subscriber' ) ) );
+                $super_admin = self::$users['subscriber'];
                 grant_super_admin( $super_admin->ID );
 
                 wp_set_current_user( $user->ID );
 
-                $this->assertFalse( current_user_can( 'edit_user', $super_admin->ID ) );
+                $can_edit_user = current_user_can( 'edit_user', $super_admin->ID );
+
+                $user->remove_cap( 'manage_network_users' );
+                revoke_super_admin( $super_admin->ID );
+
+                $this->assertFalse( $can_edit_user );
         }