diff --git a/tests/replication/slave/storage/test_events.py b/tests/replication/slave/storage/test_events.py
index 1a88c7fb80..bc667454c1 100644
--- a/tests/replication/slave/storage/test_events.py
+++ b/tests/replication/slave/storage/test_events.py
@@ -160,7 +160,7 @@ class SlavedEventStoreTestCase(BaseSlavedStoreTestCase):
self.check(
"get_unread_event_push_actions_by_room_for_user",
[ROOM_ID, USER_ID_2, event1.event_id],
- {"highlight_count": 0, "notify_count": 0},
+ {"highlight_count": 0, "notify_count": 0, "unread_count": 0},
)
self.persist(
@@ -173,7 +173,7 @@ class SlavedEventStoreTestCase(BaseSlavedStoreTestCase):
self.check(
"get_unread_event_push_actions_by_room_for_user",
[ROOM_ID, USER_ID_2, event1.event_id],
- {"highlight_count": 0, "notify_count": 1},
+ {"highlight_count": 0, "notify_count": 1, "unread_count": 1},
)
self.persist(
@@ -188,7 +188,7 @@ class SlavedEventStoreTestCase(BaseSlavedStoreTestCase):
self.check(
"get_unread_event_push_actions_by_room_for_user",
[ROOM_ID, USER_ID_2, event1.event_id],
- {"highlight_count": 1, "notify_count": 2},
+ {"highlight_count": 1, "notify_count": 2, "unread_count": 2},
)
def test_get_rooms_for_user_with_stream_ordering(self):
diff --git a/tests/storage/test_event_push_actions.py b/tests/storage/test_event_push_actions.py
index b45bc9c115..79a88a1480 100644
--- a/tests/storage/test_event_push_actions.py
+++ b/tests/storage/test_event_push_actions.py
@@ -55,13 +55,17 @@ class EventPushActionsStoreTestCase(tests.unittest.TestCase):
user_id = "@user1235:example.com"
@defer.inlineCallbacks
- def _assert_counts(noitf_count, highlight_count):
+ def _assert_counts(unread_count, notif_count, highlight_count):
counts = yield self.store.db.runInteraction(
"", self.store._get_unread_counts_by_pos_txn, room_id, user_id, 0
)
self.assertEquals(
counts,
- {"notify_count": noitf_count, "highlight_count": highlight_count},
+ {
+ "unread_count": unread_count,
+ "notify_count": notif_count,
+ "highlight_count": highlight_count,
+ },
)
@defer.inlineCallbacks
@@ -96,23 +100,23 @@ class EventPushActionsStoreTestCase(tests.unittest.TestCase):
stream,
)
- yield _assert_counts(0, 0)
+ yield _assert_counts(0, 0, 0)
yield _inject_actions(1, PlAIN_NOTIF)
- yield _assert_counts(1, 0)
+ yield _assert_counts(1, 1, 0)
yield _rotate(2)
- yield _assert_counts(1, 0)
+ yield _assert_counts(1, 1, 0)
yield _inject_actions(3, PlAIN_NOTIF)
- yield _assert_counts(2, 0)
+ yield _assert_counts(2, 2, 0)
yield _rotate(4)
- yield _assert_counts(2, 0)
+ yield _assert_counts(2, 2, 0)
yield _inject_actions(5, PlAIN_NOTIF)
yield _mark_read(3, 3)
- yield _assert_counts(1, 0)
+ yield _assert_counts(1, 1, 0)
yield _mark_read(5, 5)
- yield _assert_counts(0, 0)
+ yield _assert_counts(0, 0, 0)
yield _inject_actions(6, PlAIN_NOTIF)
yield _rotate(7)
@@ -121,17 +125,17 @@ class EventPushActionsStoreTestCase(tests.unittest.TestCase):
table="event_push_actions", keyvalues={"1": 1}, desc=""
)
- yield _assert_counts(1, 0)
+ yield _assert_counts(1, 1, 0)
yield _mark_read(7, 7)
- yield _assert_counts(0, 0)
+ yield _assert_counts(0, 0, 0)
yield _inject_actions(8, HIGHLIGHT)
- yield _assert_counts(1, 1)
+ yield _assert_counts(1, 1, 1)
yield _rotate(9)
- yield _assert_counts(1, 1)
+ yield _assert_counts(1, 1, 1)
yield _rotate(10)
- yield _assert_counts(1, 1)
+ yield _assert_counts(1, 1, 1)
@defer.inlineCallbacks
def test_find_first_stream_ordering_after_ts(self):
|
