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

trunk/tests/phpunit/tests/ajax/CustomizeMenus.php

@@ -81 +81 @@
          * @param array  $expected_results  Expected results.
          */
-        function test_ajax_load_available_items_cap_check( $role, $expected_results ) {
+        public function test_ajax_load_available_items_cap_check( $role, $expected_results ) {
 
                 if ( 'administrator' !== $role ) {
@@ -123 +123 @@
          * }
          */
-        function data_ajax_load_available_items_cap_check() {
+        public function data_ajax_load_available_items_cap_check() {
                 return array(
                         array(
@@ -159 +159 @@
          * @param mixed $expected_results Expected results.
          */
-        function test_ajax_load_available_items_error_messages( $post_args, $expected_results ) {
+        public function test_ajax_load_available_items_error_messages( $post_args, $expected_results ) {
 
                 $_POST = array_merge(
@@ -193 +193 @@
          * }
          */
-        function data_ajax_load_available_items_error_messages() {
+        public function data_ajax_load_available_items_error_messages() {
                 return array(
                         // Testing empty obj_type and type.
@@ -269 +269 @@
          * @param array $success_status  Success status.
          */
-        function test_ajax_load_available_items_success_status( $post_args, $success_status ) {
+        public function test_ajax_load_available_items_success_status( $post_args, $success_status ) {
 
                 $_POST = array_merge(
@@ -303 +303 @@
          * }
          */
-        function data_ajax_load_available_items_success_status() {
+        public function data_ajax_load_available_items_success_status() {
                 return array(
                         array(
@@ -359 +359 @@
          * @param array $post_args POST args.
          */
-        function test2_ajax_load_available_items_structure( $post_args ) {
+        public function test2_ajax_load_available_items_structure( $post_args ) {
                 do_action( 'customize_register', $this->wp_customize );
 
@@ -437 +437 @@
          * }
          */
-        function data_ajax_load_available_items_structure() {
+        public function data_ajax_load_available_items_structure() {
                 return array(
                         array(
@@ -468 +468 @@
          * @param array  $expected_results Expected results.
          */
-        function test_ajax_search_available_items_caps_check( $role, $expected_results ) {
+        public function test_ajax_search_available_items_caps_check( $role, $expected_results ) {
 
                 if ( 'administrator' !== $role ) {
@@ -511 +511 @@
          * }
          */
-        function data_ajax_search_available_items_caps_check() {
+        public function data_ajax_search_available_items_caps_check() {
                 return array(
                         array(
@@ -547 +547 @@
          * @param array $expected_results Expected results.
          */
-        function test_ajax_search_available_items_results( $post_args, $expected_results ) {
+        public function test_ajax_search_available_items_results( $post_args, $expected_results ) {
                 do_action( 'customize_register', $this->wp_customize );
 
@@ -603 +603 @@
          * }
          */
-        function data_ajax_search_available_items_results() {
+        public function data_ajax_search_available_items_results() {
                 return array(
                         array(
@@ -640 +640 @@
          * @covers WP_Customize_Nav_Menus::ajax_insert_auto_draft_post
          */
-        function test_ajax_insert_auto_draft_post_success() {
+        public function test_ajax_insert_auto_draft_post_success() {
                 $_POST                = wp_slash(
                         array(
@@ -670 +670 @@
          * @covers WP_Customize_Nav_Menus::ajax_insert_auto_draft_post
          */
-        function test_ajax_insert_auto_draft_failures() {
+        public function test_ajax_insert_auto_draft_failures() {
                 // No nonce.
                 $_POST                = array();