Commit 5a7f36a6 authored by Mario de la Ossa's avatar Mario de la Ossa

Fix Boards::Lists::CreateService

parent d8f0ada5
...@@ -24,12 +24,12 @@ module EE ...@@ -24,12 +24,12 @@ module EE
end end
def find_user(board) def find_user(board)
user_ids = user_finder(board).select(:user_id) user_ids = user_finder(board).execute.select(:user_id)
::User.where(id: user_ids).find(params[:assignee_id]) ::User.where(id: user_ids).find(params['assignee_id'])
end end
def user_finder(board) def user_finder(board)
@user_finder ||= Boards::UsersFinder.new(board, current_user) @user_finder ||= ::Boards::UsersFinder.new(board, current_user)
end end
end end
end end
......
require 'spec_helper'
describe Boards::Lists::CreateService do
describe '#execute' do
let(:parent) { create(:project) }
let(:board) { create(:board, project: parent) }
let(:label) { create(:label, project: parent, name: 'in-progress') }
let(:user) { create(:user) }
let(:other_user) { create(:user) }
subject(:service) { described_class.new(parent, user, 'assignee_id' => other_user.id) }
before do
parent.add_developer(user)
parent.add_developer(other_user)
stub_licensed_features(board_assignee_lists: true)
end
it 'creates a new assignee list' do
list = service.execute(board)
expect(list.list_type).to eq('assignee')
expect(list).to be_valid
end
end
end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment