simplycop 1.2.0 → 1.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cb033be1d0c83f6ddf82d2d096aa89992caeb1e51f55b6966ecbc69b77f9462e
-  data.tar.gz: 8e04105bcda67012c7c1e9ef83cd91e34ae7318a8e06e9ad24f0deb96f27831f
+  metadata.gz: 076036bcf8a50fab9721ef1a9bc81c58c1b37be20b72b6882fbbcd55505f7071
+  data.tar.gz: f246b21e0afd394b49dbafd4445c90734f6232488d7d1578a4e3b118dff9ef53
 SHA512:
-  metadata.gz: e9f93c07d4fd1a18109ae5561aa9246578f16656c1fb95aeb188cf0b3a6d3702a7b5cc8e1680dac2ae1bc2474569dcd5eb84f33e8563dbad0fc7a574cc71e69a
-  data.tar.gz: bc2f67423f8ff100f0f883b5dd4047ff1728f11e4ae709c0d1fa2d990ae0f75a8e380e47d7640a9a9353d9865e5c9efa78600e9e7e9d34cf04094557586cb666
+  metadata.gz: a19db9f0d33d7b4a376c3e54d3d08f58dad4c891f1349f5c4edb26e04e16b8b21aadf766fcaaef28d16fb113959d755676fa4e5b3fa7d04dc631b99cfffcb02c
+  data.tar.gz: 723b0bb560e8fb0a229b131abba0ee984bb84f6aeb21500fa3dc9d7d25157a0d457f430ada57a8c3737d24218e95e3873efc317a0140da62624aac2cc295fa6f

data/.custom_simplycop.yml

@@ -1,5 +1,6 @@
 require:
   - './lib/simplycop/custom_cops/timecop_without_block.rb'
+  - './lib/simplycop/custom_cops/dont_print_all_env.rb'
 
 AllCops:
   ExtraDetails: true
@@ -9,3 +10,8 @@ CustomCops/TimecopWithoutBlock:
   Details: >-
     Time in all tests is faked to be midday. Using `Timecop.return` rather than the block format will spoil that for all subsequent tests.
     `https://github.com/simplybusiness/chopin/pull/10607`
+
+CustomCops/DontPrintAllEnv:
+  Enabled: true
+  Details: >-
+    This cop checks if someone accidentally print all environment variables as they may contain secrets.

data/.github/dependabot.yml

@@ -0,0 +1,18 @@
+version: 2
+updates:
+  - package-ecosystem: bundler
+    directory: "/"
+    schedule:
+      interval: daily
+      time: "05:00"
+    open-pull-requests-limit: 99
+    labels:
+      - dependencies
+  - package-ecosystem: github-actions
+    directory: "/"
+    schedule:
+      interval: daily
+      time: "05:00"
+    open-pull-requests-limit: 99
+    labels:
+      - dependencies

data/.github/workflows/version_forget_me_not.yml

@@ -11,7 +11,7 @@ jobs:
     runs-on: ubuntu-18.04
 
     steps:
-      - uses: simplybusiness/version-forget-me-not@v1
+      - uses: simplybusiness/version-forget-me-not@v2
         env:
           ACCESS_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           VERSION_FILE_PATH: "lib/simplycop/version.rb"

data/.simplycop.yml

@@ -18,436 +18,395 @@ AllCops:
   DisplayStyleGuide: true
   UseCache: true
   MaxFilesInCache: 5000
+  ExtraDetails: true
 # Adapted from: https://github.com/simplybusiness/how-we-roll/blob/master/development/RUBYSTYLEGUIDE.markdown
 
-Naming/VariableName:
+Layout/EmptyLineAfterMultilineCondition:
   Enabled: true
-  EnforcedStyle: snake_case
-  SupportedStyles:
-    - snake_case
-    - camelCase
 
-Naming/FileName:
+Layout/FirstArrayElementLineBreak:
   Enabled: true
 
-Layout/TrailingEmptyLines:
+Layout/FirstMethodArgumentLineBreak:
   Enabled: true
-  Severity: warning
 
-Layout/TrailingWhitespace:
+Layout/FirstHashElementLineBreak:
   Enabled: true
-  Severity: warning
 
-Lint/DuplicateMethods:
+Layout/HeredocArgumentClosingParenthesis:
   Enabled: true
-  Severity: warning
 
-Style/SpecialGlobalVars:
+Layout/SpaceBeforeBrackets:
   Enabled: true
-  Severity: warning
-
-Style/ColonMethodCall:
-  Enabled: true
-  Severity: warning
 
-Style/RedundantReturn:
-  Enabled: true
-  Severity: warning
+###### COPS REVIEWED ABOVE THIS LINE ######
 
-Style/NestedParenthesizedCalls:
+Lint/AmbiguousAssignment:
   Enabled: true
 
-Layout/MultilineMethodCallIndentation:
-  Enabled: true
+Lint/AmbiguousBlockAssociation:
+  Exclude:
+    - spec/**/*
 
-Style/RedundantParentheses:
+Lint/AmbiguousOperator:
   Enabled: true
 
-Style/RedundantInterpolation:
-  Enabled: false
-
-Layout/AssignmentIndentation:
+Lint/AmbiguousRegexpLiteral:
   Enabled: true
 
-Style/MutableConstant:
-  Enabled: false
-
-Style/ZeroLengthPredicate:
+Lint/BinaryOperatorWithIdenticalOperands:
   Enabled: true
 
-Style/ConditionalAssignment:
+Lint/CircularArgumentReference:
   Enabled: true
 
-Style/NestedModifier:
+Lint/ConstantDefinitionInBlock:
   Enabled: true
+  Exclude:
+    - spec/**/*.rb
 
-Style/IfInsideElse:
+Lint/DeprecatedClassMethods:
   Enabled: true
 
-Lint/IneffectiveAccessModifier:
-  Enabled: false
-
-Lint/ImplicitStringConcatenation:
+Lint/DeprecatedConstants:
   Enabled: true
 
-Lint/UselessAssignment:
+Lint/DeprecatedOpenSSLConstant:
   Enabled: true
-  Severity: warning
 
-Lint/UnusedBlockArgument:
+Lint/DuplicateBranch:
   Enabled: true
-  Severity: warning
 
-Lint/UnderscorePrefixedVariableName:
+Lint/DuplicateElsifCondition:
   Enabled: true
-  Severity: warning
 
-Style/Documentation:
-  Enabled: false
-
-Layout/SpaceInsideBlockBraces:
+Lint/DuplicateHashKey:
   Enabled: false
 
-Layout/SpaceInsideArrayLiteralBrackets:
-  Enabled: true
-
-Layout/SpaceInsideReferenceBrackets:
+Lint/DuplicateMethods:
   Enabled: true
+  Severity: warning
 
-Layout/SpaceInsideHashLiteralBraces:
-  Enabled: false
-
-Layout/SpaceInsideParens:
+Lint/DuplicateRegexpCharacterClassElement:
   Enabled: true
 
-Layout/SpaceBeforeComma:
+Lint/DuplicateRequire:
   Enabled: true
 
-Layout/IndentationConsistency:
+Lint/DuplicateRescueException:
   Enabled: true
 
-Layout/CaseIndentation:
+Lint/EmptyBlock:
   Enabled: true
 
-Style/MethodDefParentheses:
+Lint/EmptyClass:
   Enabled: true
 
-Style/MultilineTernaryOperator:
+Lint/EmptyConditionalBody:
   Enabled: true
 
-Style/NestedTernaryOperator:
+Lint/EmptyFile:
   Enabled: true
 
-Style/UnlessElse:
+Lint/FloatComparison:
   Enabled: true
 
-Style/SymbolArray:
-  EnforcedStyle: brackets
-
-Style/WordArray:
-  EnforcedStyle: brackets
-
-Style/ParenthesesAroundCondition:
+Lint/IdentityComparison:
   Enabled: true
 
-Layout/MultilineBlockLayout:
+Lint/ImplicitStringConcatenation:
   Enabled: true
 
-Layout/InitialIndentation:
-  Enabled: true
+Lint/IneffectiveAccessModifier:
+  Enabled: false
 
-Style/OptionalArguments:
+Lint/LambdaWithoutLiteralBlock:
   Enabled: true
 
-Lint/DuplicateHashKey:
+Lint/MissingSuper:
   Enabled: false
 
-Lint/CircularArgumentReference:
+Lint/MixedRegexpCaptureTypes:
   Enabled: true
 
-Lint/AmbiguousOperator:
+Lint/NoReturnInBeginEndBlocks:
   Enabled: true
 
-Lint/AmbiguousRegexpLiteral:
+Lint/OutOfRangeRegexpRef:
   Enabled: true
 
-Layout/BlockAlignment:
+Lint/RaiseException:
   Enabled: true
 
-Layout/DefEndAlignment:
+Lint/RedundantDirGlobSort:
   Enabled: true
 
-Lint/DeprecatedClassMethods:
+Lint/SelfAssignment:
   Enabled: true
 
-Layout/EndAlignment:
+Lint/StructNewOverride:
   Enabled: true
 
-Lint/UselessAccessModifier:
-  Enabled: false
-
-Lint/Void:
+Lint/ToEnumArguments:
   Enabled: true
 
-Style/Alias:
-  Enabled: false
-
-Layout/SpaceAroundBlockParameters:
+Lint/TopLevelReturnWithArgument:
   Enabled: true
 
-Layout/SpaceAroundEqualsInParameterDefault:
+Lint/TrailingCommaInAttributeDeclaration:
   Enabled: true
 
-Style/RedundantBegin:
+Lint/UnderscorePrefixedVariableName:
   Enabled: true
+  Severity: warning
 
-Style/RedundantException:
+Lint/UnexpectedBlockArity:
   Enabled: true
 
-Layout/EmptyLines:
+Lint/UnmodifiedReduceAccumulator:
   Enabled: true
 
-Style/SelfAssignment:
+Lint/UnreachableLoop:
   Enabled: true
 
-Style/LambdaCall:
+Lint/UnusedBlockArgument:
   Enabled: true
+  Severity: warning
 
-Metrics/ModuleLength:
-  Enabled: true
+Lint/UselessAccessModifier:
+  Enabled: false
 
-Style/For:
+Lint/UselessAssignment:
   Enabled: true
+  Severity: warning
 
-Layout/ExtraSpacing:
-  Enabled: false
-
-Style/GuardClause:
-  Enabled: false
+Lint/UselessMethodDefinition:
+  Enabled: true
 
-Style/FrozenStringLiteralComment:
-  Enabled: false
+Lint/UselessTimes:
+  Enabled: true
 
-Style/StringLiterals:
-  Enabled: false
+Lint/Void:
+  Enabled: true
 
+Metrics/BlockLength:
+  Exclude:
+    - spec/**/*.rb
+    -
 Metrics/ClassLength:
   Max: 512
   Enabled: true
 
-Layout/LineLength:
+Metrics/ModuleLength:
   Enabled: true
-  Max: 120
 
-Layout/ArgumentAlignment:
+Naming/FileName:
   Enabled: true
 
-Style/HashEachMethods:
-  Enabled: false
-
-Style/HashTransformKeys:
+Naming/VariableName:
   Enabled: true
+  EnforcedStyle: snake_case
+  SupportedStyles:
+    - snake_case
+    - camelCase
 
-Style/HashTransformValues:
+Style/AccessorGrouping:
   Enabled: true
 
-Layout/EmptyLinesAroundAttributeAccessor:
-  Enabled: true
+Style/Alias:
+  Enabled: false
 
-Layout/SpaceAroundMethodCallOperator:
+Style/ArgumentsForwarding:
   Enabled: true
 
-Lint/DeprecatedOpenSSLConstant:
+Style/ArrayCoercion:
   Enabled: true
 
-Lint/DuplicateElsifCondition:
+Style/BisectedAttrAccessor:
   Enabled: true
 
-Lint/MixedRegexpCaptureTypes:
+Style/CaseLikeIf:
   Enabled: true
 
-Lint/RaiseException:
+Style/CollectionCompact:
   Enabled: true
 
-Lint/StructNewOverride:
+Style/ColonMethodCall:
   Enabled: true
+  Severity: warning
 
-Style/AccessorGrouping:
+Style/CombinableLoops:
   Enabled: true
 
-Style/ArrayCoercion:
+Style/ConditionalAssignment:
   Enabled: true
 
-Style/BisectedAttrAccessor:
+Style/DocumentDynamicEvalDefinition:
+  Enabled: false
+
+Style/Documentation:
+  Enabled: false
+
+Style/EndlessMethod:
   Enabled: true
 
-Style/CaseLikeIf:
+Style/ExplicitBlockArgument:
   Enabled: true
 
 Style/ExponentialNotation:
   Enabled: false
 
-Style/HashAsLastArrayItem:
+Style/For:
   Enabled: true
 
-Style/HashLikeCase:
+Style/FormatString:
   Enabled: false
 
-Style/RedundantAssignment:
-  Enabled: true
+Style/FormatStringToken:
+  Enabled: false
 
-Style/RedundantFetchBlock:
-  Enabled: true
+Style/FrozenStringLiteralComment:
+  Enabled: false
 
-Style/RedundantFileExtensionInRequire:
+Style/GlobalStdStream:
   Enabled: true
 
-Style/RedundantRegexpCharacterClass:
-  Enabled: true
+Style/GuardClause:
+  Enabled: false
 
-Style/RedundantRegexpEscape:
+Style/HashAsLastArrayItem:
   Enabled: true
 
-Style/SlicingWithRange:
+Style/HashEachMethods:
   Enabled: false
 
-Layout/BeginEndAlignment:
+Style/HashExcept:
   Enabled: true
 
-Lint/BinaryOperatorWithIdenticalOperands:
-  Enabled: true
+Style/HashLikeCase:
+  Enabled: false
 
-Lint/ConstantDefinitionInBlock:
+Style/HashTransformKeys:
   Enabled: true
-  Exclude:
-    - spec/**/*.rb
 
-Lint/DuplicateRequire:
+Style/HashTransformValues:
   Enabled: true
 
-Lint/DuplicateRescueException:
+Style/IfInsideElse:
   Enabled: true
 
-Lint/EmptyConditionalBody:
+Style/KeywordParametersOrder:
   Enabled: true
 
-Lint/EmptyFile:
+Style/LambdaCall:
   Enabled: true
 
-Lint/FloatComparison:
+Style/MethodDefParentheses:
   Enabled: true
 
-Lint/IdentityComparison:
+Style/MultilineTernaryOperator:
   Enabled: true
 
-Lint/MissingSuper:
+Style/MutableConstant:
   Enabled: false
 
-Lint/OutOfRangeRegexpRef:
+Style/NegatedIfElseCondition:
   Enabled: true
 
-Lint/SelfAssignment:
+Style/NestedModifier:
   Enabled: true
 
-Lint/TopLevelReturnWithArgument:
+Style/NestedParenthesizedCalls:
   Enabled: true
 
-Lint/TrailingCommaInAttributeDeclaration:
+Style/NestedTernaryOperator:
   Enabled: true
 
-Lint/UnreachableLoop:
+Style/NilLambda:
   Enabled: true
 
-Lint/UselessMethodDefinition:
-  Enabled: true
+Style/NumericPredicate:
+  Enabled: false
 
-Lint/UselessTimes:
+Style/OptionalArguments:
   Enabled: true
 
-Style/CombinableLoops:
+Style/OptionalBooleanParameter:
   Enabled: true
 
-Style/ExplicitBlockArgument:
+Style/ParenthesesAroundCondition:
   Enabled: true
 
-Style/GlobalStdStream:
+Style/RedundantArgument:
   Enabled: true
 
-Style/KeywordParametersOrder:
+Style/RedundantAssignment:
   Enabled: true
 
-Style/OptionalBooleanParameter:
+Style/RedundantBegin:
   Enabled: true
 
-Style/RedundantSelfAssignment:
+Style/RedundantException:
   Enabled: true
 
-Style/SingleArgumentDig:
+Style/RedundantFetchBlock:
   Enabled: true
 
-Style/SoleNestedConditional:
+Style/RedundantFileExtensionInRequire:
   Enabled: true
 
-Style/StringConcatenation:
-  Enabled: true
+Style/RedundantInterpolation:
+  Enabled: false
 
-Lint/DuplicateRegexpCharacterClassElement:
+Style/RedundantParentheses:
   Enabled: true
 
-Lint/EmptyBlock:
+Style/RedundantRegexpCharacterClass:
   Enabled: true
 
-Lint/ToEnumArguments:
+Style/RedundantRegexpEscape:
   Enabled: true
 
-Lint/UnmodifiedReduceAccumulator:
+Style/RedundantReturn:
   Enabled: true
+  Severity: warning
 
-Style/ArgumentsForwarding:
+Style/RedundantSelfAssignment:
   Enabled: true
 
-Style/DocumentDynamicEvalDefinition:
-  Enabled: false
-
-Style/SwapValues:
+Style/SelfAssignment:
   Enabled: true
 
-Lint/AmbiguousBlockAssociation:
-  Exclude:
-    - spec/**/*
-
-Layout/FirstHashElementIndentation:
-  EnforcedStyle: consistent
-
-Style/FormatString:
-  Enabled: false
-
-Style/FormatStringToken:
-  Enabled: false
+Style/SingleArgumentDig:
+  Enabled: true
 
-Style/NumericPredicate:
+Style/SlicingWithRange:
   Enabled: false
 
-Lint/NoReturnInBeginEndBlocks:
+Style/SoleNestedConditional:
   Enabled: true
 
-Style/CollectionCompact:
+Style/SpecialGlobalVars:
   Enabled: true
+  Severity: warning
 
-Style/NegatedIfElseCondition:
+Style/StringConcatenation:
   Enabled: true
 
-Lint/DuplicateBranch:
-  Enabled: true
+Style/StringLiterals:
+  Enabled: false
 
-Lint/EmptyClass:
-  Enabled: true
-  
-Style/NilLambda:
+Style/SwapValues:
   Enabled: true
 
-Lint/UnexpectedBlockArity:
+Style/SymbolArray:
+  EnforcedStyle: brackets
+
+Style/UnlessElse:
   Enabled: true
 
-Style/RedundantArgument:
+Style/WordArray:
+  EnforcedStyle: brackets
+
+Style/ZeroLengthPredicate:
   Enabled: true

data/.simplycop_metaprogramming.yml

@@ -1,6 +1,3 @@
-AllCops:
-  ExtraDetails: true
-
 require:
   - './lib/simplycop/custom_cops/constantize.rb'
   - './lib/simplycop/custom_cops/define_method.rb'

data/.simplycop_rails.yml

@@ -1,64 +1,20 @@
 require:
   - rubocop-rails
 
-AllCops:
-  Exclude:
-    - 'db/schema.rb'
-    - 'vendor/**/*'
-
-  # Cop names are not displayed in offense messages by default. Change behavior
-  # by overriding DisplayCopNames, or by giving the -D/--display-cop-names
-  # option.
-  DisplayCopNames: true
-  # Style guide URLs are not displayed in offense messages by default. Change
-  # behavior by overriding DisplayStyleGuide, or by giving the
-  # -S/--display-style-guide option.
-  DisplayStyleGuide: true
-  UseCache: true
-  MaxFilesInCache: 5000
-# Adapted from: https://github.com/simplybusiness/how-we-roll/blob/master/development/RUBYSTYLEGUIDE.markdown
-
-Rails:
-  Enabled: true
-
-## Rails Cops
-Rails/TimeZone:
-  Enabled: true
-  Severity: warning
-
-Rails/Date:
-  Enabled: true
-  Severity: warning
-
-Rails/Delegate:
-  Enabled: false
-
-Rails/Output:
-  Enabled: true
-
-Rails/Validation:
-  Enabled: true
-
-Rails/FindBy:
-  Enabled: true
-
-Rails/ActionFilter:
-  Enabled: true
-
-Rails/ReadWriteAttribute:
+Rails/ActiveRecordCallbacksOrder:
   Enabled: true
 
-Rails/PluralizationGrammar:
+Rails/AfterCommitOverride:
   Enabled: true
 
-Rails/SkipsModelValidations:
+Rails/AttributeDefaultBlockValue:
   Enabled: true
 
-Rails/ActiveRecordCallbacksOrder:
+Rails/DefaultScope:
   Enabled: true
 
-Rails/AfterCommitOverride:
-  Enabled: true
+Rails/Delegate:
+  Enabled: false
 
 Rails/FindById:
   Enabled: true
@@ -75,9 +31,11 @@ Rails/MatchRoute:
 Rails/NegateInclude:
   Enabled: true
 
+# Conflicts with Mongoid
 Rails/Pluck:
   Enabled: false
 
+# Conflicts with Mongoid
 Rails/PluckInWhere:
   Enabled: false
 
@@ -93,17 +51,13 @@ Rails/ShortI18n:
 Rails/SquishedSQLHeredocs:
   Enabled: true
 
+Rails/WhereEquals:
+  Enabled: true
+
+# Conflicts with Mongoid
 Rails/WhereExists:
   Enabled: false
 
+# Conflicts with Mongoid
 Rails/WhereNot:
   Enabled: false
-
-Rails/WhereEquals:
-  Enabled: true
-
-Rails/AttributeDefaultBlockValue:
-  Enabled: true
-
-Rails/ArelStar:
-  Enabled: true

data/.simplycop_rspec.yml

@@ -1,234 +1,48 @@
 require:
   - rubocop-rspec
 
-RSpec/ContextMethod:
-  Enabled: false
-
-RSpec/LeakyConstantDeclaration:
-  Enabled: false
-
-RSpec/EmptyLineAfterExample:
-  Enabled: false
-
-RSpec/AnyInstance:
-  Description: Check that instances are not being stubbed globally.
-  Enabled: true
-
-RSpec/AroundBlock:
-  Description: Checks that around blocks actually run the test.
-  Enabled: true
-
-RSpec/BeEql:
-  Description: Check for expectations where `be(...)` can replace `eql(...)`.
-  Enabled: true
-
 RSpec/BeforeAfterAll:
-  Description: Check that before/after(:all) isn't being used.
   Enabled: false
 
 RSpec/DescribeClass:
-  Description: Check that the first argument to the top level describe is a constant.
-  Enabled: false
-
-RSpec/DescribedClass:
-  Description: Checks that tests use `described_class`.
-  SkipBlocks: false
   Enabled: false
-  EnforcedStyle: described_class
-  SupportedStyles:
-    - described_class
-    - explicit
 
 RSpec/DescribeMethod:
-  Description: Checks that the second argument to `describe` specifies a method.
-  Enabled: false
-
-RSpec/DescribeSymbol:
-  Description: Avoid describing symbols.
   Enabled: false
 
-RSpec/IteratedExpectation:
-  Description: Check that `all` matcher is used instead of iterating over an array.
-  Enabled: true
-
-RSpec/EmptyExampleGroup:
-  Description: Checks if an example group does not include any tests.
-  Enabled: true
-
-RSpec/EmptyLineAfterFinalLet:
-  Description: Checks if there is an empty line after the last let block.
-  Enabled: false
-
-RSpec/EmptyLineAfterSubject:
-  Description: Checks if there is an empty line after subject block.
+RSpec/DescribedClass:
   Enabled: false
 
 RSpec/ExampleLength:
-  Description: Checks for long examples.
   Enabled: false
-  Max: 5
-
-RSpec/ExampleWording:
-  Description: Checks for common mistakes in example descriptions.
-  Enabled: true
-  CustomTransform:
-    be: is
-    BE: IS
-    have: has
-    HAVE: HAS
-  IgnoredWords: []
-
-RSpec/ExpectActual:
-  Description: Checks for `expect(...)` calls containing literal values.
-  Enabled: true
 
 RSpec/ExpectOutput:
-  Description: Checks for opportunities to use `expect { ... }.to output`.
   Enabled: false
 
 RSpec/FilePath:
-  Description: Checks that spec file paths are consistent with the test subject.
-  Enabled: false
-  CustomTransform:
-    RuboCop: rubocop
-    RSpec: rspec
-  IgnoreMethods: false
-
-RSpec/Focus:
-  Description: Checks if examples are focused.
-  Enabled: false
-
-RSpec/HookArgument:
-  Description: Checks the arguments passed to `before`, `around`, and `after`.
-  Enabled: true
-  EnforcedStyle: implicit
-  SupportedStyles:
-    - implicit
-    - each
-    - example
-
-RSpec/ImplicitExpect:
-  Description: Check that a consistent implicit expectation style is used.
-  Enabled: true
-  EnforcedStyle: is_expected
-  SupportedStyles:
-    - is_expected
-    - should
-
-RSpec/InstanceSpy:
-  Description: Checks for `instance_double` used with `have_received`.
   Enabled: false
 
 RSpec/InstanceVariable:
-  Description: Checks for instance variable usage in specs.
-  AssignmentOnly: false
   Enabled: false
 
-RSpec/ItBehavesLike:
-  Description: Checks that only one `it_behaves_like` style is used.
-  Enabled: true
-  EnforcedStyle: it_behaves_like
-  SupportedStyles:
-    - it_behaves_like
-    - it_should_behave_like
-
-RSpec/LeadingSubject:
-  Description: Checks for `subject` definitions that come after `let` definitions.
-  Enabled: true
-
-RSpec/LetSetup:
-  Description: Checks unreferenced `let!` calls being used for test setup.
-  Enabled: true
-
-RSpec/MessageChain:
-  Description: Check that chains of messages are not being stubbed.
-  Enabled: true
-
-RSpec/MessageExpectation:
-  Description: Checks for consistent message expectation style.
-  Enabled: false
-  EnforcedStyle: allow
-  SupportedStyles:
-    - allow
-    - expect
-
 RSpec/MessageSpies:
-  Description: Checks that message expectations are set using spies.
   Enabled: false
-  EnforcedStyle: have_received
-  SupportedStyles:
-    - have_received
-    - receive
-
-RSpec/MultipleDescribes:
-  Description: Checks for multiple top level describes.
-  Enabled: true
 
 RSpec/MultipleExpectations:
-  Description: Checks if examples contain too many `expect` calls.
   Enabled: false
-  Max: 1
 
 RSpec/MultipleMemoizedHelpers:
   Enabled: false
 
-RSpec/NamedSubject:
-  Description: Checks for explicitly referenced test subjects.
-  Enabled: true
-
 RSpec/NestedGroups:
-  Description: Checks for nested example groups.
   Enabled: true
   Max: 4
 
 RSpec/NotToNot:
-  Description: Checks for consistent method usage for negating expectations.
-  EnforcedStyle: not_to
-  SupportedStyles:
-    - not_to
-    - to_not
   Enabled: false
 
-RSpec/OverwritingSetup:
-  Enabled: true
-  Description: Checks if there is a let/subject that overwrites an existing one.
-
-RSpec/RepeatedDescription:
-  Enabled: true
-  Description: Check for repeated description strings in example groups.
-
-RSpec/RepeatedExample:
-  Enabled: true
-  Description: Check for repeated examples within example groups.
-
-RSpec/SharedContext:
-  Description: Checks for proper shared_context and shared_examples usage.
-  Enabled: true
-
-RSpec/SingleArgumentMessageChain:
-  Description: Checks that chains of messages contain more than one element.
-  Enabled: true
-
-RSpec/ScatteredLet:
-  Description: Checks for let scattered across the example group.
-  Enabled: true
-
 RSpec/ScatteredSetup:
-  Description: Checks for setup scattered across multiple hooks in an example group.
-  Enabled: false
-
-RSpec/SubjectStub:
-  Description: Checks for stubbed test subjects.
-  Enabled: true
-
-RSpec/VerifiedDoubles:
-  Description: Prefer using verifying doubles over normal doubles.
   Enabled: false
-  IgnoreSymbolicNames: true
-
-Metrics/BlockLength:
-  Exclude:
-    - 'spec/**/*.rb'
 
 RSpec/StubbedMock:
   Enabled: false

data/.simplycop_security.yml

@@ -3,9 +3,6 @@ require:
   - './lib/simplycop/security/reject_all_requests_local.rb'
   - './lib/simplycop/security/check_for_vulnerable_code.rb'
 
-AllCops:
-  ExtraDetails: true
-
 Security/CSRFTokenValidation:
   Enabled: true
   Details: >-

data/lib/simplycop/custom_cops/dont_print_all_env.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module CustomCops
+  class DontPrintAllEnv < RuboCop::Cop::Cop
+    #  This cop checks if someone accidentally print all environment variables
+    #  because some of them may contain secrets.
+    #
+    # @example
+    #   # bad
+    #   puts ENV.to_h
+    #   puts `env`
+    #   puts ENVIRON.to_h
+    #
+    #   # good
+    #   puts ENV['SOME_KEY']
+    #   puts ENVIRON['SOME_KEY']
+    MSG = 'Printing all Environment Variables is extremely risky'\
+    ' If this code has been run, then it is likely that secrets have been'\
+    ' exposed in plaintext. Please alert `#infosec` about this so it can be'\
+    ' investigated immediately.'\
+
+    def_node_matcher :convert_env_to_hash_or_array?, <<~PATTERN
+      (send (const nil? {:ENVIRON :ENV}) {:to_h :to_a :to_hash})
+    PATTERN
+
+    def_node_matcher :print_all_env_shell?, <<~PATTERN
+      (send nil? {:puts :p :print} (xstr(str "env")))
+    PATTERN
+
+    def on_send(node)
+      return unless convert_env_to_hash_or_array?(node) || print_all_env_shell?(node)
+
+      add_offense(node, location: :selector)
+    end
+  end
+end

data/lib/simplycop/version.rb

@@ -7,5 +7,5 @@
 #
 
 module Simplycop
-  VERSION = '1.2.0'
+  VERSION = '1.5.0'
 end

data/simplycop.gemspec

@@ -17,9 +17,9 @@ Gem::Specification.new do |spec|
   spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'rubocop', '~> 1.6.1'
+  spec.add_dependency 'rubocop', '~> 1.8.1'
   spec.add_dependency 'rubocop-rails', '~> 2.9.0'
-  spec.add_dependency 'rubocop-rspec', '~> 2.0.1'
+  spec.add_dependency 'rubocop-rspec', '~> 2.1.0'
   spec.add_development_dependency 'bundler'
   spec.add_development_dependency 'rake', '>= 12.3.3'
   spec.add_development_dependency 'rspec', '~> 3.10'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: simplycop
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.5.0
 platform: ruby
 authors:
 - Simply Business
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-12-15 00:00:00.000000000 Z
+date: 2021-01-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.6.1
+        version: 1.8.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.6.1
+        version: 1.8.1
 - !ruby/object:Gem::Dependency
   name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
@@ -44,14 +44,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.0.1
+        version: 2.1.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.0.1
+        version: 2.1.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -103,6 +103,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".custom_simplycop.yml"
+- ".github/dependabot.yml"
 - ".github/workflows/ci.yml"
 - ".github/workflows/publish_gem.yml"
 - ".github/workflows/version_forget_me_not.yml"
@@ -124,6 +125,7 @@ files:
 - lib/simplycop.rb
 - lib/simplycop/custom_cops/constantize.rb
 - lib/simplycop/custom_cops/define_method.rb
+- lib/simplycop/custom_cops/dont_print_all_env.rb
 - lib/simplycop/custom_cops/instance_eval.rb
 - lib/simplycop/custom_cops/method_missing.rb
 - lib/simplycop/custom_cops/timecop_without_block.rb
@@ -152,7 +154,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
+rubygems_version: 3.2.3
 signing_key: 
 specification_version: 4
 summary: Provides a single point of reference for common rubocop rules.