autocomplete: Exclude muted users from user-mention autocomplete

Fixes: #296

Author: sm-sayedi

lib/model/autocomplete.dart

@@ -12,6 +12,7 @@ import 'compose.dart';
 import 'emoji.dart';
 import 'narrow.dart';
 import 'store.dart';
+import 'user.dart';
 
 extension ComposeContentAutocomplete on ComposeContentController {
   AutocompleteIntent<ComposeAutocompleteQuery>? autocompleteIntent() {
@@ -449,6 +450,8 @@ class MentionAutocompleteView extends AutocompleteView<MentionAutocompleteQuery,
     required PerAccountStore store,
     required Narrow narrow,
   }) {
+    // return store.allUsers.whereNot((user) => store.isUserMuted(user.userId)).toList()
+    //   ..sort(_comparator(store: store, narrow: narrow));
     return store.allUsers.toList()
       ..sort(_comparator(store: store, narrow: narrow));
   }
@@ -649,7 +652,7 @@ class MentionAutocompleteView extends AutocompleteView<MentionAutocompleteQuery,
   }
 
   MentionAutocompleteResult? _testUser(MentionAutocompleteQuery query, User user) {
-    if (query.testUser(user, store.autocompleteViewManager.autocompleteDataCache)) {
+    if (query.testUser(user, store.autocompleteViewManager.autocompleteDataCache, store)) {
       return UserMentionAutocompleteResult(userId: user.userId);
     }
     return null;
@@ -754,9 +757,10 @@ class MentionAutocompleteQuery extends ComposeAutocompleteQuery {
       || wildcardOption.localizedCanonicalString(localizations).contains(_lowercase);
   }
 
-  bool testUser(User user, AutocompleteDataCache cache) {
+  bool testUser(User user, AutocompleteDataCache cache, UserStore userStore) {
     // TODO(#236) test email too, not just name
     if (!user.isActive) return false;
+    if (userStore.isUserMuted(user.userId)) return false;
 
     return _testName(user, cache);
   }

test/model/autocomplete_test.dart

@@ -404,9 +404,12 @@ void main() {
   });
 
   group('MentionAutocompleteQuery.testUser', () {
+    final store = eg.store(initialSnapshot: eg.initialSnapshot(
+      mutedUsers: [MutedUserItem(id: 1)]));
+
     void doCheck(String rawQuery, User user, bool expected) {
       final result = MentionAutocompleteQuery(rawQuery)
-        .testUser(user, AutocompleteDataCache());
+        .testUser(user, AutocompleteDataCache(), store);
       expected ? check(result).isTrue() : check(result).isFalse();
     }
 
@@ -416,6 +419,12 @@ void main() {
       doCheck('Full Name', eg.user(fullName: 'Full Name', isActive: true), true);
     });
 
+    test('user is always excluded when muted, regardless of other criteria', () {
+      doCheck('Full Name', eg.user(userId: 1, fullName: 'Full Name'), false);
+      // When not muted, then other criteria will be checked
+      doCheck('Full Name', eg.user(userId: 2, fullName: 'Full Name'), true);
+    });
+
     test('user is included if fullname words match the query', () {
       doCheck('', eg.user(fullName: 'Full Name'), true);
       doCheck('', eg.user(fullName: ''), true); // Unlikely case, but should not crash