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

trunk/tests/phpunit/tests/formatting/escAttr.php

@@ -5 +5 @@
  */
 class Tests_Formatting_EscAttr extends WP_UnitTestCase {
-    function test_esc_attr_quotes() {
+    public function test_esc_attr_quotes() {
         $attr = '"double quotes"';
         $this->assertSame( '"double quotes"', esc_attr( $attr ) );
@@ -26 +26 @@
     }
 
-    function test_esc_attr_amp() {
+    public function test_esc_attr_amp() {
         $out = esc_attr( 'foo & bar &baz;  ' );
         $this->assertSame( 'foo & bar &baz;  ', $out );