Merge branch 'fix-api-deprecation' into 'master'

Fix a Grape deprecation, use `#request_method` instead of `#route_method`

See merge request !8297
This commit is contained in:
Robert Schilling 2016-12-28 11:36:28 +00:00
commit 465700ddc2
3 changed files with 7 additions and 3 deletions

View File

@ -0,0 +1,4 @@
---
title: Fix a Grape deprecation, use `#request_method` instead of `#route_method`
merge_request:
author:

View File

@ -96,7 +96,7 @@ module API
end
def authenticate_non_get!
authenticate! unless %w[GET HEAD].include?(route.route_method)
authenticate! unless %w[GET HEAD].include?(route.request_method)
end
def authenticate_by_gitlab_shell_token!

View File

@ -396,7 +396,7 @@ describe API::Helpers, api: true do
%w[HEAD GET].each do |method_name|
context "method is #{method_name}" do
before do
expect_any_instance_of(self.class).to receive(:route).and_return(double(route_method: method_name))
expect_any_instance_of(self.class).to receive(:route).and_return(double(request_method: method_name))
end
it 'does not raise an error' do
@ -410,7 +410,7 @@ describe API::Helpers, api: true do
%w[POST PUT PATCH DELETE].each do |method_name|
context "method is #{method_name}" do
before do
expect_any_instance_of(self.class).to receive(:route).and_return(double(route_method: method_name))
expect_any_instance_of(self.class).to receive(:route).and_return(double(request_method: method_name))
end
it 'calls authenticate!' do