[Multiple issue assignees] fix specs

app/models/concerns/milestoneish.rb

@@ -40,7 +40,7 @@ module Milestoneish
   def issues_visible_to_user(user)
     memoize_per_user(user, :issues_visible_to_user) do
       IssuesFinder.new(user, issues_finder_params)
-        .execute.where(milestone_id: milestoneish_ids)
+        .execute.includes(:assignees).where(milestone_id: milestoneish_ids)
     end
   end
 

spec/features/unsubscribe_links_spec.rb

@@ -6,7 +6,7 @@ describe 'Unsubscribe links', feature: true do
   let(:recipient) { create(:user) }
   let(:author) { create(:user) }
   let(:project) { create(:empty_project, :public) }
-  let(:params) { { title: 'A bug!', description: 'Fix it!', assignee: recipient } }
+  let(:params) { { title: 'A bug!', description: 'Fix it!', assignees: [recipient] } }
   let(:issue) { Issues::CreateService.new(project, author, params).execute }
 
   let(:mail) { ActionMailer::Base.deliveries.last }

spec/services/merge_requests/assign_issues_service_spec.rb

@@ -21,11 +21,11 @@ describe MergeRequests::AssignIssuesService, services: true do
     expect(service.assignable_issues).to be_empty
   end
 
-   it 'ignores issues already assigned to any user' do
-     issue.assignees = [create(:user)]
+  it 'ignores issues already assigned to any user' do
+    issue.assignees = [create(:user)]
 
-     expect(service.assignable_issues).to be_empty
-   end
+    expect(service.assignable_issues).to be_empty
+  end
 
   it 'ignores all issues unless current_user is merge_request.author' do
     merge_request.update!(author: create(:user))