Changeset 52010 for trunk/tests/phpunit/tests/multisite/getSpaceUsed.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/multisite/getSpaceUsed.php (modified) (3 diffs)

trunk/tests/phpunit/tests/multisite/getSpaceUsed.php

@@ -9 +9 @@
         class Tests_Multisite_GetSpaceUsed extends WP_UnitTestCase {
 
-                function test_get_space_used_switched_site() {
+                public function test_get_space_used_switched_site() {
                         $blog_id = self::factory()->blog->create();
                         switch_to_blog( $blog_id );
@@ -43 +43 @@
                  * the main site.
                  */
-                function test_get_space_used_main_site() {
+                public function test_get_space_used_main_site() {
                         $space_used = get_space_used();
 
@@ -77 +77 @@
                 }
 
-                function test_get_space_used_pre_get_spaced_used_filter() {
-                        add_filter( 'pre_get_space_used', array( $this, '_filter_space_used' ) );
+                public function test_get_space_used_pre_get_spaced_used_filter() {
+                        add_filter( 'pre_get_space_used', array( $this, 'filter_space_used' ) );
 
                         $this->assertSame( 300, get_space_used() );
 
-                        remove_filter( 'pre_get_space_used', array( $this, '_filter_space_used' ) );
+                        remove_filter( 'pre_get_space_used', array( $this, 'filter_space_used' ) );
                 }
 
-                function _filter_space_used() {
+                public function filter_space_used() {
                         return 300;
                 }