Include assigned users in /reassign autocomplete

Already assigned users can sometimes be passed to /reassign as a way to
signify that this assignee is meant to be kept while unassigning the
rest.

app/assets/javascripts/gfm_auto_complete.js

@@ -276,11 +276,13 @@ class GfmAutoComplete {
             return $.fn.atwho.default.callbacks.filter(query, data, searchKey);
           }
 
-          if (command === MEMBER_COMMAND.ASSIGN || command === MEMBER_COMMAND.REASSIGN) {
+          if (command === MEMBER_COMMAND.ASSIGN) {
             // Only include members which are not assigned to Issuable currently
             return data.filter(
               member => member.type === 'User' && !assignees.includes(member.search),
             );
+          } else if (command === MEMBER_COMMAND.REASSIGN) {
+            return data.filter(member => member.type === 'User');
           } else if (command === MEMBER_COMMAND.UNASSIGN) {
             // Only include members which are assigned to Issuable currently
             return data.filter(member => assignees.includes(member.search));

app/assets/javascripts/vue_shared/components/gl_mentions.vue

@@ -65,16 +65,13 @@ const autoCompleteMap = {
           SidebarMediator.singleton?.store?.assignees?.map(assignee => assignee.username) || [];
       }
 
-      if (
-        doesCurrentLineStartWith('/assign', fullText, selectionStart) ||
-        doesCurrentLineStartWith('/reassign', fullText, selectionStart)
-      ) {
+      if (doesCurrentLineStartWith('/assign', fullText, selectionStart)) {
         return this.members.filter(
           member => member.type === 'User' && !this.assignees.includes(member.username),
         );
-      }
-
-      if (doesCurrentLineStartWith('/unassign', fullText, selectionStart)) {
+      } else if (doesCurrentLineStartWith('/reassign', fullText, selectionStart)) {
+        return this.members.filter(member => member.type === 'User');
+      } else if (doesCurrentLineStartWith('/unassign', fullText, selectionStart)) {
         return this.members.filter(member => this.assignees.includes(member.username));
       }
 

ee/spec/features/issues/gfm_autocomplete_ee_spec.rb

@@ -30,7 +30,7 @@ RSpec.describe 'GFM autocomplete EE', :js do
         wait_for_requests
       end
 
-      it 'lists users who are currently not assigned to the issue when using /reassign' do
+      it 'excludes groups when using /reassign' do
         note = find('#note-body')
         page.within '.timeline-content-form' do
           note.native.send_keys('/reas')
@@ -41,8 +41,8 @@ RSpec.describe 'GFM autocomplete EE', :js do
 
         wait_for_requests
 
-        expect(find('#at-view-users .atwho-view-ul')).not_to have_content(user.username)
         expect(find('#at-view-users .atwho-view-ul')).not_to have_content(group.name)
+        expect(find('#at-view-users .atwho-view-ul')).to have_content(user.username)
         expect(find('#at-view-users .atwho-view-ul')).to have_content(another_user.username)
       end
 
@@ -73,7 +73,7 @@ RSpec.describe 'GFM autocomplete EE', :js do
         wait_for_requests
       end
 
-      it 'lists users who are currently not assigned to the issue when using /reassign' do
+      it 'excludes groups when using /reassign' do
         note = find('#note-body')
         page.within '.timeline-content-form' do
           note.native.send_keys('/reas')
@@ -85,8 +85,8 @@ RSpec.describe 'GFM autocomplete EE', :js do
 
         wait_for_requests
 
-        expect(find('.tribute-container ul', visible: true)).not_to have_content(user.username)
         expect(find('.tribute-container ul', visible: true)).not_to have_content(group.name)
+        expect(find('.tribute-container ul', visible: true)).to have_content(user.username)
         expect(find('.tribute-container ul', visible: true)).to have_content(another_user.username)
       end