Changeset 11739 for trunk/tests/phpunit/testcases/groups/types.php

Timestamp:

11/03/2017 09:11:52 PM (7 years ago)

Author:

boonebgorges

Message:

Ensure that shared user fixtures are fully cleaned up.

It's not possible to inherit WP 4.4's user cleanup between tests, because
the the deletion routine runs after the core test suite has unhooked
certain actions (such as BP's that are hooked to delete_user). So
we are forced to run necessary cleanup tasks in our own delete_user()
method, and ensure that it's this method that is called in every case
where we're cleaning up after statically generated shared fixtures.
Otherwise leftover content in the activity table can leak to other
tests.

See #7620.

File:

• trunk/tests/phpunit/testcases/groups/types.php (modified) (2 diffs)

trunk/tests/phpunit/testcases/groups/types.php

@@ -14 +14 @@
     }
 
-    public static function setUpBeforeClass() {
-        $f = new BP_UnitTest_Factory();
-
+    public static function wpSetUpBeforeClass( $f ) {
         self::$u1 = $f->user->create( array(
             'user_email' => 'group-types-tests@example.com',
@@ -23 +21 @@
     }
 
-    public static function tearDownAfterClass() {
-        if ( is_multisite() ) {
-            wpmu_delete_user( self::$u1 );
-        } else {
-            wp_delete_user( self::$u1 );
-        }
+    public static function wpTearDownAfterClass() {
+        self::delete_user( self::$u1 );
     }