Changeset 7043

Timestamp:

05/09/2013 01:20:11 PM (11 years ago)

Author:

boonebgorges

Message:

Remove all users at the beginning of each testcase round

There's an oddity when running the test suite on multisite that causes user
1 not to be rolled back properly between tests. This messes up the unit test
sequence generator for usernames, because attempts to create new users through
the factory use duplicate user_login values. By wiping the users table at the
beginning of each test set, we ensure we're starting with a clean slate.

This is not an ideal solution - it may be a bug in the WP test suite - but it's
a harmless enough hack to get our own tests working properly for now.

File:

• trunk/tests/includes/testcase.php (modified) (2 diffs)

trunk/tests/includes/testcase.php

@@ -13 +13 @@
     public function setUp() {
         parent::setUp();
+
+        // Make sure all users are deleted
+        // There's a bug in the multisite tests that causes the
+        // transaction rollback to fail for the first user created,
+        // which busts every other attempt to create users. This is a
+        // hack workaround
+        global $wpdb;
+        $wpdb->query( "TRUNCATE TABLE {$wpdb->users}" );
+
         $this->factory = new BP_UnitTest_Factory;
     }
@@ -89 +98 @@
 
         // For BuddyPress, James.
+        $GLOBALS['bp']->loggedin_user = NULL;
         do_action( 'bp_init' );
     }