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

trunk/tests/phpunit/tests/widgets/wpWidgetMediaVideo.php

@@ -20 +20 @@
      * @global WP_Styles $wp_styles
      */
-    function clean_up_global_scope() {
+    public function clean_up_global_scope() {
         global $wp_scripts, $wp_styles;
         parent::clean_up_global_scope();
@@ -32 +32 @@
      * @covers WP_Widget_Media_Video::get_instance_schema
      */
-    function test_get_instance_schema() {
+    public function test_get_instance_schema() {
         $widget = new WP_Widget_Media_Video();
         $schema = $widget->get_instance_schema();
@@ -59 +59 @@
      * @ticket 45029
      */
-    function test_get_instance_schema_filtering() {
+    public function test_get_instance_schema_filtering() {
         $widget = new WP_Widget_Media_Video();
         $schema = $widget->get_instance_schema();
@@ -89 +89 @@
      * @covers WP_Widget_Media_Video::__construct
      */
-    function test_constructor() {
+    public function test_constructor() {
         $widget = new WP_Widget_Media_Video();
 
@@ -118 +118 @@
      * @covers WP_Widget_Media_Video::update
      */
-    function test_update() {
+    public function test_update() {
         $widget   = new WP_Widget_Media_Video();
         $instance = array();
@@ -219 +219 @@
      * @covers WP_Widget_Media_Video::inject_video_max_width_style
      */
-    function test_render_media() {
+    public function test_render_media() {
         $test_movie_file = __FILE__ . '../../data/uploads/small-video.m4v';
         $widget          = new WP_Widget_Media_Video();
@@ -306 +306 @@
      * @covers WP_Widget_Media_Video::enqueue_preview_scripts
      */
-    function test_enqueue_preview_scripts() {
+    public function test_enqueue_preview_scripts() {
         global $wp_scripts, $wp_styles;
         $widget = new WP_Widget_Media_Video();
@@ -331 +331 @@
      * @covers WP_Widget_Media_Video::enqueue_admin_scripts
      */
-    function test_enqueue_admin_scripts() {
+    public function test_enqueue_admin_scripts() {
         set_current_screen( 'widgets.php' );
         $widget = new WP_Widget_Media_Video();
@@ -344 +344 @@
      * @covers WP_Widget_Media_Video::render_control_template_scripts
      */
-    function test_render_control_template_scripts() {
+    public function test_render_control_template_scripts() {
         $widget = new WP_Widget_Media_Video();