Changeset 52010 for trunk/tests/phpunit/tests/formatting/cleanPre.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/formatting/cleanPre.php (modified) (4 diffs)

trunk/tests/phpunit/tests/formatting/cleanPre.php

@@ -10 +10 @@
 class Tests_Formatting_CleanPre extends WP_UnitTestCase {
 
-    function test_removes_self_closing_br_with_space() {
+    public function test_removes_self_closing_br_with_space() {
         $source = 'a b c\n<br />sldfj<br />';
         $res    = 'a b c\nsldfj';
@@ -17 +17 @@
     }
 
-    function test_removes_self_closing_br_without_space() {
+    public function test_removes_self_closing_br_without_space() {
         $source = 'a b c\n<br/>sldfj<br/>';
         $res    = 'a b c\nsldfj';
@@ -27 +27 @@
     // that replacement shouldn't happen (what if you want
     // HTML 4 output?).
-    function test_removes_html_br() {
+    public function test_removes_html_br() {
         $source = 'a b c\n<br>sldfj<br>';
         $res    = 'a b c\nsldfj';
@@ -33 +33 @@
     }
 
-    function test_removes_p() {
+    public function test_removes_p() {
         $source = "<p>isn't this exciting!</p><p>oh indeed!</p>";
         $res    = "\nisn't this exciting!\noh indeed!";