boxes: Use user_ids in autocomplete for ambiguous user mentions.

Prior to this commit, user mentions through autocomplete used only
names in the typeahead. This commit appends `user_id`s to all the
multiple instances of the same name to use the **Name|User_id** format of
user mentions. The appended user_id would serve as a distinction
between users having the same name.

Tests added and updated.

Author: prah23

tests/ui_tools/test_boxes.py

@@ -128,19 +128,19 @@ def test_generic_autocomplete_set_footer(self, mocker, write_box,
         ('@Human', 0, '@**Human Myself**'),
         ('@Human', 1, '@**Human 1**'),
         ('@Human', 2, '@**Human 2**'),
-        ('@Human', 3, '@**Human Duplicate**'),
-        ('@Human', 4, '@**Human Duplicate**'),
-        ('@Human', -1, '@**Human Duplicate**'),
-        ('@Human', -2, '@**Human Duplicate**'),
+        ('@Human', 3, '@**Human Duplicate|13**'),
+        ('@Human', 4, '@**Human Duplicate|14**'),
+        ('@Human', -1, '@**Human Duplicate|14**'),
+        ('@Human', -2, '@**Human Duplicate|13**'),
         ('@Human', -3, '@**Human 2**'),
         ('@Human', -4, '@**Human 1**'),
         ('@Human', -5, '@**Human Myself**'),
         ('@Human', -6, None),
         ('@_Human', 0, '@_**Human Myself**'),
         ('@_Human', 1, '@_**Human 1**'),
         ('@_Human', 2, '@_**Human 2**'),
-        ('@_Human', 3, '@_**Human Duplicate**'),
-        ('@_Human', 4, '@_**Human Duplicate**'),
+        ('@_Human', 3, '@_**Human Duplicate|13**'),
+        ('@_Human', 4, '@_**Human Duplicate|14**'),
         ('@H', 1, '@**Human 1**'),
         ('@Hu', 1, '@**Human 1**'),
         ('@Hum', 1, '@**Human 1**'),
@@ -167,8 +167,8 @@ def test_generic_autocomplete_set_footer(self, mocker, write_box,
         ('@', 0, '@**Human Myself**'),
         ('@', 1, '@**Human 1**'),
         ('@', 2, '@**Human 2**'),
-        ('@', 3, '@**Human Duplicate**'),
-        ('@', 4, '@**Human Duplicate**'),
+        ('@', 3, '@**Human Duplicate|13**'),
+        ('@', 4, '@**Human Duplicate|14**'),
         ('@', 5, '@*Group 1*'),
         ('@', 6, '@*Group 2*'),
         ('@', 7, '@*Group 3*'),
@@ -179,8 +179,8 @@ def test_generic_autocomplete_set_footer(self, mocker, write_box,
         ('@**', 0, '@**Human Myself**'),
         ('@**', 1, '@**Human 1**'),
         ('@**', 2, '@**Human 2**'),
-        ('@', 3, '@**Human Duplicate**'),
-        ('@', 4, '@**Human Duplicate**'),
+        ('@', 3, '@**Human Duplicate|13**'),
+        ('@', 4, '@**Human Duplicate|14**'),
         ('@**', 5, None),  # Reached last match
         ('@**', 6, None),  # Beyond end
         # Expected sequence of autocompletes from '@*' (only groups)
@@ -194,11 +194,11 @@ def test_generic_autocomplete_set_footer(self, mocker, write_box,
         ('@_', 0, '@_**Human Myself**'),  # NOTE: No silent group mention
         ('@_', 1, '@_**Human 1**'),
         ('@_', 2, '@_**Human 2**'),
-        ('@_', 3, '@_**Human Duplicate**'),
-        ('@_', 4, '@_**Human Duplicate**'),
+        ('@_', 3, '@_**Human Duplicate|13**'),
+        ('@_', 4, '@_**Human Duplicate|14**'),
         ('@_', 5, None),  # Reached last match
         ('@_', 6, None),  # Beyond end
-        ('@_', -1, '@_**Human Duplicate**'),
+        ('@_', -1, '@_**Human Duplicate|14**'),
         # Complex autocomplete prefixes.
         ('(@H', 0, '(@**Human Myself**'),
         ('(@H', 1, '(@**Human 1**'),
@@ -242,6 +242,49 @@ def test_generic_autocomplete_mentions_subscribers(self, write_box, text,
         typeahead_string = write_box.generic_autocomplete(text, state)
         assert typeahead_string == required_typeahead
 
+    @pytest.mark.parametrize('text', [
+        '@H', '@Hu', '@Hum', '@Huma', '@Human',
+        '@**', '@**H', '@**Hu', '@**Hum', '@**Huma', '@**Human'
+    ])
+    @pytest.mark.parametrize('matching_users, distinct_matching_users', [
+        (['Human Myself', 'Human 1', 'Human 2', 'Human Duplicate',
+          'Human Duplicate'],
+         ['@**Human Myself**', '@**Human 1**', '@**Human 2**',
+          '@**Human Duplicate|13**', '@**Human Duplicate|14**'])
+    ])
+    def test_generic_autocomplete_user_mentions(self, write_box, text, mocker,
+                                                distinct_matching_users,
+                                                matching_users, state=1):
+        _process_typeaheads = mocker.patch(BOXES
+                                           + '.WriteBox._process_typeaheads')
+
+        write_box.generic_autocomplete(text, state)
+
+        _process_typeaheads.assert_called_once_with(distinct_matching_users,
+                                                    state, matching_users)
+
+    @pytest.mark.parametrize('text', [
+        '@_', '@_H', '@_Hu', '@_Hum', '@_Huma', '@_Human'
+    ])
+    @pytest.mark.parametrize('matching_users, distinct_matching_users', [
+        (['Human Myself', 'Human 1', 'Human 2', 'Human Duplicate',
+          'Human Duplicate'],
+         ['@_**Human Myself**', '@_**Human 1**', '@_**Human 2**',
+          '@_**Human Duplicate|13**', '@_**Human Duplicate|14**'])
+    ])
+    def test_generic_autocomplete_silent_user_mentions(self, write_box, text,
+                                                       mocker,
+                                                       matching_users,
+                                                       distinct_matching_users,
+                                                       state=1):
+        _process_typeaheads = mocker.patch(BOXES
+                                           + '.WriteBox._process_typeaheads')
+
+        write_box.generic_autocomplete(text, state)
+
+        _process_typeaheads.assert_called_once_with(distinct_matching_users,
+                                                    state, matching_users)
+
     @pytest.mark.parametrize('text, state, required_typeahead, to_pin', [
         # With no streams pinned.
         ('#Stream', 0, '#**Stream 1**', []),  # 1st-word startswith match.

zulipterminal/ui_tools/boxes.py

@@ -445,11 +445,23 @@ def autocomplete_users(self, text: str, prefix_string: str
                                        reverse=True)
 
         user_names = [user['full_name'] for user in sorted_matching_users]
+
+        user_name_duplicates = [user_name
+                                for count, user_name in enumerate(user_names)
+                                if user_name in user_names[:count]]
+
+        user_names_with_distinct_duplicates = [
+            f"{user['full_name']}|{user['user_id']}"
+            if user['full_name'] in user_name_duplicates
+            else user['full_name']
+            for user in sorted_matching_users
+        ]
+
         extra_prefix = "{}{}".format(
             '*' if prefix_string[-1] != '*' else '',
             '*' if prefix_string[-2:] != '**' else '',
         )
-        user_typeahead = format_string(user_names,
+        user_typeahead = format_string(user_names_with_distinct_duplicates,
                                        prefix_string + extra_prefix + '{}**')
 
         return user_typeahead, user_names