Changeset 40543 for trunk/tests/phpunit/tests/user/wpAuthenticateSpamCheck.php

Timestamp:

04/23/2017 09:57:31 PM (8 years ago)

Author:

johnbillion

Message:

Build/Test Tools: Introduce skipWithoutMultisite() and skipWithMultisite() methods into the test suite.

This brings much needed uniformity to test skipping when a test requires Multisite or when a test should be excluded from running when Multisite is enabled.

Used in conjunction with the @group ms-required and @group ms-excluded notation, this removes a significant number of skipped tests from the default test suite run.

Fixes #40531

File:

• trunk/tests/phpunit/tests/user/wpAuthenticateSpamCheck.php (modified) (3 diffs)

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

@@ -10 +10 @@
      */
     function test_wp_authenticate_spam_check_returns_user_when_single_site() {
-        if ( is_multisite() ) {
-            $this->markTestSkipped( 'This test applies to single site only.' );
-        }
+        $this->skipWithMultisite();
 
         $user_id = self::factory()->user->create( array( 'role' => 'contributor' ) );
@@ -26 +24 @@
      */
     function test_wp_authenticate_spam_check_returns_user_when_not_flagged() {
-        if ( ! is_multisite() ) {
-            $this->markTestSkipped( 'This test applies to multisite only.' );
-        }
+        $this->skipWithoutMultisite();
 
         $user_id = self::factory()->user->create( array( 'role' => 'contributor' ) );
@@ -42 +38 @@
      */
     function test_wp_authenticate_spam_check_returns_wp_error_when_flagged() {
-        if ( ! is_multisite() ) {
-            $this->markTestSkipped( 'This test applies to multisite only.' );
-        }
+        $this->skipWithoutMultisite();
 
         $user_id = self::factory()->user->create( array( 'role' => 'contributor' ) );