Changeset 52010 for trunk/tests/phpunit/tests/xmlrpc/wp/editTerm.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/xmlrpc/wp/editTerm.php (modified) (14 diffs)

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

@@ -27 +27 @@
     }
 
-    function test_invalid_username_password() {
+    public function test_invalid_username_password() {
         $result = $this->myxmlrpcserver->wp_editTerm( array( 1, 'username', 'password', 'category', 1 ) );
         $this->assertIXRError( $result );
@@ -33 +33 @@
     }
 
-    function test_empty_taxonomy() {
+    public function test_empty_taxonomy() {
         $this->make_user_by_role( 'subscriber' );
 
@@ -42 +42 @@
     }
 
-    function test_invalid_taxonomy() {
+    public function test_invalid_taxonomy() {
         $this->make_user_by_role( 'subscriber' );
 
@@ -51 +51 @@
     }
 
-    function test_incapable_user() {
+    public function test_incapable_user() {
         $this->make_user_by_role( 'subscriber' );
 
@@ -60 +60 @@
     }
 
-    function test_term_not_exists() {
+    public function test_term_not_exists() {
         $this->make_user_by_role( 'editor' );
 
@@ -69 +69 @@
     }
 
-    function test_empty_term() {
+    public function test_empty_term() {
         $this->make_user_by_role( 'editor' );
 
@@ -78 +78 @@
     }
 
-    function test_empty_term_name() {
+    public function test_empty_term_name() {
         $this->make_user_by_role( 'editor' );
 
@@ -98 +98 @@
     }
 
-    function test_parent_for_nonhierarchical() {
+    public function test_parent_for_nonhierarchical() {
         $this->make_user_by_role( 'editor' );
 
@@ -118 +118 @@
     }
 
-    function test_parent_empty() {
+    public function test_parent_empty() {
         $this->make_user_by_role( 'editor' );
 
@@ -138 +138 @@
     }
 
-    function test_parent_null() {
+    public function test_parent_null() {
         $this->make_user_by_role( 'editor' );
 
@@ -162 +162 @@
     }
 
-    function test_parent_invalid() {
+    public function test_parent_invalid() {
         $this->make_user_by_role( 'editor' );
 
@@ -182 +182 @@
     }
 
-    function test_parent_not_existing() {
+    public function test_parent_not_existing() {
         $this->make_user_by_role( 'editor' );
 
@@ -203 +203 @@
     }
 
-    function test_parent_duplicate_slug() {
+    public function test_parent_duplicate_slug() {
         $this->make_user_by_role( 'editor' );
 
@@ -224 +224 @@
     }
 
-    function test_edit_all_fields() {
+    public function test_edit_all_fields() {
         $this->make_user_by_role( 'editor' );