rubocop-petal 0.3.1 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 85df3da29b781802cb5b82029453d2c2045f6940de4379684593eb2bbac8e293
-  data.tar.gz: b139d8a8158cb1be68636d86ed22fc3edc99df08f85c476b03800081f032d1b8
+  metadata.gz: 8b48ddf9490ef1ea965a112b54d5b298eb38056026c773ee89106e9bc3bbf432
+  data.tar.gz: 12e03252072b61ec108a3f3d05a9410d4c5808b4544c9399593355cc934e0835
 SHA512:
-  metadata.gz: 13450196ed8434f7f47b228ff05f2d4cd4890bd0dda71e2e7a8bb17de4c4a064f4946e185b0edf0dfbbabbb59a6b2d6c4840b4ef091074f026de34be0405f67c
-  data.tar.gz: 3d622112d46ce09c5e351f90420b6d35a391204ca4af35c5de54e65c9df39e1ac608608aef8995e3a30d6adbeb739b8f77452146eb41f6d7f14571cd1be33e23
+  metadata.gz: 4e192999e69e750045f9e55b737348db33609a27a616f07ba1c168569c3d8af090f09ee21a6a3dac13e6c85d84c5b5bb1325d5cfb6aef17bbd8768603701e1e2
+  data.tar.gz: c87d8b6e74a8250d2062d6a2d0d54ae7d896391db9163a459d9ad16f3ddb8658f53a0ebfe7802a8625ef62e11f561da1c63cc608d2cdcfefcc487d6db33734b2

data/CHANGELOG.md

@@ -2,9 +2,13 @@
 
 # main
 
+#v0.4.0
+
+* Added cop `RSpec/StubProducts` ([#18](https://github.com/petalmd/rubocop-petal/pull/18))
+
 # v0.3.1
 
-Correct cop name `Migration/SchemaStatementsMethods` in config ([#17](https://github.com/petalmd/rubocop-petal/pull/17))
+* Correct cop name `Migration/SchemaStatementsMethods` in config ([#17](https://github.com/petalmd/rubocop-petal/pull/17))
 
 # v0.3.0
 

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    rubocop-petal (0.3.1)
+    rubocop-petal (0.4.0)
       rubocop (>= 1.7.0, < 2.0)
       rubocop-rails (~> 2.10)
 
@@ -51,7 +51,7 @@ GEM
       unicode-display_width (>= 1.4.0, < 3.0)
     rubocop-ast (1.15.0)
       parser (>= 3.0.1.1)
-    rubocop-rails (2.13.2)
+    rubocop-rails (2.14.2)
       activesupport (>= 4.2.0)
       rack (>= 1.1)
       rubocop (>= 1.7.0, < 2.0)

data/config/default.yml

@@ -30,6 +30,13 @@ RSpec/CreateListMax:
   Include:
     - spec/**/*
 
+RSpec/StubProducts:
+  Description: 'Suggest to use stub_products instead of veil/unveil_product.'
+  Enabled: true
+  SafeAutocorrect: false
+  Include:
+    - spec/**/*
+
 Grape/HelpersIncludeModule:
   Description: 'Prevent using helpers with block to include module'
   Enabled: true

data/lib/rubocop/cop/rspec/stub_products.rb

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module RSpec
+      # Suggest to use stub_products instead of veil/unveil_product
+      # which queries the database and can cause flaky tests.
+      #
+      #   # bad
+      #   unveil_product('MY_PRODUCT')
+      #   veil_product('MY_PRODUCT')
+      #
+      #   # good
+      #   stub_products('MY_PRODUCT' => true)
+      #   stub_products('MY_PRODUCT' => false)
+      #   stub_products('MY_PRODUCT' => group)
+      #   stub_products('MY_PRODUCT' => [group1, group2])
+      #   stub_products(MY_PRODUCT: true)
+      #
+      class StubProducts < Base
+        extend AutoCorrector
+
+        MSG = 'Use `stub_products` instead of veil/unveil_product.'
+
+        def_node_search :veil_product?, <<~PATTERN
+          (send nil? :veil_product _)
+        PATTERN
+
+        def_node_search :unveil_product?, <<~PATTERN
+          (send nil? :unveil_product _)
+        PATTERN
+
+        def on_send(node)
+          return unless veil_product?(node) || unveil_product?(node)
+
+          add_offense(node) do |corrector|
+            if (match = /^\S*\s+(\S+)|\(([^)]+)\)/.match(node.source))
+              match1, match2 = match.captures
+              product_code = match1 || match2
+
+              product_is_available = !veil_product?(node)
+              subst = "stub_products(#{product_code} => #{product_is_available})"
+
+              corrector.replace(node, subst)
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/petal/version.rb

@@ -2,6 +2,6 @@
 
 module RuboCop
   module Petal
-    VERSION = '0.3.1'
+    VERSION = '0.4.0'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-petal
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.4.0
 platform: ruby
 authors:
 - Jean-Francis Bastien
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-03-09 00:00:00.000000000 Z
+date: 2022-04-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -77,6 +77,7 @@ files:
 - lib/rubocop/cop/rails/table_name.rb
 - lib/rubocop/cop/rspec/authenticated_as.rb
 - lib/rubocop/cop/rspec/create_list_max.rb
+- lib/rubocop/cop/rspec/stub_products.rb
 - lib/rubocop/petal.rb
 - lib/rubocop/petal/inject.rb
 - lib/rubocop/petal/version.rb