Changeset 52010 for trunk/tests/phpunit/tests/xmlrpc/wp/getUsers.php

Timestamp:

11/04/2021 03:22:47 PM (3 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/xmlrpc/wp/getUsers.php (modified) (7 diffs)

trunk/tests/phpunit/tests/xmlrpc/wp/getUsers.php

@@ -7 +7 @@
 class Tests_XMLRPC_wp_getUsers extends WP_XMLRPC_UnitTestCase {
 
-    function test_invalid_username_password() {
+    public function test_invalid_username_password() {
         $results = $this->myxmlrpcserver->wp_getUsers( array( 1, 'username', 'password' ) );
         $this->assertIXRError( $results );
@@ -13 +13 @@
     }
 
-    function test_incapable_user() {
+    public function test_incapable_user() {
         $this->make_user_by_role( 'subscriber' );
 
@@ -21 +21 @@
     }
 
-    function test_capable_user() {
+    public function test_capable_user() {
         $this->make_user_by_role( 'administrator' );
 
@@ -43 +43 @@
     }
 
-    function test_invalid_role() {
+    public function test_invalid_role() {
         $administrator_id = $this->make_user_by_role( 'administrator' );
         if ( is_multisite() ) {
@@ -58 +58 @@
      * @expectedDeprecated WP_User_Query
      */
-    function test_role_filter() {
+    public function test_role_filter() {
         $author_id        = $this->make_user_by_role( 'author' );
         $editor_id        = $this->make_user_by_role( 'editor' );
@@ -80 +80 @@
     }
 
-    function test_paging_filters() {
+    public function test_paging_filters() {
         $administrator_id = $this->make_user_by_role( 'administrator' );
         if ( is_multisite() ) {
@@ -109 +109 @@
     }
 
-    function test_order_filters() {
+    public function test_order_filters() {
         $this->make_user_by_role( 'administrator' );