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

Timestamp:

03/05/2018 12:57:24 PM (6 years ago)

Author:

boonebgorges

Message:

Move notification-grouping logic to MySQL.

The notification-grouping logic required to show the collapsed Notifications
toolbar - "You have 3 pending friend requests", etc - is currently done in
PHP, in a manner that requires all notifications to be loaded into memory
and processed. On sites where users have large numbers of notifications, this
operation requires extensive resources. By leaving the grouping to MySQL,
we can speed up processing in almost all situations.

This changeset also adds specific caching for grouped notifications.

Props m_uysl.
See #7130.

File:

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

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

@@ -404 +404 @@
         $this->n_args = compact( 'action', 'item_id', 'secondary_item_id', 'total_items', 'id', 'format' );
     }
+
+    /**
+     * @group cache
+     * @ticket BP7130
+     */
+    public function test_get_grouped_notifications_for_user_cache_invalidation() {
+        $u = self::factory()->user->create();
+
+        $n1 = self::factory()->notification->create( array(
+            'component_name'    => 'activity',
+            'component_action'  => 'new_at_mention',
+            'item_id'           => 99,
+            'user_id'           => $u,
+        ) );
+
+        // Prime cache.
+        $found = bp_notifications_get_grouped_notifications_for_user( $u );
+        $this->assertEquals( 1, $found[0]->total_count );
+
+        $n2 = self::factory()->notification->create( array(
+            'component_name'    => 'activity',
+            'component_action'  => 'new_at_mention',
+            'item_id'           => 100,
+            'user_id'           => $u,
+        ) );
+
+        $found = bp_notifications_get_grouped_notifications_for_user( $u );
+        $this->assertEquals( 2, $found[0]->total_count );
+    }
 }