remove PUT fir v3/service_instances|service-bindings from rate limit because PUT in v3 is not present

Author: kathap

middleware/service_broker_rate_limiter.rb

@@ -7,8 +7,7 @@ module Middleware
 
     RATE_LIMITED_ENDPOINTS = [
       RateLimitEndpoint.new(%r{\A/v2/(service_instances|service_credential_bindings|service_route_bindings)}, %w[PUT POST DELETE PATCH]),
-      RateLimitEndpoint.new(%r{\A/v3/(service_instances|service_credential_bindings|service_route_bindings)/.+/parameters\z}, %w[GET]),
-      RateLimitEndpoint.new(%r{\A/v3/(service_instances|service_credential_bindings|service_route_bindings)}, %w[PUT])
+      RateLimitEndpoint.new(%r{\A/v3/(service_instances|service_credential_bindings|service_route_bindings)/.+/parameters\z}, %w[GET])
     ].freeze
 
     class ConcurrentRequestCounter

spec/unit/middleware/service_broker_rate_limiter_spec.rb

@@ -25,7 +25,7 @@ module Middleware
       end
 
       describe 'included requests' do
-        let(:fake_request) { instance_double(ActionDispatch::Request, fullpath: '/v3/service_instances', method: 'PUT') }
+        let(:fake_request) { instance_double(ActionDispatch::Request, fullpath: '/v2/service_instances', method: 'PUT') }
 
         it 'allows a service broker request within the limit' do
           status, = middleware.call(user_env)
@@ -120,7 +120,7 @@ module Middleware
       end
 
       describe 'excluded requests' do
-        let(:request_method) { 'POST' }
+        let(:fake_request) { instance_double(ActionDispatch::Request, fullpath: '/v3/service_instances', method: 'POST') }
 
         it 'allows requests that are no longer rate limited' do
           status, = middleware.call(user_env)