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

Timestamp:

04/23/2017 09:57:31 PM (9 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/wpDeleteUser.php (modified) (2 diffs)

trunk/tests/phpunit/tests/user/wpDeleteUser.php

@@ -128 +128 @@
      */
     public function test_numeric_string_user_id() {
-        if ( is_multisite() ) {
-            $this->markTestSkipped( 'wp_delete_user() does not delete user records in Multisite.' );
-        }
+        $this->skipWithMultisite();
 
         $u = self::factory()->user->create();
@@ -151 +149 @@
      */
     public function test_should_return_false_for_object_user_id() {
-        if ( is_multisite() ) {
-            $this->markTestSkipped( 'wp_delete_user() does not delete user records in Multisite.' );
-        }
+        $this->skipWithMultisite();
 
         $u_obj = self::factory()->user->create_and_get();