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

trunk/tests/phpunit/tests/xmlrpc/mw/editPost.php

@@ -6 +6 @@
 class Tests_XMLRPC_mw_editPost extends WP_XMLRPC_UnitTestCase {
 
-    function test_invalid_username_password() {
+    public function test_invalid_username_password() {
         $post   = array();
         $result = $this->myxmlrpcserver->mw_editPost( array( 1, 'username', 'password', $post ) );
@@ -13 +13 @@
     }
 
-    function test_edit_own_post() {
+    public function test_edit_own_post() {
         $contributor_id = $this->make_user_by_role( 'contributor' );
         $post           = array(
@@ -31 +31 @@
     }
 
-    function test_capable_edit_others_post() {
+    public function test_capable_edit_others_post() {
         $this->make_user_by_role( 'editor' );
         $contributor_id = $this->make_user_by_role( 'contributor' );
@@ -51 +51 @@
     }
 
-    function test_incapable_edit_others_post() {
+    public function test_incapable_edit_others_post() {
         $this->make_user_by_role( 'contributor' );
         $author_id = $this->make_user_by_role( 'author' );
@@ -72 +72 @@
     }
 
-    function test_capable_reassign_author() {
+    public function test_capable_reassign_author() {
         $contributor_id = $this->make_user_by_role( 'contributor' );
         $author_id      = $this->make_user_by_role( 'author' );
@@ -92 +92 @@
     }
 
-    function test_incapable_reassign_author() {
+    public function test_incapable_reassign_author() {
         $contributor_id = $this->make_user_by_role( 'contributor' );
         $author_id      = $this->make_user_by_role( 'author' );
@@ -114 +114 @@
      * @ticket 24916
      */
-    function test_capable_reassign_author_to_self() {
+    public function test_capable_reassign_author_to_self() {
         $contributor_id = $this->make_user_by_role( 'contributor' );
         $editor_id      = $this->make_user_by_role( 'editor' );
@@ -136 +136 @@
      * @requires function imagejpeg
      */
-    function test_post_thumbnail() {
+    public function test_post_thumbnail() {
         add_theme_support( 'post-thumbnails' );
 
@@ -183 +183 @@
     }
 
-    function test_edit_basic_post_info() {
+    public function test_edit_basic_post_info() {
         $contributor_id = $this->make_user_by_role( 'contributor' );
 
@@ -234 +234 @@
      * @ticket 20662
      */
-    function test_make_post_sticky() {
+    public function test_make_post_sticky() {
         $author_id = $this->make_user_by_role( 'editor' );
 
@@ -251 +251 @@
 
     // Not allowed since [19914].
-    function test_change_post_type() {
+    public function test_change_post_type() {
         $contributor_id = $this->make_user_by_role( 'contributor' );
 
@@ -269 +269 @@
      * @ticket 16980
      */
-    function test_empty_not_null() {
+    public function test_empty_not_null() {
         $editor_id = $this->make_user_by_role( 'editor' );
 
@@ -301 +301 @@
      * @ticket 35874
      */
-    function test_draft_not_prematurely_published() {
-        if ( PHP_VERSION_ID >= 80100 ) {
-            /*
-             * For the time being, ignoring PHP 8.1 "null to non-nullable" deprecations coming in
-             * via hooked in filter functions until a more structural solution to the
-             * "missing input validation" conundrum has been architected and implemented.
-             */
-            $this->expectDeprecation();
-            $this->expectDeprecationMessageMatches( '`Passing null to parameter \#[0-9]+ \(\$[^\)]+\) of type [^ ]+ is deprecated`' );
-        }
-
+    public function test_draft_not_prematurely_published() {
         $editor_id = $this->make_user_by_role( 'editor' );