rubocop: move rspec include to root config file

To avoid confusion and bugs in rubocop robucop-spec should be required
from the root config.
This commit is contained in:
Jan Viljanen 2018-10-04 18:22:23 +02:00
parent bca922fd5b
commit aa6912425e
3 changed files with 41 additions and 10 deletions

View File

@ -4,8 +4,13 @@ AllCops:
- '**/Casks/**/*' - '**/Casks/**/*'
- '**/vendor/**/*' - '**/vendor/**/*'
DisplayCopNames: false DisplayCopNames: false
RSpec:
Patterns:
- Homebrew/test/*
require: ./Homebrew/rubocops.rb require:
- ./Homebrew/rubocops.rb
- rubocop-rspec
# enable all formulae audits # enable all formulae audits
FormulaAudit: FormulaAudit:

View File

@ -2,4 +2,3 @@ inherit_from:
- ../.rubocop.yml - ../.rubocop.yml
- .rubocop_todo.yml - .rubocop_todo.yml
require: rubocop-rspec

View File

@ -1,14 +1,15 @@
# This configuration was generated by # This configuration was generated by
# `rubocop --auto-gen-config` # `rubocop --auto-gen-config`
# on 2018-09-30 14:57:59 +0200 using RuboCop version 0.59.2. # on 2018-10-08 06:47:14 +0200 using RuboCop version 0.59.2.
# The point is for the user to remove these configuration records # The point is for the user to remove these configuration records
# one by one as the offenses are removed from the code base. # one by one as the offenses are removed from the code base.
# Note that changes in the inspected code, or installation of new # Note that changes in the inspected code, or installation of new
# versions of RuboCop, may require this file to be generated again. # versions of RuboCop, may require this file to be generated again.
# Offense count: 27 # Offense count: 28
RSpec/AnyInstance: RSpec/AnyInstance:
Exclude: Exclude:
- 'cask/artifact/uninstall_zap_shared_examples.rb'
- 'cask/cmd/create_spec.rb' - 'cask/cmd/create_spec.rb'
- 'cask/cmd/edit_spec.rb' - 'cask/cmd/edit_spec.rb'
- 'cask/cmd/outdated_spec.rb' - 'cask/cmd/outdated_spec.rb'
@ -21,7 +22,7 @@ RSpec/AnyInstance:
- 'utils/analytics_spec.rb' - 'utils/analytics_spec.rb'
- 'utils/git_spec.rb' - 'utils/git_spec.rb'
# Offense count: 139 # Offense count: 146
# Configuration parameters: Prefixes. # Configuration parameters: Prefixes.
# Prefixes: when, with, without # Prefixes: when, with, without
RSpec/ContextWording: RSpec/ContextWording:
@ -31,7 +32,7 @@ RSpec/ContextWording:
RSpec/DescribeClass: RSpec/DescribeClass:
Enabled: false Enabled: false
# Offense count: 573 # Offense count: 581
# Configuration parameters: Max. # Configuration parameters: Max.
RSpec/ExampleLength: RSpec/ExampleLength:
Enabled: false Enabled: false
@ -58,6 +59,16 @@ RSpec/ExpectInHook:
RSpec/FilePath: RSpec/FilePath:
Enabled: false Enabled: false
# Offense count: 6
# Cop supports --auto-correct.
# Configuration parameters: EnforcedStyle.
# SupportedStyles: implicit, each, example
RSpec/HookArgument:
Exclude:
- 'cask/artifact/uninstall_zap_shared_examples.rb'
- 'spec_helper.rb'
- 'support/helper/cask/fake_system_command.rb'
# Offense count: 6 # Offense count: 6
# Configuration parameters: AssignmentOnly. # Configuration parameters: AssignmentOnly.
RSpec/InstanceVariable: RSpec/InstanceVariable:
@ -67,7 +78,18 @@ RSpec/InstanceVariable:
- 'utils/git_spec.rb' - 'utils/git_spec.rb'
- 'version_spec.rb' - 'version_spec.rb'
# Offense count: 116 # Offense count: 1
RSpec/IteratedExpectation:
Exclude:
- 'cask/artifact/uninstall_zap_shared_examples.rb'
# Offense count: 1
# Cop supports --auto-correct.
RSpec/LeadingSubject:
Exclude:
- 'cask/artifact/uninstall_zap_shared_examples.rb'
# Offense count: 122
# Configuration parameters: EnforcedStyle. # Configuration parameters: EnforcedStyle.
# SupportedStyles: have_received, receive # SupportedStyles: have_received, receive
RSpec/MessageSpies: RSpec/MessageSpies:
@ -82,12 +104,12 @@ RSpec/MissingExampleGroupArgument:
RSpec/MultipleDescribes: RSpec/MultipleDescribes:
Enabled: false Enabled: false
# Offense count: 544 # Offense count: 548
# Configuration parameters: AggregateFailuresByDefault. # Configuration parameters: AggregateFailuresByDefault.
RSpec/MultipleExpectations: RSpec/MultipleExpectations:
Max: 26 Max: 26
# Offense count: 936 # Offense count: 946
RSpec/NamedSubject: RSpec/NamedSubject:
Enabled: false Enabled: false
@ -110,6 +132,11 @@ RSpec/RepeatedExample:
- 'compiler_selector_spec.rb' - 'compiler_selector_spec.rb'
- 'utils/shell_spec.rb' - 'utils/shell_spec.rb'
# Offense count: 2
RSpec/ScatteredLet:
Exclude:
- 'cask/artifact/uninstall_zap_shared_examples.rb'
# Offense count: 31 # Offense count: 31
RSpec/SubjectStub: RSpec/SubjectStub:
Exclude: Exclude:
@ -121,7 +148,7 @@ RSpec/SubjectStub:
- 'language/python_spec.rb' - 'language/python_spec.rb'
- 'os/mac/java_requirement_spec.rb' - 'os/mac/java_requirement_spec.rb'
# Offense count: 64 # Offense count: 65
# Configuration parameters: IgnoreSymbolicNames. # Configuration parameters: IgnoreSymbolicNames.
RSpec/VerifiedDoubles: RSpec/VerifiedDoubles:
Enabled: false Enabled: false