Skip to content

Commit 86ce2a0

Browse files
committed
fix tests
1 parent 5feca0e commit 86ce2a0

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

middleware/service_broker_rate_limiter.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ def admin?
123123

124124
def rate_limit_method?(request)
125125
RATE_LIMITED_ENDPOINTS.any? do |endpoint|
126-
endpoint.endpoint_pattern.match?(request.fullpath) && endpoint.methods.include?(request.method)
126+
endpoint.endpoint_pattern.match?(request.fullpath) && endpoint.request_methods.include?(request.method)
127127
end
128128
end
129129

spec/unit/middleware/service_broker_rate_limiter_spec.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ module Middleware
88
let(:path_info) { '/v3/service_instances' }
99
let(:user_guid) { SecureRandom.uuid }
1010
let(:user_env) { { 'cf.user_guid' => user_guid, 'PATH_INFO' => path_info } }
11-
let(:fake_request) { instance_double(ActionDispatch::Request, fullpath: '/v3/service_instances', method: 'PUT') }
11+
let(:fake_request) { instance_double(ActionDispatch::Request, fullpath: '/v3/service_instances', method: 'POST') }
1212
let(:max_concurrent_requests) { 1 }
1313
let(:broker_timeout) { 60 }
1414
let(:middleware) do
@@ -25,7 +25,7 @@ module Middleware
2525
end
2626

2727
describe 'included requests' do
28-
let(:request_method) { 'PUT' }
28+
let(:fake_request) { instance_double(ActionDispatch::Request, fullpath: '/v3/service_instances', method: 'PUT') }
2929

3030
it 'allows a service broker request within the limit' do
3131
status, = middleware.call(user_env)

0 commit comments

Comments
 (0)