Commit f0211a4e authored by Grzegorz Bizon's avatar Grzegorz Bizon

Do not pass params that are not used in issue move service

parent a2f25ed6
...@@ -92,7 +92,7 @@ class Projects::IssuesController < Projects::ApplicationController ...@@ -92,7 +92,7 @@ class Projects::IssuesController < Projects::ApplicationController
if params[:move_to_project_id].to_i > 0 if params[:move_to_project_id].to_i > 0
new_project = Project.find(params[:move_to_project_id]) new_project = Project.find(params[:move_to_project_id])
move_service = Issues::MoveService.new(project, current_user, issue_params) move_service = Issues::MoveService.new(project, current_user)
@issue = move_service.execute(@issue, new_project) @issue = move_service.execute(@issue, new_project)
end end
......
...@@ -7,7 +7,6 @@ describe Issues::MoveService, services: true do ...@@ -7,7 +7,6 @@ describe Issues::MoveService, services: true do
let(:description) { 'Some issue description' } let(:description) { 'Some issue description' }
let(:old_project) { create(:project) } let(:old_project) { create(:project) }
let(:new_project) { create(:project) } let(:new_project) { create(:project) }
let(:issue_params) { old_issue.serializable_hash }
let(:old_issue) do let(:old_issue) do
create(:issue, title: title, description: description, create(:issue, title: title, description: description,
...@@ -15,7 +14,7 @@ describe Issues::MoveService, services: true do ...@@ -15,7 +14,7 @@ describe Issues::MoveService, services: true do
end end
let(:move_service) do let(:move_service) do
described_class.new(old_project, user, issue_params) described_class.new(old_project, user)
end end
shared_context 'user can move issue' do shared_context 'user can move issue' do
......
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