WordPress.org
Get WordPress

Make WordPress Core

Changeset 51287


Ignore:
Timestamp:
06/30/2021 06:21:52 PM (5 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.

Location:
trunk/tests/phpunit/tests
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/tests/rest-api/rest-sidebars-controller.php

    r51235 r51287  
    6363        }
    6464
    65         function clean_up_global_scope() {
     65        public function clean_up_global_scope() {
    6666                global $wp_widget_factory, $wp_registered_sidebars, $wp_registered_widgets, $wp_registered_widget_controls, $wp_registered_widget_updates;
    6767

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

    r50967 r51287  
    1010 */
    1111class Tests_Theme_wpTheme extends WP_UnitTestCase {
    12         function setUp() {
     12
     13        public function setUp() {
    1314                parent::setUp();
    1415                $this->theme_root = realpath( DIR_TESTDATA . '/themedir1' );
    … …  
    2526        }
    2627
    27         function tearDown() {
     28        public function tearDown() {
    2829                $GLOBALS['wp_theme_directories'] = $this->orig_theme_dir;
    2930                wp_clean_themes_cache();
    … …  
    3334
    3435        // Replace the normal theme root directory with our premade test directory.
    35         function _theme_root( $dir ) {
     36        public function _theme_root( $dir ) {
    3637                return $this->theme_root;
    3738        }
    38         function test_new_WP_Theme_top_level() {
     39
     40        public function test_new_WP_Theme_top_level() {
    3941                $theme = new WP_Theme( 'theme1', $this->theme_root );
    4042
    … …  
    5557        }
    5658
    57         function test_new_WP_Theme_subdir() {
     59        public function test_new_WP_Theme_subdir() {
    5860                $theme = new WP_Theme( 'subdir/theme2', $this->theme_root );
    5961
    … …  
    7779         * @ticket 20313
    7880         */
    79         function test_new_WP_Theme_subdir_bad_root() {
     81        public function test_new_WP_Theme_subdir_bad_root() {
    8082                // This is what get_theme_data() does when you pass it a style.css file for a theme in a subdirectory.
    8183                $theme = new WP_Theme( 'theme2', $this->theme_root . '/subdir' );
    … …  
    100102         * @ticket 21749
    101103         */
    102         function test_wp_theme_uris_with_spaces() {
     104        public function test_wp_theme_uris_with_spaces() {
    103105                $theme = new WP_Theme( 'theme with spaces', $this->theme_root . '/subdir' );
    104106                // Make sure subdir/ is considered part of the stylesheet, as we must avoid encoding /'s.
    … …  
    117119         * @ticket 21969
    118120         */
    119         function test_theme_uris_with_spaces() {
     121        public function test_theme_uris_with_spaces() {
    120122                $callback = array( $this, 'filter_theme_with_spaces' );
    121123                add_filter( 'stylesheet', $callback );
    … …  
    129131        }
    130132
    131         function filter_theme_with_spaces() {
     133        public function filter_theme_with_spaces() {
    132134                return 'subdir/theme with spaces';
    133135        }
    … …  
    136138         * @ticket 26873
    137139         */
    138         function test_display_method_on_get_method_failure() {
     140        public function test_display_method_on_get_method_failure() {
    139141                $theme = new WP_Theme( 'nonexistent', $this->theme_root );
    140142                $this->assertSame( 'nonexistent', $theme->get( 'Name' ) );
    … …  
    147149         * @ticket 40820
    148150         */
    149         function test_child_theme_with_itself_as_parent_should_appear_as_broken() {
     151        public function test_child_theme_with_itself_as_parent_should_appear_as_broken() {
    150152                $theme  = new WP_Theme( 'child-parent-itself', $this->theme_root );
    151153                $errors = $theme->errors();
    … …  
    161163         * @group ms-required
    162164         */
    163         function test_wp_theme_network_enable_single_theme() {
     165        public function test_wp_theme_network_enable_single_theme() {
    164166                $theme                  = 'testtheme-1';
    165167                $current_allowed_themes = get_site_option( 'allowedthemes' );
    … …  
    178180         * @group ms-required
    179181         */
    180         function test_wp_theme_network_enable_multiple_themes() {
     182        public function test_wp_theme_network_enable_multiple_themes() {
    181183                $themes                 = array( 'testtheme-2', 'testtheme-3' );
    182184                $current_allowed_themes = get_site_option( 'allowedthemes' );
    … …  
    201203         * @group ms-required
    202204         */
    203         function test_network_disable_single_theme() {
     205        public function test_network_disable_single_theme() {
    204206                $current_allowed_themes = get_site_option( 'allowedthemes' );
    205207
    … …  
    226228         * @group ms-required
    227229         */
    228         function test_network_disable_multiple_themes() {
     230        public function test_network_disable_multiple_themes() {
    229231                $current_allowed_themes = get_site_option( 'allowedthemes' );
    230232

  • trunk/tests/phpunit/tests/theme/wpThemeJson.php

    r51281 r51287  
    1717         * @ticket 52991
    1818         */
    19         function test_get_settings() {
     19        public function test_get_settings() {
    2020                $theme_json = new WP_Theme_JSON(
    2121                        array(
    … …  
    6868         * @ticket 53397
    6969         */
    70         function test_get_settings_presets_are_keyed_by_origin() {
     70        public function test_get_settings_presets_are_keyed_by_origin() {
    7171                $core_origin = new WP_Theme_JSON(
    7272                        array(
    … …  
    196196         * @ticket 53175
    197197         */
    198         function test_get_stylesheet() {
     198        public function test_get_stylesheet() {
    199199                $theme_json = new WP_Theme_JSON(
    200200                        array(
    … …  
    317317         * @ticket 52991
    318318         */
    319         function test_get_stylesheet_preset_classes_work_with_compounded_selectors() {
     319        public function test_get_stylesheet_preset_classes_work_with_compounded_selectors() {
    320320                $theme_json = new WP_Theme_JSON(
    321321                        array(
    … …  
    347347         * @ticket 53175
    348348         */
    349         function test_get_stylesheet_preset_rules_come_after_block_rules() {
     349        public function test_get_stylesheet_preset_rules_come_after_block_rules() {
    350350                $theme_json = new WP_Theme_JSON(
    351351                        array(
    … …  
    671671         * @ticket 52991
    672672         */
    673         function test_get_from_editor_settings() {
     673        public function test_get_from_editor_settings() {
    674674                $input = array(
    675675                        'disableCustomColors'    => true,
    … …  
    747747         * @ticket 52991
    748748         */
    749         function test_get_editor_settings_no_theme_support() {
     749        public function test_get_editor_settings_no_theme_support() {
    750750                $input = array(
    751751                        '__unstableEnableFullSiteEditingBlocks' => false,
    … …  
    802802         * @ticket 52991
    803803         */
    804         function test_get_editor_settings_blank() {
     804        public function test_get_editor_settings_blank() {
    805805                $expected = array(
    806806                        'version'  => WP_Theme_JSON::LATEST_SCHEMA,
    … …  
    815815         * @ticket 52991
    816816         */
    817         function test_get_editor_settings_custom_units_can_be_disabled() {
     817        public function test_get_editor_settings_custom_units_can_be_disabled() {
    818818                add_theme_support( 'custom-units', array() );
    819819                $input = get_default_block_editor_settings();
    … …  
    832832         * @ticket 52991
    833833         */
    834         function test_get_editor_settings_custom_units_can_be_enabled() {
     834        public function test_get_editor_settings_custom_units_can_be_enabled() {
    835835                add_theme_support( 'custom-units' );
    836836                $input = get_default_block_editor_settings();
    … …  
    849849         * @ticket 52991
    850850         */
    851         function test_get_editor_settings_custom_units_can_be_filtered() {
     851        public function test_get_editor_settings_custom_units_can_be_filtered() {
    852852                add_theme_support( 'custom-units', 'rem', 'em' );
    853853                $input = get_default_block_editor_settings();

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

    r51196 r51287  
    1313class Tests_Theme_wpThemeJsonResolver extends WP_UnitTestCase {
    1414
    15         function setUp() {
     15        public function setUp() {
    1616                parent::setUp();
    1717                $this->theme_root = realpath( DIR_TESTDATA . '/themedir1' );
    … …  
    3030        }
    3131
    32         function tearDown() {
     32        public function tearDown() {
    3333                $GLOBALS['wp_theme_directories'] = $this->orig_theme_dir;
    3434                wp_clean_themes_cache();
    … …  
    3737        }
    3838
    39         function filter_set_theme_root() {
     39        public function filter_set_theme_root() {
    4040                return $this->theme_root;
    4141        }
    4242
    43         function filter_set_locale_to_polish() {
     43        public function filter_set_locale_to_polish() {
    4444                return 'pl_PL';
    4545        }
    … …  
    4848         * @ticket 52991
    4949         */
    50         function test_fields_are_extracted() {
     50        public function test_fields_are_extracted() {
    5151                $actual = WP_Theme_JSON_Resolver::get_fields_to_translate();
    5252
    … …  
    9595         * @ticket 52991
    9696         */
    97         function test_translations_are_applied() {
     97        public function test_translations_are_applied() {
    9898                add_filter( 'locale', array( $this, 'filter_set_locale_to_polish' ) );
    9999                load_textdomain( 'fse', realpath( DIR_TESTDATA . '/languages/themes/fse-pl_PL.mo' ) );
    … …  
    149149         * @ticket 52991
    150150         */
    151         function test_switching_themes_recalculates_data() {
     151        public function test_switching_themes_recalculates_data() {
    152152                // By default, the theme for unit tests is "default",
    153153                // which doesn't have theme.json support.
Note: See TracChangeset for help on using the changeset viewer.