Changeset 11739 for trunk/tests/phpunit/includes/testcase.php

Timestamp:

11/03/2017 09:11:52 PM (9 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/includes/testcase.php (modified) (3 diffs)

trunk/tests/phpunit/includes/testcase.php

@@ -26 +26 @@
         protected $deactivated_components = array();
 
+        /**
+         * Cribbed from WP so that the self::factory() call comes from this class.
+         *
+         * @since 3.0.0
+         */
         public static function setUpBeforeClass() {
+                global $wpdb;
 
                 // Fake WP mail globals, to avoid errors
@@ -32 +38 @@
                 add_filter( 'wp_mail_from', array( 'BP_UnitTestCase', 'tearDown_wp_mail' ) );
 
-                parent::setUpBeforeClass();
+                $c = self::get_called_class();
+                if ( ! method_exists( $c, 'wpSetUpBeforeClass' ) ) {
+                        self::commit_transaction();
+                        return;
+                }
+
+                call_user_func( array( $c, 'wpSetUpBeforeClass' ), self::factory() );
+
+                self::commit_transaction();
         }
 
@@ -92 +106 @@
                 }
                 $this->deactivated_components = array();
+        }
+
+        /**
+         * Multisite-agnostic way to delete a user from the database.
+         *
+         * @since 3.0.0
+         */
+        public static function delete_user( $user_id ) {
+                $deleted = parent::delete_user( $user_id );
+
+                // When called in tearDownAfterClass(), BP's cleanup functions may no longer be hooked.
+                if ( bp_is_active( 'activity' ) ) {
+                        bp_activity_remove_all_user_data( $user_id );
+                }
+
+                return $deleted;
         }