diff --git a/Gemfile b/Gemfile index 5c055a84d74..c06b6e39088 100644 --- a/Gemfile +++ b/Gemfile @@ -79,7 +79,7 @@ group :test do gem 'rspec-its' gem 'rspec-rails', '~> 8.0.2' gem 'rspec-wait' - gem 'rubocop', '~> 1.84.1' + gem 'rubocop', '~> 1.84.2' gem 'rubocop-capybara' gem 'rubocop-factory_bot' gem 'rubocop-rails', '~> 2.34' diff --git a/Gemfile.lock b/Gemfile.lock index 342d6786000..17d56bf3021 100644 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -476,7 +476,7 @@ GEM activesupport (>= 3.0.0) mustache (~> 1.0, >= 0.99.4) rspec (~> 3.0) - rubocop (1.84.1) + rubocop (1.84.2) json (~> 2.3) language_server-protocol (~> 3.17.0.2) lint_roller (~> 1.1.0) @@ -680,7 +680,7 @@ DEPENDENCIES rspec-rails (~> 8.0.2) rspec-wait rspec_api_documentation (>= 6.1.0) - rubocop (~> 1.84.1) + rubocop (~> 1.84.2) rubocop-capybara rubocop-factory_bot rubocop-rails (~> 2.34) diff --git a/spec/request_spec_shared_examples.rb b/spec/request_spec_shared_examples.rb index 95f817491d5..7281736e92e 100644 --- a/spec/request_spec_shared_examples.rb +++ b/spec/request_spec_shared_examples.rb @@ -18,15 +18,15 @@ ALL_PERMISSIONS = (LOCAL_ROLES + GLOBAL_SCOPES).freeze CF_NOT_AUTHORIZED = [ - detail: 'You are not authorized to perform the requested action', - title: 'CF-NotAuthorized', - code: 10_003 + { detail: 'You are not authorized to perform the requested action', + title: 'CF-NotAuthorized', + code: 10_003 } ].freeze CF_ORG_SUSPENDED = [ - detail: 'The organization is suspended', - title: 'CF-OrgSuspended', - code: 10_017 + { detail: 'The organization is suspended', + title: 'CF-OrgSuspended', + code: 10_017 } ].freeze RSpec.shared_examples 'paginated response' do |endpoint| diff --git a/spec/unit/controllers/v3/space_manifests_controller_spec.rb b/spec/unit/controllers/v3/space_manifests_controller_spec.rb index a9740924979..d4964572719 100644 --- a/spec/unit/controllers/v3/space_manifests_controller_spec.rb +++ b/spec/unit/controllers/v3/space_manifests_controller_spec.rb @@ -261,9 +261,9 @@ def error_message(detail) errors = parsed_body['errors'] expect(errors.size).to eq(1) expected_error = [ - 'detail' => "For application 'blah': Buildpack cannot be configured for a docker lifecycle app.", - 'title' => 'CF-UnprocessableEntity', - 'code' => 10_008 + { 'detail' => "For application 'blah': Buildpack cannot be configured for a docker lifecycle app.", + 'title' => 'CF-UnprocessableEntity', + 'code' => 10_008 } ] expect(errors.map { |h| h.except('test_mode_info') }).to match_array(expected_error) end @@ -305,9 +305,9 @@ def error_message(detail) errors = parsed_body['errors'] expect(errors.size).to eq(1) expected_error = [ - 'detail' => "For application 'blah': Buildpacks cannot be configured for a docker lifecycle app.", - 'title' => 'CF-UnprocessableEntity', - 'code' => 10_008 + { 'detail' => "For application 'blah': Buildpacks cannot be configured for a docker lifecycle app.", + 'title' => 'CF-UnprocessableEntity', + 'code' => 10_008 } ] expect(errors.map { |h| h.except('test_mode_info') }).to match_array(expected_error) end diff --git a/spec/unit/lib/cloud_controller/diego/cnb/staging_completion_handler_spec.rb b/spec/unit/lib/cloud_controller/diego/cnb/staging_completion_handler_spec.rb index f0196e314d4..68c86f3b477 100644 --- a/spec/unit/lib/cloud_controller/diego/cnb/staging_completion_handler_spec.rb +++ b/spec/unit/lib/cloud_controller/diego/cnb/staging_completion_handler_spec.rb @@ -13,9 +13,9 @@ module CNB lifecycle_type: 'cnb', lifecycle_metadata: { buildpacks: [ - key: 'foo', - name: 'nodejs', - version: '1.0.0' + { key: 'foo', + name: 'nodejs', + version: '1.0.0' } ] }, process_types: {