From d53755da68c660b2393a931e063e46e455adcec9 Mon Sep 17 00:00:00 2001 From: Jasper Maes Date: Wed, 2 Jan 2019 21:47:45 +0100 Subject: Fix deprecation: Using positional arguments in integration tests --- changelogs/unreleased/deprecated-positional-spec-arguments.yml | 5 +++++ spec/controllers/projects/merge_requests_controller_spec.rb | 8 ++++---- spec/controllers/projects/snippets_controller_spec.rb | 2 +- spec/controllers/projects_controller_spec.rb | 2 +- spec/controllers/snippets_controller_spec.rb | 6 +++--- 5 files changed, 14 insertions(+), 9 deletions(-) create mode 100644 changelogs/unreleased/deprecated-positional-spec-arguments.yml diff --git a/changelogs/unreleased/deprecated-positional-spec-arguments.yml b/changelogs/unreleased/deprecated-positional-spec-arguments.yml new file mode 100644 index 00000000000..8e541df1ad4 --- /dev/null +++ b/changelogs/unreleased/deprecated-positional-spec-arguments.yml @@ -0,0 +1,5 @@ +--- +title: 'Fix deprecation: Using positional arguments in integration tests' +merge_request: 24110 +author: Jasper Maes +type: other diff --git a/spec/controllers/projects/merge_requests_controller_spec.rb b/spec/controllers/projects/merge_requests_controller_spec.rb index d46b9ffb3ce..4f4d3ca226f 100644 --- a/spec/controllers/projects/merge_requests_controller_spec.rb +++ b/spec/controllers/projects/merge_requests_controller_spec.rb @@ -950,7 +950,7 @@ describe Projects::MergeRequestsController do end it 'returns 200' do - get :discussions, namespace_id: project.namespace, project_id: project, id: merge_request.iid + get :discussions, params: { namespace_id: project.namespace, project_id: project, id: merge_request.iid } expect(response.status).to eq(200) end @@ -969,7 +969,7 @@ describe Projects::MergeRequestsController do expect(collection).to receive(:find_by_id).with(note_diff_file.id).and_call_original end - get :discussions, namespace_id: project.namespace, project_id: project, id: merge_request.iid + get :discussions, params: { namespace_id: project.namespace, project_id: project, id: merge_request.iid } end end @@ -986,7 +986,7 @@ describe Projects::MergeRequestsController do expect(collection).to receive(:find_by_id).with(note_diff_file.id).and_call_original end - get :discussions, namespace_id: project.namespace, project_id: project, id: merge_request.iid + get :discussions, params: { namespace_id: project.namespace, project_id: project, id: merge_request.iid } end it 'does not preload highlights when diff note is resolved' do @@ -999,7 +999,7 @@ describe Projects::MergeRequestsController do expect(collection).to receive(:find_by_id).with(note_diff_file.id).and_call_original end - get :discussions, namespace_id: project.namespace, project_id: project, id: merge_request.iid + get :discussions, params: { namespace_id: project.namespace, project_id: project, id: merge_request.iid } end end end diff --git a/spec/controllers/projects/snippets_controller_spec.rb b/spec/controllers/projects/snippets_controller_spec.rb index e4b78aff25d..75c9839dd9b 100644 --- a/spec/controllers/projects/snippets_controller_spec.rb +++ b/spec/controllers/projects/snippets_controller_spec.rb @@ -385,7 +385,7 @@ describe Projects::SnippetsController do before do sign_in(user) - get :show, namespace_id: project.namespace, project_id: project, id: project_snippet.to_param, format: :js + get :show, params: { namespace_id: project.namespace, project_id: project, id: project_snippet.to_param }, format: :js end context 'when snippet is private' do diff --git a/spec/controllers/projects_controller_spec.rb b/spec/controllers/projects_controller_spec.rb index 4747d837273..f84f069f4db 100644 --- a/spec/controllers/projects_controller_spec.rb +++ b/spec/controllers/projects_controller_spec.rb @@ -667,7 +667,7 @@ describe ProjectsController do project.add_guest(user) sign_in(user) - get :refs, namespace_id: project.namespace, id: project + get :refs, params: { namespace_id: project.namespace, id: project } expect(response).to have_gitlab_http_status(404) end diff --git a/spec/controllers/snippets_controller_spec.rb b/spec/controllers/snippets_controller_spec.rb index d762531da7e..5c6858dc7b2 100644 --- a/spec/controllers/snippets_controller_spec.rb +++ b/spec/controllers/snippets_controller_spec.rb @@ -82,7 +82,7 @@ describe SnippetsController do end it 'responds with status 404 when embeddable content is requested' do - get :show, id: personal_snippet.to_param, format: :js + get :show, params: { id: personal_snippet.to_param }, format: :js expect(response).to have_gitlab_http_status(404) end @@ -114,7 +114,7 @@ describe SnippetsController do end it 'responds with status 404 when embeddable content is requested' do - get :show, id: personal_snippet.to_param, format: :js + get :show, params: { id: personal_snippet.to_param }, format: :js expect(response).to have_gitlab_http_status(404) end @@ -145,7 +145,7 @@ describe SnippetsController do end it 'responds with status 200 when embeddable content is requested' do - get :show, id: personal_snippet.to_param, format: :js + get :show, params: { id: personal_snippet.to_param }, format: :js expect(assigns(:snippet)).to eq(personal_snippet) expect(response).to have_gitlab_http_status(200) -- cgit v1.2.3