Changeset 9368

Timestamp:

01/17/2015 03:24:11 AM (10 years ago)

Author:

imath

Message:

Fix at mention notifications unit tests for WordPress < 4.0

Fixes #6129

File:

• trunk/tests/phpunit/testcases/activity/notifications.php (modified) (2 diffs)

trunk/tests/phpunit/testcases/activity/notifications.php

@@ -18 +18 @@
         $this->u2 = $this->factory->user->create();
         $this->set_current_user( $this->u1 );
+
+        /**
+         * Tests suite in WP < 4.0 does not include the WP_UnitTestCase->_restore_hooks() function
+         * When updating an activity, the following filter is fired to prevent sending more than one
+         * notification. Once we've reached this filter all at_mentions tests fails so we need to
+         * temporarly remove it and restore it in $this->tearDown()
+         */
+        remove_filter( 'bp_activity_at_name_do_notifications', '__return_false' );
     }
 
@@ -23 +31 @@
         $this->set_current_user( $this->current_user );
         parent::tearDown();
+
+        // Restore the filter
+        add_filter( 'bp_activity_at_name_do_notifications', '__return_false' );
     }