Changeset 54889 for trunk/tests/phpunit/tests/theme/wpThemeJsonResolver.php

Timestamp:

11/29/2022 12:32:37 PM (4 years ago)

Author:

SergeyBiryukov

Message:

Coding Standards: Add visibility to methods in tests/phpunit/tests/.

Adds a public visibility to test fixtures, tests, data providers, and callbacks methods. This continues the previous efforts to make sure visibility is declared on all methods.

Note: This will be enforced by WPCS 3.0.0.

Follow-up to [51919], [52009], [52010].

Props jrf.
See #56791.

File:

• trunk/tests/phpunit/tests/theme/wpThemeJsonResolver.php (modified) (10 diffs)

trunk/tests/phpunit/tests/theme/wpThemeJsonResolver.php

@@ -451 +451 @@
          * @ticket 54336
          */
-        function test_add_theme_supports_are_loaded_for_themes_without_theme_json() {
+        public function test_add_theme_supports_are_loaded_for_themes_without_theme_json() {
                 switch_theme( 'default' );
                 $color_palette = array(
@@ -499 +499 @@
          * @ticket 56611
          */
-        function test_merges_child_theme_json_into_parent_theme_json() {
+        public function test_merges_child_theme_json_into_parent_theme_json() {
                 switch_theme( 'block-theme-child' );
 
@@ -620 +620 @@
          * @covers WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles
          */
-        function test_get_user_data_from_wp_global_styles_does_not_use_uncached_queries() {
+        public function test_get_user_data_from_wp_global_styles_does_not_use_uncached_queries() {
                 // Switch to a theme that does have support.
                 switch_theme( 'block-theme' );
@@ -660 +660 @@
          * @covers WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles
          */
-        function test_get_user_data_from_wp_global_styles_does_not_use_uncached_queries_for_logged_out_users() {
+        public function test_get_user_data_from_wp_global_styles_does_not_use_uncached_queries_for_logged_out_users() {
                 // Switch to a theme that does have support.
                 switch_theme( 'block-theme' );
@@ -681 +681 @@
          * @covers WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles
          */
-        function test_get_user_data_from_wp_global_styles_does_not_run_for_theme_without_support() {
+        public function test_get_user_data_from_wp_global_styles_does_not_run_for_theme_without_support() {
                 // The 'default' theme does not support theme.json.
                 switch_theme( 'default' );
@@ -703 +703 @@
          * @covers WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles
          */
-        function test_get_user_data_from_wp_global_styles_does_exist() {
+        public function test_get_user_data_from_wp_global_styles_does_exist() {
                 // Switch to a theme that does have support.
                 switch_theme( 'block-theme' );
@@ -720 +720 @@
          * @covers WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles
          */
-        function test_get_user_data_from_wp_global_styles_create_post() {
+        public function test_get_user_data_from_wp_global_styles_create_post() {
                 // Switch to a theme that does have support.
                 switch_theme( 'block-theme' );
@@ -739 +739 @@
          * @covers WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles
          */
-        function test_get_user_data_from_wp_global_styles_filter_state() {
+        public function test_get_user_data_from_wp_global_styles_filter_state() {
                 // Switch to a theme that does have support.
                 switch_theme( 'block-theme' );
@@ -755 +755 @@
          * @covers WP_Theme_JSON_Resolver::get_theme_data
          */
-        function test_get_theme_data_theme_supports_overrides_theme_json() {
+        public function test_get_theme_data_theme_supports_overrides_theme_json() {
                 // Test that get_theme_data() returns a WP_Theme_JSON object.
                 $theme_json_resolver = new WP_Theme_JSON_Resolver();
@@ -778 +778 @@
          * @covers WP_Theme_JSON_Resolver::get_theme_data
          */
-        function test_get_theme_data_does_not_parse_theme_json_if_not_present() {
+        public function test_get_theme_data_does_not_parse_theme_json_if_not_present() {
                 // The 'default' theme does not support theme.json.
                 switch_theme( 'default' );