Changeset 52010 for trunk/tests/phpunit/tests/customize/section.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/customize/section.php (modified) (14 diffs)

trunk/tests/phpunit/tests/customize/section.php

@@ -20 +20 @@
         protected $manager;
 
-        function set_up() {
+        public function set_up() {
                 parent::set_up();
                 require_once ABSPATH . WPINC . '/class-wp-customize-manager.php';
@@ -28 +28 @@
         }
 
-        function tear_down() {
+        public function tear_down() {
                 $this->manager = null;
                 unset( $GLOBALS['wp_customize'] );
@@ -37 +37 @@
          * @see WP_Customize_Section::__construct()
          */
-        function test_construct_default_args() {
+        public function test_construct_default_args() {
                 $section = new WP_Customize_Section( $this->manager, 'foo' );
                 $this->assertIsInt( $section->instance_number );
@@ -55 +55 @@
          * @see WP_Customize_Section::__construct()
          */
-        function test_construct_custom_args() {
+        public function test_construct_custom_args() {
                 $args = array(
                         'priority'        => 200,
@@ -78 +78 @@
          * @see WP_Customize_Section::__construct()
          */
-        function test_construct_custom_type() {
+        public function test_construct_custom_type() {
                 $section = new Custom_Section_Test( $this->manager, 'foo' );
                 $this->assertSame( 'titleless', $section->type );
@@ -87 +87 @@
          * @see WP_Customize_Section::active_callback()
          */
-        function test_active() {
+        public function test_active() {
                 $section = new WP_Customize_Section( $this->manager, 'foo' );
                 $this->assertTrue( $section->active() );
@@ -108 +108 @@
          * @return bool
          */
-        function filter_active_test( $active, $section ) {
+        public function filter_active_test( $active, $section ) {
                 $this->assertFalse( $active );
                 $this->assertInstanceOf( 'WP_Customize_Section', $section );
@@ -118 +118 @@
          * @see WP_Customize_Section::json()
          */
-        function test_json() {
+        public function test_json() {
                 $args = array(
                         'priority'        => 200,
@@ -146 +146 @@
          * @see WP_Customize_Section::check_capabilities()
          */
-        function test_check_capabilities() {
+        public function test_check_capabilities() {
                 wp_set_current_user( self::$admin_id );
 
@@ -163 +163 @@
          * @see WP_Customize_Section::get_content()
          */
-        function test_get_content() {
+        public function test_get_content() {
                 $section = new WP_Customize_Section( $this->manager, 'foo' );
                 $this->assertEmpty( $section->get_content() );
@@ -171 +171 @@
          * @see WP_Customize_Section::maybe_render()
          */
-        function test_maybe_render() {
+        public function test_maybe_render() {
                 wp_set_current_user( self::$admin_id );
                 $section                        = new WP_Customize_Section( $this->manager, 'bar' );
@@ -189 +189 @@
          * @param WP_Customize_Section $section
          */
-        function action_customize_render_section_test( $section ) {
+        public function action_customize_render_section_test( $section ) {
                 $this->assertInstanceOf( 'WP_Customize_Section', $section );
         }
@@ -196 +196 @@
          * @see WP_Customize_Section::print_template()
          */
-        function test_print_templates_standard() {
+        public function test_print_templates_standard() {
                 wp_set_current_user( self::$admin_id );
 
@@ -211 +211 @@
          * @see WP_Customize_Section::print_template()
          */
-        function test_print_templates_custom() {
+        public function test_print_templates_custom() {
                 wp_set_current_user( self::$admin_id );