From 60c9472b14f24e720f2be3aeb25644bde8df6e55 Mon Sep 17 00:00:00 2001 From: Mike McQuaid Date: Tue, 5 May 2020 15:50:10 +0100 Subject: [PATCH] rubocop_rspec: disable RSpec/FilePath. We keep adding to the list of exceptions and this fails if it isn't run in the right directory. --- Library/.rubocop_rspec.yml | 6 ++-- Library/Homebrew/test/.rubocop_todo.yml | 48 ------------------------- 2 files changed, 4 insertions(+), 50 deletions(-) diff --git a/Library/.rubocop_rspec.yml b/Library/.rubocop_rspec.yml index e855cce1cc..cb607442c2 100644 --- a/Library/.rubocop_rspec.yml +++ b/Library/.rubocop_rspec.yml @@ -15,12 +15,14 @@ Style/DisableCopsWithinSourceCodeDirective: # Intentionally disabled as it doesn't fit with our code style. RSpec/AnyInstance: Enabled: false +RSpec/FilePath: + Enabled: false RSpec/ImplicitBlockExpectation: Enabled: false RSpec/SubjectStub: Enabled: false -# TODO: try to enable these (also requires fixing Homebrew/bundle) +# TODO: try to enable these RSpec/ContextWording: Enabled: false RSpec/DescribeClass: @@ -34,7 +36,7 @@ RSpec/RepeatedDescription: RSpec/RepeatedExampleGroupDescription: Enabled: false -# TODO: try to reduce these (also requires fixing Homebrew/bundle) +# TODO: try to reduce these RSpec/ExampleLength: Max: 75 RSpec/MultipleExpectations: diff --git a/Library/Homebrew/test/.rubocop_todo.yml b/Library/Homebrew/test/.rubocop_todo.yml index a5a6d80cc5..18986235db 100644 --- a/Library/Homebrew/test/.rubocop_todo.yml +++ b/Library/Homebrew/test/.rubocop_todo.yml @@ -12,54 +12,6 @@ RSpec/ExampleLength: Exclude: - 'rubocops/patches_spec.rb' -# Offense count: 41 -# Configuration parameters: CustomTransform, IgnoreMethods. -RSpec/FilePath: - Exclude: - - 'ARGV_spec.rb' - - 'PATH_spec.rb' - - 'bottle_filename_spec.rb' - - 'cache_store_spec.rb' - - 'cask/artifact/alt_target_spec.rb' - - 'cask/artifact/generic_artifact_spec.rb' - - 'cask/artifact/two_apps_correct_spec.rb' - - 'cask/artifact/uninstall_no_zap_spec.rb' - - 'cask/cask_loader/from__path_loader_spec.rb' - - 'cask/macos_spec.rb' - - 'cli/parser_spec.rb' - - 'checksum_verification_spec.rb' - - 'cxxstdlib_spec.rb' - - 'diagnostic_checks_spec.rb' - - 'env_config_spec.rb' - - 'missing_formula_spec.rb' - - 'os/linux/diagnostic_spec.rb' - - 'os/mac/diagnostic_spec.rb' - - 'requirements/macos_requirement_spec.rb' - - 'rubocops/cask/homepage_matches_url_spec.rb' - - 'rubocops/cask/homepage_url_trailing_slash_spec.rb' - - 'rubocops/cask/no_dsl_version_spec.rb' - - 'rubocops/cask/stanza_grouping_spec.rb' - - 'rubocops/cask/stanza_order_spec.rb' - - 'rubocops/caveats_spec.rb' - - 'rubocops/components_order_spec.rb' - - 'rubocops/components_redundancy_spec.rb' - - 'rubocops/conflicts_spec.rb' - - 'rubocops/dependency_order_spec.rb' - - 'rubocops/files_spec.rb' - - 'rubocops/keg_only_spec.rb' - - 'rubocops/homepage_spec.rb' - - 'rubocops/options_spec.rb' - - 'rubocops/patches_spec.rb' - - 'rubocops/text_spec.rb' - - 'rubocops/uses_from_macos_spec.rb' - - 'rubocops/formula_desc_spec.rb' - - 'search_spec.rb' - - 'string_spec.rb' - - 'style_spec.rb' - - 'system_command_result_spec.rb' - - 'unpack_strategy/p7zip_spec.rb' - - 'utils/github_spec.rb' - # Offense count: 6 # Configuration parameters: AssignmentOnly. RSpec/InstanceVariable: