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

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

@@ -6 +6 @@
 class Tests_XMLRPC_wp_getPosts extends WP_XMLRPC_UnitTestCase {
 
-    function test_invalid_username_password() {
+    public function test_invalid_username_password() {
         $result = $this->myxmlrpcserver->wp_getPosts( array( 1, 'username', 'password' ) );
         $this->assertIXRError( $result );
@@ -15 +15 @@
      * @ticket 20991
      */
-    function test_incapable_user() {
+    public function test_incapable_user() {
         $this->make_user_by_role( 'subscriber' );
 
@@ -28 +28 @@
     }
 
-    function test_capable_user() {
+    public function test_capable_user() {
         $this->make_user_by_role( 'editor' );
 
@@ -35 +35 @@
     }
 
-    function test_invalid_post_type() {
+    public function test_invalid_post_type() {
         $this->make_user_by_role( 'editor' );
 
@@ -43 +43 @@
     }
 
-    function test_filters() {
+    public function test_filters() {
         $this->make_user_by_role( 'editor' );
 
@@ -124 +124 @@
     }
 
-    function test_fields() {
+    public function test_fields() {
         $this->make_user_by_role( 'editor' );
         self::factory()->post->create();
@@ -150 +150 @@
      * @ticket 21623
      */
-    function test_search() {
+    public function test_search() {
         $this->make_user_by_role( 'editor' );