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

trunk/tests/phpunit/tests/user/wpAuthenticateSpamCheck.php

@@ -9 +9 @@
      * @group ms-excluded
      */
-    function test_wp_authenticate_spam_check_returns_user_when_single_site() {
+    public function test_wp_authenticate_spam_check_returns_user_when_single_site() {
         $user_id     = self::factory()->user->create( array( 'role' => 'contributor' ) );
         $user        = new WP_User( $user_id );
@@ -21 +21 @@
      * @group ms-required
      */
-    function test_wp_authenticate_spam_check_returns_user_when_not_flagged() {
+    public function test_wp_authenticate_spam_check_returns_user_when_not_flagged() {
         $user_id     = self::factory()->user->create( array( 'role' => 'contributor' ) );
         $user        = new WP_User( $user_id );
@@ -33 +33 @@
      * @group ms-required
      */
-    function test_wp_authenticate_spam_check_returns_wp_error_when_flagged() {
+    public function test_wp_authenticate_spam_check_returns_wp_error_when_flagged() {
         $user_id = self::factory()->user->create( array( 'role' => 'contributor' ) );
         wp_update_user(