Changeset 51287 for trunk/tests/phpunit/tests/theme/wpTheme.php

Timestamp:

06/30/2021 06:21:52 PM (3 years ago)

Author:

SergeyBiryukov

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].

See #52627.

File:

• trunk/tests/phpunit/tests/theme/wpTheme.php (modified) (14 diffs)

trunk/tests/phpunit/tests/theme/wpTheme.php

@@ -10 +10 @@
  */
 class Tests_Theme_wpTheme extends WP_UnitTestCase {
-    function setUp() {
+
+    public function setUp() {
         parent::setUp();
         $this->theme_root = realpath( DIR_TESTDATA . '/themedir1' );
@@ -25 +26 @@
     }
 
-    function tearDown() {
+    public function tearDown() {
         $GLOBALS['wp_theme_directories'] = $this->orig_theme_dir;
         wp_clean_themes_cache();
@@ -33 +34 @@
 
     // Replace the normal theme root directory with our premade test directory.
-    function _theme_root( $dir ) {
+    public function _theme_root( $dir ) {
         return $this->theme_root;
     }
-    function test_new_WP_Theme_top_level() {
+
+    public function test_new_WP_Theme_top_level() {
         $theme = new WP_Theme( 'theme1', $this->theme_root );
 
@@ -55 +57 @@
     }
 
-    function test_new_WP_Theme_subdir() {
+    public function test_new_WP_Theme_subdir() {
         $theme = new WP_Theme( 'subdir/theme2', $this->theme_root );
 
@@ -77 +79 @@
      * @ticket 20313
      */
-    function test_new_WP_Theme_subdir_bad_root() {
+    public function test_new_WP_Theme_subdir_bad_root() {
         // This is what get_theme_data() does when you pass it a style.css file for a theme in a subdirectory.
         $theme = new WP_Theme( 'theme2', $this->theme_root . '/subdir' );
@@ -100 +102 @@
      * @ticket 21749
      */
-    function test_wp_theme_uris_with_spaces() {
+    public function test_wp_theme_uris_with_spaces() {
         $theme = new WP_Theme( 'theme with spaces', $this->theme_root . '/subdir' );
         // Make sure subdir/ is considered part of the stylesheet, as we must avoid encoding /'s.
@@ -117 +119 @@
      * @ticket 21969
      */
-    function test_theme_uris_with_spaces() {
+    public function test_theme_uris_with_spaces() {
         $callback = array( $this, 'filter_theme_with_spaces' );
         add_filter( 'stylesheet', $callback );
@@ -129 +131 @@
     }
 
-    function filter_theme_with_spaces() {
+    public function filter_theme_with_spaces() {
         return 'subdir/theme with spaces';
     }
@@ -136 +138 @@
      * @ticket 26873
      */
-    function test_display_method_on_get_method_failure() {
+    public function test_display_method_on_get_method_failure() {
         $theme = new WP_Theme( 'nonexistent', $this->theme_root );
         $this->assertSame( 'nonexistent', $theme->get( 'Name' ) );
@@ -147 +149 @@
      * @ticket 40820
      */
-    function test_child_theme_with_itself_as_parent_should_appear_as_broken() {
+    public function test_child_theme_with_itself_as_parent_should_appear_as_broken() {
         $theme  = new WP_Theme( 'child-parent-itself', $this->theme_root );
         $errors = $theme->errors();
@@ -161 +163 @@
      * @group ms-required
      */
-    function test_wp_theme_network_enable_single_theme() {
+    public function test_wp_theme_network_enable_single_theme() {
         $theme                  = 'testtheme-1';
         $current_allowed_themes = get_site_option( 'allowedthemes' );
@@ -178 +180 @@
      * @group ms-required
      */
-    function test_wp_theme_network_enable_multiple_themes() {
+    public function test_wp_theme_network_enable_multiple_themes() {
         $themes                 = array( 'testtheme-2', 'testtheme-3' );
         $current_allowed_themes = get_site_option( 'allowedthemes' );
@@ -201 +203 @@
      * @group ms-required
      */
-    function test_network_disable_single_theme() {
+    public function test_network_disable_single_theme() {
         $current_allowed_themes = get_site_option( 'allowedthemes' );
 
@@ -226 +228 @@
      * @group ms-required
      */
-    function test_network_disable_multiple_themes() {
+    public function test_network_disable_multiple_themes() {
         $current_allowed_themes = get_site_option( 'allowedthemes' );