Changeset 52010 for trunk/tests/phpunit/tests/widgets/wpWidgetText.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/widgets/wpWidgetText.php (modified) (20 diffs)

trunk/tests/phpunit/tests/widgets/wpWidgetText.php

@@ -34 +34 @@
      * @global WP_Customize_Manager $wp_customize
      */
-    function clean_up_global_scope() {
+    public function clean_up_global_scope() {
         global $wp_scripts, $wp_styles, $wp_customize;
         parent::clean_up_global_scope();
@@ -47 +47 @@
      * @covers WP_Widget_Text::__construct
      */
-    function test_construct() {
+    public function test_construct() {
         $widget = new WP_Widget_Text();
         $this->assertSame( 'text', $widget->id_base );
@@ -61 +61 @@
      * @covers WP_Widget_Text::_register
      */
-    function test__register() {
+    public function test__register() {
         set_current_screen( 'widgets.php' );
         $widget = new WP_Widget_Text();
@@ -79 +79 @@
      * @covers WP_Widget_Text::_register
      */
-    function test__register_in_customize_preview() {
+    public function test__register_in_customize_preview() {
         global $wp_customize;
         wp_set_current_user(
@@ -108 +108 @@
      * @covers WP_Widget_Text::enqueue_preview_scripts
      */
-    function test_enqueue_preview_scripts() {
+    public function test_enqueue_preview_scripts() {
         global $wp_scripts, $wp_styles;
         $wp_scripts = null;
@@ -130 +130 @@
      * @covers WP_Widget_Text::widget
      */
-    function test_widget() {
+    public function test_widget() {
         $widget = new WP_Widget_Text();
         $text   = "Lorem ipsum dolor sit amet, consectetur adipiscing elit.\n Praesent ut turpis consequat lorem volutpat bibendum vitae vitae ante.";
@@ -303 +303 @@
      * @return string Shortcode content.
      */
-    function do_example_shortcode() {
+    public function do_example_shortcode() {
         $this->post_during_shortcode = get_post();
         $this->shortcode_render_count++;
@@ -314 +314 @@
      * @covers WP_Widget_Text::widget
      */
-    function test_widget_shortcodes() {
+    public function test_widget_shortcodes() {
         global $post;
         $post_id = $this->factory()->post->create();
@@ -439 +439 @@
      * @return string Widget text.
      */
-    function filter_widget_text( $widget_text, $instance, $widget ) {
+    public function filter_widget_text( $widget_text, $instance, $widget ) {
         $this->widget_text_args = func_get_args();
 
@@ -454 +454 @@
      * @return string Widget content.
      */
-    function filter_widget_text_content( $widget_text, $instance, $widget ) {
+    public function filter_widget_text_content( $widget_text, $instance, $widget ) {
         $this->widget_text_content_args = func_get_args();
 
@@ -466 +466 @@
      * @covers WP_Widget_Text::is_legacy_instance
      */
-    function test_is_legacy_instance() {
+    public function test_is_legacy_instance() {
         $widget        = new WP_Widget_Text();
         $base_instance = array(
@@ -631 +631 @@
      * @covers WP_Widget_Text::form
      */
-    function test_form() {
+    public function test_form() {
         add_filter( 'user_can_richedit', '__return_true' );
         $widget = new WP_Widget_Text();
@@ -707 +707 @@
      * @covers WP_Widget_Text::update
      */
-    function test_update() {
+    public function test_update() {
         $widget   = new WP_Widget_Text();
         $instance = array(
@@ -756 +756 @@
      * @covers WP_Widget_Text::update
      */
-    function test_update_legacy() {
+    public function test_update_legacy() {
         $widget = new WP_Widget_Text();
 
@@ -954 +954 @@
      * @return array Caps.
      */
-    function grant_unfiltered_html_cap( $caps, $cap ) {
+    public function grant_unfiltered_html_cap( $caps, $cap ) {
         if ( 'unfiltered_html' === $cap ) {
             $caps   = array_diff( $caps, array( 'do_not_allow' ) );
@@ -969 +969 @@
      * @return array Caps.
      */
-    function revoke_unfiltered_html_cap( $caps, $cap ) {
+    public function revoke_unfiltered_html_cap( $caps, $cap ) {
         if ( 'unfiltered_html' === $cap ) {
             $caps   = array_diff( $caps, array( 'unfiltered_html' ) );
@@ -982 +982 @@
      * @covers WP_Widget_Text::enqueue_admin_scripts
      */
-    function test_enqueue_admin_scripts() {
+    public function test_enqueue_admin_scripts() {
         set_current_screen( 'widgets.php' );
         $widget = new WP_Widget_Text();
@@ -995 +995 @@
      * @covers WP_Widget_Text::render_control_template_scripts
      */
-    function test_render_control_template_scripts() {
+    public function test_render_control_template_scripts() {
         ob_start();
         WP_Widget_Text::render_control_template_scripts();
@@ -1008 +1008 @@
      * @ticket 46421
      */
-    function test_render_links_with_target() {
+    public function test_render_links_with_target() {
         $widget = new WP_Widget_Text();
 
@@ -1035 +1035 @@
      * @ticket 46421
      */
-    function test_render_links_without_target() {
+    public function test_render_links_without_target() {
         $widget = new WP_Widget_Text();