unreads: Exclude muted DM conversations in combined-feed and all-dm counts

Author: chrisbobbe

lib/model/unreads.dart

@@ -230,7 +230,8 @@ class Unreads extends PerAccountStoreBase with ChangeNotifier {
 
   int countInAllDms() {
     int c = 0;
-    for (final messageIds in dms.values) {
+    for (final MapEntry(key: narrow, value: messageIds) in dms.entries) {
+      if (channelStore.shouldMuteDmConversation(narrow)) continue;
       c += messageIds.length;
     }
     return c;

test/model/unreads_test.dart

@@ -191,16 +191,18 @@ void main() {
       await store.addSubscription(eg.subscription(stream2));
       await store.addSubscription(eg.subscription(stream3, isMuted: true));
       await store.setUserTopic(stream1, 'a', UserTopicVisibilityPolicy.muted);
+      await store.setMutedUsers([eg.thirdUser.userId]);
       fillWithMessages([
         eg.streamMessage(stream: stream1, topic: 'a', flags: []),
         eg.streamMessage(stream: stream1, topic: 'b', flags: []),
         eg.streamMessage(stream: stream1, topic: 'b', flags: []),
         eg.streamMessage(stream: stream2, topic: 'c', flags: []),
         eg.streamMessage(stream: stream3, topic: 'd', flags: []),
         eg.dmMessage(from: eg.otherUser, to: [eg.selfUser], flags: []),
+        // Exclude because user is muted
         eg.dmMessage(from: eg.thirdUser, to: [eg.selfUser], flags: []),
       ]);
-      check(model.countInCombinedFeedNarrow()).equals(5);
+      check(model.countInCombinedFeedNarrow()).equals(4);
     });
 
     test('countInChannel/Narrow', () async {
@@ -274,12 +276,16 @@ void main() {
 
     test('countInAllDms', () async {
       prepare();
+      await store.setMutedUsers([eg.thirdUser.userId]);
       fillWithMessages([
+        // No one is muted: don't exclude
         eg.dmMessage(from: eg.otherUser, to: [eg.selfUser], flags: []),
+        // Everyone is muted: exclude
         eg.dmMessage(from: eg.thirdUser, to: [eg.selfUser], flags: []),
+        // One is muted, one isn't: don't exclude
         eg.dmMessage(from: eg.thirdUser, to: [eg.selfUser, eg.otherUser], flags: []),
       ]);
-      check(model.countInCombinedFeedNarrow()).equals(3);
+      check(model.countInCombinedFeedNarrow()).equals(2);
     });
   });