Changeset 52010 for trunk/tests/phpunit/tests/user/multisite.php

Timestamp:

11/04/2021 03:22:47 PM (4 years ago)

Author:

hellofromTonya

Message:

Coding Standards: Add visibility to methods in tests/phpunit/tests/.

Adds a public visibility to test fixtures, tests, data providers, and callbacks methods.

Adds a private visibility to helper methods within test classes.

Renames callbacks and helpers that previously started with a _ prefix. Why? For consistency and to leverage using the method visibility. Further naming standardizations is beyond the scope of this commit.

Props costdev, jrf, hellofromTonya.
Fixes #54177.

File:

• trunk/tests/phpunit/tests/user/multisite.php (modified) (10 diffs)

trunk/tests/phpunit/tests/user/multisite.php

@@ -12 +12 @@
     class Tests_Multisite_User extends WP_UnitTestCase {
 
-        function test_remove_user_from_blog() {
+        public function test_remove_user_from_blog() {
             $user1 = self::factory()->user->create_and_get();
             $user2 = self::factory()->user->create_and_get();
@@ -29 +29 @@
          * Test the returned data from get_blogs_of_user()
          */
-        function test_get_blogs_of_user() {
+        public function test_get_blogs_of_user() {
             $user1_id = self::factory()->user->create( array( 'role' => 'administrator' ) );
 
@@ -99 +99 @@
          * @expectedDeprecated is_blog_user
          */
-        function test_is_blog_user() {
+        public function test_is_blog_user() {
             global $wpdb;
 
@@ -120 +120 @@
         }
 
-        function test_is_user_member_of_blog() {
+        public function test_is_user_member_of_blog() {
             global $wpdb;
 
@@ -181 +181 @@
          * @ticket 23192
          */
-        function test_is_user_spammy() {
+        public function test_is_user_spammy() {
             $user_id = self::factory()->user->create(
                 array(
@@ -210 +210 @@
          * @ticket 20601
          */
-        function test_user_member_of_blog() {
+        public function test_user_member_of_blog() {
             global $wp_rewrite;
 
@@ -249 +249 @@
         }
 
-        function test_revoked_super_admin_can_be_deleted() {
+        public function test_revoked_super_admin_can_be_deleted() {
             if ( isset( $GLOBALS['super_admins'] ) ) {
                 $old_global = $GLOBALS['super_admins'];
@@ -266 +266 @@
         }
 
-        function test_revoked_super_admin_is_deleted() {
+        public function test_revoked_super_admin_is_deleted() {
             if ( isset( $GLOBALS['super_admins'] ) ) {
                 $old_global = $GLOBALS['super_admins'];
@@ -285 +285 @@
         }
 
-        function test_super_admin_cannot_be_deleted() {
+        public function test_super_admin_cannot_be_deleted() {
             if ( isset( $GLOBALS['super_admins'] ) ) {
                 $old_global = $GLOBALS['super_admins'];
@@ -304 +304 @@
          * @ticket 27205
          */
-        function test_granting_super_admins() {
+        public function test_granting_super_admins() {
             if ( isset( $GLOBALS['super_admins'] ) ) {
                 $old_global = $GLOBALS['super_admins'];