Changeset 52010 for trunk/tests/phpunit/tests/customize/panel.php

Timestamp:

11/04/2021 03:22:47 PM (3 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/panel.php (modified) (14 diffs)

trunk/tests/phpunit/tests/customize/panel.php

@@ -13 +13 @@
     protected $manager;
 
-    function set_up() {
+    public function set_up() {
         parent::set_up();
         require_once ABSPATH . WPINC . '/class-wp-customize-manager.php';
@@ -21 +21 @@
     }
 
-    function tear_down() {
+    public function tear_down() {
         $this->manager = null;
         unset( $GLOBALS['wp_customize'] );
@@ -30 +30 @@
      * @see WP_Customize_Panel::__construct()
      */
-    function test_construct_default_args() {
+    public function test_construct_default_args() {
         $panel = new WP_Customize_Panel( $this->manager, 'foo' );
         $this->assertIsInt( $panel->instance_number );
@@ -48 +48 @@
      * @see WP_Customize_Panel::__construct()
      */
-    function test_construct_custom_args() {
+    public function test_construct_custom_args() {
         $args = array(
             'priority'        => 200,
@@ -68 +68 @@
      * @see WP_Customize_Panel::__construct()
      */
-    function test_construct_custom_type() {
+    public function test_construct_custom_type() {
         $panel = new Custom_Panel_Test( $this->manager, 'foo' );
         $this->assertSame( 'titleless', $panel->type );
@@ -77 +77 @@
      * @see WP_Customize_Panel::active_callback()
      */
-    function test_active() {
+    public function test_active() {
         $panel = new WP_Customize_Panel( $this->manager, 'foo' );
         $this->assertTrue( $panel->active() );
@@ -98 +98 @@
      * @return bool
      */
-    function filter_active_test( $active, $panel ) {
+    public function filter_active_test( $active, $panel ) {
         $this->assertFalse( $active );
         $this->assertInstanceOf( 'WP_Customize_Panel', $panel );
@@ -108 +108 @@
      * @see WP_Customize_Panel::json()
      */
-    function test_json() {
+    public function test_json() {
         $args  = array(
             'priority'        => 200,
@@ -132 +132 @@
      * @see WP_Customize_Panel::check_capabilities()
      */
-    function test_check_capabilities() {
+    public function test_check_capabilities() {
         $user_id = self::factory()->user->create( array( 'role' => 'administrator' ) );
         wp_set_current_user( $user_id );
@@ -150 +150 @@
      * @see WP_Customize_Panel::get_content()
      */
-    function test_get_content() {
+    public function test_get_content() {
         $panel = new WP_Customize_Panel( $this->manager, 'foo' );
         $this->assertEmpty( $panel->get_content() );
@@ -158 +158 @@
      * @see WP_Customize_Panel::maybe_render()
      */
-    function test_maybe_render() {
+    public function test_maybe_render() {
         wp_set_current_user( self::factory()->user->create( array( 'role' => 'administrator' ) ) );
         $panel                        = new WP_Customize_Panel( $this->manager, 'bar' );
@@ -176 +176 @@
      * @param WP_Customize_Panel $panel
      */
-    function action_customize_render_panel_test( $panel ) {
+    public function action_customize_render_panel_test( $panel ) {
         $this->assertInstanceOf( 'WP_Customize_Panel', $panel );
     }
@@ -183 +183 @@
      * @see WP_Customize_Panel::print_template()
      */
-    function test_print_templates_standard() {
+    public function test_print_templates_standard() {
         wp_set_current_user( self::factory()->user->create( array( 'role' => 'administrator' ) ) );
 
@@ -201 +201 @@
      * @see WP_Customize_Panel::print_template()
      */
-    function test_print_templates_custom() {
+    public function test_print_templates_custom() {
         wp_set_current_user( self::factory()->user->create( array( 'role' => 'administrator' ) ) );