Changeset 52010 for trunk/tests/phpunit/tests/customize/selective-refresh-ajax.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/selective-refresh-ajax.php (modified) (18 diffs)

trunk/tests/phpunit/tests/customize/selective-refresh-ajax.php

@@ -35 +35 @@
      * Set up the test fixture.
      */
-    function set_up() {
+    public function set_up() {
         parent::set_up();
 
@@ -53 +53 @@
      * Do Customizer boot actions.
      */
-    function do_customize_boot_actions() {
+    private function do_customize_boot_actions() {
         $_SERVER['REQUEST_METHOD'] = 'POST';
         do_action( 'setup_theme' );
@@ -68 +68 @@
      * @see WP_Customize_Selective_Refresh::handle_render_partials_request()
      */
-    function test_handle_render_partials_request_for_unauthenticated_user() {
+    public function test_handle_render_partials_request_for_unauthenticated_user() {
         $_POST[ WP_Customize_Selective_Refresh::RENDER_QUERY_VAR ] = '1';
 
@@ -124 +124 @@
      * Set the current user to be an admin, add the preview nonce, and set the query var.
      */
-    function setup_valid_render_partials_request_environment() {
+    private function setup_valid_render_partials_request_environment() {
         wp_set_current_user( self::factory()->user->create( array( 'role' => 'administrator' ) ) );
         $_REQUEST['nonce'] = wp_create_nonce( 'preview-customize_' . $this->wp_customize->theme()->get_stylesheet() );
@@ -136 +136 @@
      * @see WP_Customize_Selective_Refresh::handle_render_partials_request()
      */
-    function test_handle_render_partials_request_for_unrecognized_partial() {
+    public function test_handle_render_partials_request_for_unrecognized_partial() {
         $this->setup_valid_render_partials_request_environment();
         $context_data = array();
@@ -173 +173 @@
      * @see WP_Customize_Selective_Refresh::handle_render_partials_request()
      */
-    function test_handle_render_partials_request_for_non_rendering_partial() {
+    public function test_handle_render_partials_request_for_non_rendering_partial() {
         $this->setup_valid_render_partials_request_environment();
         wp_set_current_user( self::factory()->user->create( array( 'role' => 'administrator' ) ) );
@@ -212 +212 @@
      * @see WP_Customize_Selective_Refresh::handle_render_partials_request()
      */
-    function test_handle_rendering_disallowed_partial() {
+    public function test_handle_rendering_disallowed_partial() {
         $this->setup_valid_render_partials_request_environment();
         wp_set_current_user( self::factory()->user->create( array( 'role' => 'administrator' ) ) );
@@ -248 +248 @@
      * @see WP_Customize_Selective_Refresh::handle_render_partials_request()
      */
-    function test_handle_rendering_partial_with_missing_settings() {
+    public function test_handle_rendering_partial_with_missing_settings() {
         $this->setup_valid_render_partials_request_environment();
         wp_set_current_user( self::factory()->user->create( array( 'role' => 'administrator' ) ) );
@@ -280 +280 @@
      * @return string
      */
-    function render_callback_blogname( $partial, $context ) {
+    public function render_callback_blogname( $partial, $context ) {
         $this->assertIsArray( $context );
         $this->assertInstanceOf( 'WP_Customize_Partial', $partial );
@@ -293 +293 @@
      * @return string
      */
-    function render_callback_blogdescription( $partial, $context ) {
+    public function render_callback_blogdescription( $partial, $context ) {
         $this->assertIsArray( $context );
         $this->assertInstanceOf( 'WP_Customize_Partial', $partial );
@@ -305 +305 @@
      * @see WP_Customize_Selective_Refresh::handle_render_partials_request()
      */
-    function test_handle_render_partials_request_with_single_valid_placement() {
+    public function test_handle_render_partials_request_with_single_valid_placement() {
         $this->setup_valid_render_partials_request_environment();
 
@@ -354 +354 @@
      * @return array|false Args.
      */
-    function filter_customize_dynamic_partial_args( $partial_args, $partial_id ) {
+    public function filter_customize_dynamic_partial_args( $partial_args, $partial_id ) {
         if ( 'test_dynamic_blogname' === $partial_id ) {
             $partial_args = array(
@@ -374 +374 @@
      * @return array Response.
      */
-    function filter_customize_render_partials_response( $response, $component, $partial_placements ) {
+    public function filter_customize_render_partials_response( $response, $component, $partial_placements ) {
         $this->assertIsArray( $response );
         $this->assertInstanceOf( 'WP_Customize_Selective_Refresh', $component );
@@ -396 +396 @@
      * @param array                          $partial_placements Partial IDs.
      */
-    function handle_action_customize_render_partials_after( $component, $partial_placements ) {
+    public function handle_action_customize_render_partials_after( $component, $partial_placements ) {
         $this->assertInstanceOf( 'WP_Customize_Selective_Refresh', $component );
         if ( isset( $this->expected_partial_ids ) ) {
@@ -409 +409 @@
      * @param array                          $partial_placements Partial IDs.
      */
-    function handle_action_customize_render_partials_before( $component, $partial_placements ) {
+    public function handle_action_customize_render_partials_before( $component, $partial_placements ) {
         $this->assertInstanceOf( 'WP_Customize_Selective_Refresh', $component );
         if ( isset( $this->expected_partial_ids ) ) {
@@ -421 +421 @@
      * @see WP_Customize_Selective_Refresh::handle_render_partials_request()
      */
-    function test_handle_render_partials_request_for_dynamic_partial() {
+    public function test_handle_render_partials_request_for_dynamic_partial() {
         $this->setup_valid_render_partials_request_environment();
         add_filter( 'customize_dynamic_partial_args', array( $this, 'filter_customize_dynamic_partial_args' ), 10, 2 );
@@ -459 +459 @@
      * @see WP_Customize_Selective_Refresh::handle_render_partials_request()
      */
-    function test_handle_render_partials_request_for_multiple_partials_placements() {
+    public function test_handle_render_partials_request_for_multiple_partials_placements() {
         $this->setup_valid_render_partials_request_environment();
 
@@ -510 +510 @@
      * Tear down.
      */
-    function tear_down() {
+    public function tear_down() {
         $this->expected_partial_ids = null;
         $this->wp_customize         = null;