Changeset 8761 for trunk/tests/phpunit/testcases/notifications/functions.php

Timestamp:

08/05/2014 10:40:49 PM (10 years ago)

Author:

r-a-y

Message:

Allow notifications loop to accept all available parameters.

Previously, the notifications loop accepted only a subset of parameters
from the BP_Notification_Notification::get() method. This made
notification loop filtering very restrictive.

This commit opens up the notifications loop so all parameters are available
as per the BP_Notifications_Notification::get() fetch method.

Fixes #5558.

File:

• trunk/tests/phpunit/testcases/notifications/functions.php (modified) (1 diff)

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

@@ -121 +121 @@
         $this->assertEquals( 0, $n );
     }
+
+    /**
+     * @group bp_has_notifications
+     */
+    public function test_bp_has_notifications_filtering() {
+        $u1 = $this->create_user();
+        $u2 = $this->create_user();
+
+        // create a mixture of different notifications
+        $n1 = $this->factory->notification->create( array(
+            'component_name'    => 'messages',
+            'component_action'  => 'new_message',
+            'item_id'           => 99,
+            'user_id'           => $u2,
+            'secondary_item_id' => $u1,
+            'is_new'            => true,
+        ) );
+
+        $n2 = $this->factory->notification->create( array(
+            'component_name'    => 'activity',
+            'component_action'  => 'new_at_mention',
+            'item_id'           => 99,
+            'user_id'           => $u2,
+            'secondary_item_id' => $u1,
+            'is_new'            => true,
+        ) );
+
+        $n3 = $this->factory->notification->create( array(
+            'component_name'    => 'activity',
+            'component_action'  => 'new_at_mention',
+            'item_id'           => 100,
+            'user_id'           => $u2,
+            'secondary_item_id' => $u1,
+            'is_new'            => true,
+        ) );
+
+        // now fetch only activity notifications
+        bp_has_notifications( array(
+            'component_name' => 'activity',
+        ) );
+
+        // assert
+        $this->assertEquals( 2, buddypress()->notifications->query_loop->total_notification_count );
+    }
 }