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

Timestamp:

11/04/2021 03:22:47 PM (5 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/mapMetaCap.php (modified) (13 diffs)

trunk/tests/phpunit/tests/user/mapMetaCap.php

@@ -42 +42 @@
          * @ticket 13905
          */
-        function test_capability_type_post_with_invalid_id() {
+        public function test_capability_type_post_with_invalid_id() {
                 $this->assertSame(
                         array( 'do_not_allow' ),
@@ -49 +49 @@
         }
 
-        function test_capability_type_post_with_no_extra_caps() {
+        public function test_capability_type_post_with_no_extra_caps() {
 
                 register_post_type(
@@ -89 +89 @@
         }
 
-        function test_custom_capability_type_with_map_meta_cap() {
+        public function test_custom_capability_type_with_map_meta_cap() {
                 register_post_type(
                         self::$post_type,
@@ -128 +128 @@
         }
 
-        function test_capability_type_post_with_one_renamed_cap() {
+        public function test_capability_type_post_with_one_renamed_cap() {
                 register_post_type(
                         self::$post_type,
@@ -169 +169 @@
         }
 
-        function test_capability_type_post_map_meta_cap_true_with_renamed_cap() {
+        public function test_capability_type_post_map_meta_cap_true_with_renamed_cap() {
                 register_post_type(
                         self::$post_type,
@@ -214 +214 @@
         }
 
-        function test_capability_type_post_with_all_meta_caps_renamed() {
+        public function test_capability_type_post_with_all_meta_caps_renamed() {
                 register_post_type(
                         self::$post_type,
@@ -259 +259 @@
         }
 
-        function test_capability_type_post_with_all_meta_caps_renamed_mapped() {
+        public function test_capability_type_post_with_all_meta_caps_renamed_mapped() {
                 register_post_type(
                         self::$post_type,
@@ -308 +308 @@
          * @ticket 30991
          */
-        function test_delete_posts_cap_without_map_meta_cap() {
+        public function test_delete_posts_cap_without_map_meta_cap() {
                 register_post_type(
                         self::$post_type,
@@ -323 +323 @@
         }
 
-        function test_unfiltered_html_cap() {
+        public function test_unfiltered_html_cap() {
                 if ( defined( 'DISALLOW_UNFILTERED_HTML' ) ) {
                         $this->assertFalse( DISALLOW_UNFILTERED_HTML );
@@ -339 +339 @@
          * @ticket 20488
          */
-        function test_file_edit_caps_not_reliant_on_unfiltered_html_constant() {
+        public function test_file_edit_caps_not_reliant_on_unfiltered_html_constant() {
                 $this->assertFalse( defined( 'DISALLOW_FILE_MODS' ) );
                 $this->assertFalse( defined( 'DISALLOW_FILE_EDIT' ) );
@@ -357 +357 @@
          * @ticket 27020
          */
-        function test_authorless_posts_capabilties() {
+        public function test_authorless_posts_capabilties() {
                 $post_id = self::factory()->post->create(
                         array(
@@ -377 +377 @@
          * @ticket 37580
          */
-        function test_only_users_who_can_manage_options_can_delete_page_on_front() {
+        public function test_only_users_who_can_manage_options_can_delete_page_on_front() {
                 $post_id = self::factory()->post->create(
                         array(
@@ -397 +397 @@
          * @ticket 37580
          */
-        function test_only_users_who_can_manage_options_can_delete_page_for_posts() {
+        public function test_only_users_who_can_manage_options_can_delete_page_for_posts() {
                 $post_id = self::factory()->post->create(
                         array(