Changeset 51305 for branches/5.8/tests/phpunit/tests/theme/wpThemeJsonResolver.php

Timestamp:

07/02/2021 12:24:08 PM (4 years ago)

Author:

desrosj

Message:

Coding Standards: Add missing visibility keywords to WP_Theme, WP_Theme_JSON, and WP_Theme_JSON_Resolver tests.

Follow-up to [50959], [50960], [50967].

Props SergeyBiryukov.
Merges [51287] to the 5.8 branch.
Fixes #52627.

Location:

branches/5.8

Files:

• . (modified) (1 prop)
• tests/phpunit/tests/theme/wpThemeJsonResolver.php (modified) (6 diffs)

branches/5.8/tests/phpunit/tests/theme/wpThemeJsonResolver.php

@@ -13 +13 @@
 class Tests_Theme_wpThemeJsonResolver extends WP_UnitTestCase {
 
-    function setUp() {
+    public function setUp() {
         parent::setUp();
         $this->theme_root = realpath( DIR_TESTDATA . '/themedir1' );
@@ -30 +30 @@
     }
 
-    function tearDown() {
+    public function tearDown() {
         $GLOBALS['wp_theme_directories'] = $this->orig_theme_dir;
         wp_clean_themes_cache();
@@ -37 +37 @@
     }
 
-    function filter_set_theme_root() {
+    public function filter_set_theme_root() {
         return $this->theme_root;
     }
 
-    function filter_set_locale_to_polish() {
+    public function filter_set_locale_to_polish() {
         return 'pl_PL';
     }
@@ -48 +48 @@
      * @ticket 52991
      */
-    function test_fields_are_extracted() {
+    public function test_fields_are_extracted() {
         $actual = WP_Theme_JSON_Resolver::get_fields_to_translate();
 
@@ -95 +95 @@
      * @ticket 52991
      */
-    function test_translations_are_applied() {
+    public function test_translations_are_applied() {
         add_filter( 'locale', array( $this, 'filter_set_locale_to_polish' ) );
         load_textdomain( 'fse', realpath( DIR_TESTDATA . '/languages/themes/fse-pl_PL.mo' ) );
@@ -149 +149 @@
      * @ticket 52991
      */
-    function test_switching_themes_recalculates_data() {
+    public function test_switching_themes_recalculates_data() {
         // By default, the theme for unit tests is "default",
         // which doesn't have theme.json support.