Changeset 52010 for trunk/tests/phpunit/tests/ajax/ManageThemes.php

Timestamp:

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

trunk/tests/phpunit/tests/ajax/ManageThemes.php

@@ -14 +14 @@
     private $theme_root;
 
-    function set_up() {
+    public function set_up() {
         parent::set_up();
 
@@ -31 +31 @@
     }
 
-    function tear_down() {
+    public function tear_down() {
         $GLOBALS['wp_theme_directories'] = $this->orig_theme_dir;
         remove_filter( 'theme_root', array( $this, 'filter_theme_root' ) );
@@ -145 +145 @@
      * @group ms-excluded
      */
-    function test_uppercase_theme_slug() {
+    public function test_uppercase_theme_slug() {
         $this->_setRole( 'administrator' );