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

Timestamp:

02/08/2018 03:20:59 PM (7 years ago)

Author:

boonebgorges

Message:

Notifications: User count query should be separately fetched and cached.

Previously, notification counts were calculated by fetching the entire
notification objects.

Props m_uysl.
See #7130.

File:

• trunk/tests/phpunit/testcases/notifications/functions.php (modified) (4 diffs)

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

@@ -36 +36 @@
 
         $this->assertFalse( wp_cache_get( 'all_for_user_' . $u, 'bp_notifications' ) );
+        $this->assertFalse( wp_cache_get( $u, 'bp_notifications_unread_count' ) );
     }
 
@@ -62 +63 @@
 
         $this->assertFalse( wp_cache_get( 'all_for_user_' . $u, 'bp_notifications' ) );
+        $this->assertFalse( wp_cache_get( $u, 'bp_notifications_unread_count' ) );
     }
 
@@ -92 +94 @@
 
         $this->assertFalse( wp_cache_get( 'all_for_user_' . $u, 'bp_notifications' ) );
+        $this->assertFalse( wp_cache_get( $u, 'bp_notifications_unread_count' ) );
     }
 
@@ -122 +125 @@
 
         $this->assertFalse( wp_cache_get( 'all_for_user_' . $u, 'bp_notifications' ) );
+        $this->assertFalse( wp_cache_get( $u, 'bp_notifications_unread_count' ) );
     }