Changeset 52010 for trunk/tests/phpunit/tests/dependencies/jquery.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/dependencies/jquery.php (modified) (4 diffs)

trunk/tests/phpunit/tests/dependencies/jquery.php

@@ -10 +10 @@
          * @covers WP_Scripts::query
          */
-        function test_location_of_jquery() {
+        public function test_location_of_jquery() {
                 $scripts = new WP_Scripts;
                 wp_default_scripts( $scripts );
@@ -47 +47 @@
          * @covers ::wp_script_is
          */
-        function test_dont_allow_deregister_core_scripts_in_admin() {
+        public function test_dont_allow_deregister_core_scripts_in_admin() {
                 set_current_screen( 'edit.php' );
                 $this->assertTrue( is_admin() );
@@ -90 +90 @@
          * @covers ::wp_script_is
          */
-        function test_wp_script_is_dep_enqueued() {
+        public function test_wp_script_is_dep_enqueued() {
                 wp_enqueue_script( 'jquery-ui-accordion' );
 
@@ -106 +106 @@
          * @covers WP_Scripts::do_items
          */
-        function test_jquery_in_footer() {
+        public function test_jquery_in_footer() {
                 $scripts = new WP_Scripts;
                 $scripts->add( 'jquery', false, array( 'jquery-core', 'jquery-migrate' ) );