simplycop 1.19.3 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 340380f1fa0cc31faa8be3cce2be00d8cb11aec98a6561df1692574951b6761b
-  data.tar.gz: 5714996ab72a477ea3bfcdcf8966606ee8e8890ef4fa0d889bdb14ac2e5de6ed
+  metadata.gz: 05bff7f3846c7dcd738db0d321c424e490ff3e5ff5ce1d26b0ba3bbeb5ad8164
+  data.tar.gz: 872c229795cd379265ac594ec523f7e815f276c50b44f318ca56fd498a7c1d04
 SHA512:
-  metadata.gz: 62fe9bdd0071394480397fcf53cd636144f204c4bccf040aba06dffdef736a2d8b109fad225240120872f27a4c1481c07fb0313bdf66fe2d3fe612bbb819e5a6
-  data.tar.gz: fbe61769aba6efa7bb418401c31092883cad62213651b839a7ab4fc76e17496381f2d01b22d7c6c3eb74028fe586857c409a36c3ba1ce3a9bcd8bc0d5b122854
+  metadata.gz: 5459ad328d03406a89c52540fc62be75f196f4eefd6064213355b797727a80024fb0f376b0d0bef5e4311abff0d02e5ce4c4bf5025e5ecbc242df2089ba384d4
+  data.tar.gz: e6fef72378871e962d74dd973b0d0883b2218e1703bbfd498497fdb2c5063d19d400745b3dc428bbb704060517cdfbebba69e42fd8e35aefc6ed3b4f63a03b34

data/.rubocop.yml

@@ -1,6 +1,13 @@
 inherit_from:
   - .simplycop.yml
+  - .simplycop_capybara.yml
+  - .simplycop_factory_bot.yml
+  - .simplycop_performance.yml
+  - .simplycop_rails.yml
   - .simplycop_rspec.yml
+  - .simplycop_rspec_rails.yml
 
 AllCops:
-  TargetRubyVersion: 3.1
+  TargetRubyVersion: 3.1
+  Exclude:
+    - simplycop.gemspec

data/.simplycop_capybara.yml

@@ -0,0 +1,35 @@
+require:
+  - rubocop-capybara
+
+Capybara/ClickLinkOrButtonStyle:
+  Enabled: false
+
+Capybara/CurrentPathExpectation:
+  Enabled: true
+
+Capybara/MatchStyle:
+  Enabled: true
+
+Capybara/NegationMatcher:
+  Enabled: true
+
+Capybara/RedundantWithinFind:
+  Enabled: true
+
+Capybara/SpecificActions:
+  Enabled: true
+
+Capybara/SpecificFinders:
+  Enabled: true
+
+Capybara/SpecificMatcher:
+  Enabled: true
+
+Capybara/VisibilityMatcher:
+  Enabled: true
+
+Capybara/RSpec/HaveSelector:
+  Enabled: true
+
+Capybara/RSpec/PredicateMatcher:
+  Enabled: true

data/.simplycop_factory_bot.yml

@@ -0,0 +1,36 @@
+require:
+  - rubocop-factory_bot
+
+FactoryBot/AssociationStyle:
+  Enabled: true
+
+FactoryBot/AttributeDefinedStatically:
+  Enabled: true
+
+FactoryBot/ConsistentParenthesesStyle:
+  Enabled: true
+
+FactoryBot/CreateList:
+  Enabled: false
+
+FactoryBot/ExcessiveCreateList:
+  Enabled: true
+
+FactoryBot/FactoryAssociationWithStrategy:
+  Enabled: true
+
+FactoryBot/FactoryClassName:
+  Enabled: true
+
+FactoryBot/FactoryNameStyle:
+  Enabled: true
+
+FactoryBot/IdSequence:
+  Enabled: true
+
+FactoryBot/RedundantFactoryOption:
+  Enabled: true
+
+# Prefer explicit factorybot call
+FactoryBot/SyntaxMethods:
+  Enabled: false

data/.simplycop_rails.yml

@@ -415,3 +415,6 @@ Rails/WhereNot:
 # Anticipate conflicts with Mongoid, like Rails/WhereNot
 Rails/WhereNotWithMultipleConditions:
   Enabled: false
+
+Rails/WhereRange:
+  Enabled: true

data/.simplycop_rspec.yml

@@ -70,7 +70,14 @@ RSpec/DescribedClassModuleWrapping:
   Enabled: false
 
 RSpec/Dialect:
-  Enabled: false
+  Enabled: true
+  PreferredMethods:
+    background: :before
+    scenario:   :it
+    xscenario:  :xit
+    given:      :let
+    given!:     :let!
+    feature:    :describe
 
 RSpec/DuplicatedMetadata:
   Enabled: true
@@ -126,10 +133,10 @@ RSpec/ExpectChange:
 RSpec/ExpectInHook:
   Enabled: true
 
-RSpec/ExpectOutput:
-  Enabled: false
+RSpec/ExpectInLet:
+  Enabled: true
 
-RSpec/FilePath:
+RSpec/ExpectOutput:
   Enabled: false
 
 RSpec/Focus:
@@ -200,6 +207,9 @@ RSpec/MetadataStyle:
 RSpec/MissingExampleGroupArgument:
   Enabled: true
 
+RSpec/MissingExpectationTargetMethod:
+  Enabled: true
+
 RSpec/MultipleDescribes:
   Enabled: true
 
@@ -333,119 +343,3 @@ RSpec/VoidExpect:
 
 RSpec/Yield:
   Enabled: true
-
-# Rspec/Capybara
-
-# Similar to FactoryBot (see below) there is an RSpec/Capybara department and a separate Capybara
-# package which is not installed in simplycop. This configuration is just for the RSpec department.
-# The namespacing has been changed to avoid warnings. 
-
-RSpec/Capybara:
-  Enabled: true
-  Include:
-    - spec/integration/*.rb
-
-Capybara/CurrentPathExpectation:
-  Enabled: true
-
-RSpec/Capybara/FeatureMethods:
-  Enabled: true
-
-Capybara/MatchStyle:
-  Enabled: true
-
-Capybara/NegationMatcher:
-  Enabled: true
-
-Capybara/SpecificActions:
-  Enabled: true
-
-Capybara/SpecificFinders:
-  Enabled: true
-
-Capybara/SpecificMatcher:
-  Enabled: true
-
-Capybara/VisibilityMatcher:
-  Enabled: true
-
-# RSpec/FactoryBot
-# NB: There is an RSpec/FactoryBot department and a separate FactoryBot package. The FactoryBot package is 
-# not installed on simplycop. The configuration is just for the RSpec department, namespaced to avoid warnings.
-
-FactoryBot/AttributeDefinedStatically:
-  Enabled: true
-
-FactoryBot/ConsistentParenthesesStyle:
-  Enabled: true
-
-FactoryBot/CreateList:
-  Enabled: false
-
-FactoryBot/FactoryClassName:
-  Enabled: true
-
-FactoryBot/FactoryNameStyle:
-  Enabled: true
-
-# Prefer explicit factorybot call
-FactoryBot/SyntaxMethods:
-  Enabled: false
-
-# Previously some FactoryBot package cops were configured due to the above confusion. There is less crossover between 
-# RSpec/FactoryBot and FactoryBot than Capybara, so it generated a number of errors when they weren't found.
-# This is how they were configured:
-
-# FactoryBot/AssociationStyle:
-#   Enabled: true
-
-# FactoryBot/AttributeDefinedStatically:
-#   Enabled: true
-
-# FactoryBot/ConsistentParenthesesStyle:
-#   Enabled: true
-
-# FactoryBot/CreateList:
-#   Enabled: false
-
-# FactoryBot/FactoryAssociationWithStrategy:
-#   Enabled: true
-
-# FactoryBot/FactoryClassName:
-#   Enabled: true
-
-# FactoryBot/FactoryNameStyle:
-#   Enabled: true
-
-# FactoryBot/IdSequence:
-#   Enabled: true
-
-# FactoryBot/RedundantFactoryOption:
-#   Enabled: true
-
-# # Prefer explicit factorybot call
-# FactoryBot/SyntaxMethods:
-#   Enabled: false
-
-# Rspec/Rails
-
-RSpecRails/AvoidSetupHook: # new in 2.4
-  Enabled: true
-
-RSpecRails/HaveHttpStatus:
-  Enabled: true
-
-RSpecRails/HttpStatus:
-  Enabled: true
-
-RSpecRails/InferredSpecType:
-  Enabled: true
-
-RSpecRails/MinitestAssertions:
-  Enabled: true
-
-RSpecRails/NegationBeValid:
-  Enabled: true
-
-RSpecRails/TravelAround:
-  Enabled: true

data/.simplycop_rspec_rails.yml

@@ -0,0 +1,23 @@
+require:
+  - rubocop-rspec_rails
+
+RSpecRails/AvoidSetupHook:
+  Enabled: true
+
+RSpecRails/HaveHttpStatus:
+  Enabled: true
+
+RSpecRails/HttpStatus:
+  Enabled: true
+
+RSpecRails/InferredSpecType:
+  Enabled: true
+
+RSpecRails/MinitestAssertions:
+  Enabled: true
+
+RSpecRails/NegationBeValid:
+  Enabled: true
+
+RSpecRails/TravelAround:
+  Enabled: true

data/docs/index.md

@@ -56,6 +56,16 @@ inherit_gem:
     - .simplycop_rspec.yml
 ```
 
+You may also want the following packages:
+```yaml
+
+    - .simplycop_capybara.yml
+    - .simplycop_factory_bot.yml
+    - .simplycop_rspec_rails.yml
+```
+
+NB: Capybara, FactoryBot and RSpecRails cops were previously available either via departments within `rubocop_rspec` or as packages in their own right. As of `rubocop_rspec` v3, they no longer exist as departments within `rubocop_rspec` and the packages need installing independently. If you use simplycop v1, you will still have access to some of these cops by inheriting `.simplycop_rspec`. As of simplycop v2 (which uses `rubocop_rspec` v3) you would need to inherit the packages individually.
+
 ## Usage
 
 Run Rubocop as you would usually do, i.e.

data/lib/simplycop/security/reject_all_requests_local.rb

@@ -19,7 +19,7 @@ module Security
     end
 
     def found_match(string)
-      string.match(/config.consider_all_requests\S?.*=\s?.*true/) ? true : false
+      /config.consider_all_requests\S?.*=\s?.*true/.match?(string)
     end
   end
 end

data/lib/simplycop/version.rb

@@ -7,5 +7,5 @@
 #
 
 module Simplycop
-  VERSION = '1.19.3'
+  VERSION = '2.0.1'
 end

data/simplycop.gemspec

@@ -17,10 +17,13 @@ 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.64.0'
-  spec.add_dependency 'rubocop-performance', '1.21.0'
-  spec.add_dependency 'rubocop-rails', '2.25.0'
-  spec.add_dependency 'rubocop-rspec', '2.29.2'
+  spec.add_dependency 'rubocop', '1.64.1'
+  spec.add_dependency 'rubocop-capybara', '2.21.0'
+  spec.add_dependency 'rubocop-factory_bot', '2.26.1'
+  spec.add_dependency 'rubocop-performance', '1.21.1'
+  spec.add_dependency 'rubocop-rails', '2.25.1'
+  spec.add_dependency 'rubocop-rspec', '3.0.2'
+  spec.add_dependency 'rubocop-rspec_rails', '2.30.0'
   spec.add_development_dependency 'bundler', '>= 2.2.15'
   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.19.3
+  version: 2.0.1
 platform: ruby
 authors:
 - Simply Business
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-06-26 00:00:00.000000000 Z
+date: 2024-07-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,56 +16,98 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.64.0
+        version: 1.64.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.64.0
+        version: 1.64.1
+- !ruby/object:Gem::Dependency
+  name: rubocop-capybara
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 2.21.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 2.21.0
+- !ruby/object:Gem::Dependency
+  name: rubocop-factory_bot
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 2.26.1
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 2.26.1
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.21.0
+        version: 1.21.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.21.0
+        version: 1.21.1
 - !ruby/object:Gem::Dependency
   name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.25.0
+        version: 2.25.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.25.0
+        version: 2.25.1
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.29.2
+        version: 3.0.2
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 3.0.2
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec_rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 2.30.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.29.2
+        version: 2.30.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -132,6 +174,8 @@ files:
 - ".ruby-version"
 - ".simplycop.yml"
 - ".simplycop_bundler.yml"
+- ".simplycop_capybara.yml"
+- ".simplycop_factory_bot.yml"
 - ".simplycop_gemspec.yml"
 - ".simplycop_layout.yml"
 - ".simplycop_lint.yml"
@@ -142,6 +186,7 @@ files:
 - ".simplycop_performance.yml"
 - ".simplycop_rails.yml"
 - ".simplycop_rspec.yml"
+- ".simplycop_rspec_rails.yml"
 - ".simplycop_security.yml"
 - ".simplycop_style.yml"
 - CODEOWNERS
@@ -152,7 +197,6 @@ files:
 - _pipeline/stage_publishdocs.yml
 - _pipeline/stage_sast_tools.yml
 - bin/console
-- bin/setup
 - catalog-info.yaml
 - docs/index.md
 - lib/simplycop.rb

data/bin/setup

@@ -1,8 +0,0 @@
-#!/usr/bin/env bash
-set -euo pipefail
-IFS=$'\n\t'
-set -vx
-
-bundle install
-
-# Do any other automated setup that you need to do here