From 8910a5157f863b9737b38cbfd760a06c7967044e Mon Sep 17 00:00:00 2001 From: Justin Coyne Date: Wed, 22 May 2024 10:12:09 -0500 Subject: [PATCH] Remove unecessary rubocop exclusions --- .rubocop.yml | 27 ++++--------------------- spec/requests/iiif_auth_request_spec.rb | 1 - 2 files changed, 4 insertions(+), 24 deletions(-) diff --git a/.rubocop.yml b/.rubocop.yml index 8f225be4..c0b10250 100644 --- a/.rubocop.yml +++ b/.rubocop.yml @@ -16,42 +16,24 @@ AllCops: - "db/**/*" - "config/**/*" - "spec/spec_helper.rb" - - "spec/teaspoon_env.rb" - - "vendor/**/*" - - "app/controllers/concerns/action_controller/**/*" + - vendor/bundle/**/* # Needed on Github Actions for airbrussh, which ships an incompatible .rubocop.yml Rails: Enabled: true Metrics/CyclomaticComplexity: Exclude: - - "app/models/ability.rb" - "app/models/cocina_ability.rb" Metrics/PerceivedComplexity: Exclude: - - "app/models/ability.rb" - "app/models/cocina_ability.rb" Metrics/AbcSize: Exclude: - - "app/models/ability.rb" - "app/models/cocina_ability.rb" - "app/controllers/iiif_controller.rb" -Layout/EmptyLinesAroundBlockBody: - Exclude: - - "spec/requests/file_auth_request_spec.rb" - - "spec/requests/iiif_auth_request_spec.rb" - - "spec/requests/media_auth_request_spec.rb" - -Lint/AssignmentInCondition: - Enabled: false - -Metrics/BlockLength: - Exclude: - - "spec/**/*" - Metrics/ClassLength: Exclude: - "app/controllers/iiif_controller.rb" @@ -62,19 +44,18 @@ Layout/LineLength: Metrics/MethodLength: Exclude: - - "app/models/ability.rb" - "app/models/cocina_ability.rb" - - "app/services/cdl_service.rb" Naming/HeredocDelimiterNaming: - Enabled: false + Exclude: + - spec/requests/**/* Naming/MemoizedInstanceVariableName: Exclude: - "app/controllers/**/*" Naming/PredicateName: - NamePrefix: + ForbiddenPrefixes: - "is_" Style/StringLiterals: diff --git a/spec/requests/iiif_auth_request_spec.rb b/spec/requests/iiif_auth_request_spec.rb index a29dafbe..b0aeb1d0 100644 --- a/spec/requests/iiif_auth_request_spec.rb +++ b/spec/requests/iiif_auth_request_spec.rb @@ -3,7 +3,6 @@ require 'rails_helper' RSpec.describe "Authentication for IIIF requests" do - let(:allowed_loc) { 'ip.address1' } let(:user_no_loc_no_webauth) { User.new } let(:user_loc_no_webauth) { User.new(ip_address: allowed_loc) }