gl_rubocop 0.5.2 → 0.5.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fa594b531f0d00238170541e84317172b54acf43dde2e41eede7ad26f7d20d19
-  data.tar.gz: 6f3a07046f7d082f0bcbfe6f98754350ab5966d10c97de0cbac00ad3e189f955
+  metadata.gz: 1047a8465eb718c62b4582a7c18f339e17acb15bf0861c60460d21f2f1d74a65
+  data.tar.gz: c85258ff5eb9526fc05b9eb6d8dd15aa8b63f94fe868ed434356259080153fc8
 SHA512:
-  metadata.gz: 6726a89cc1d5de58ecb536f729af9ac08a44735ff91e03cba605e50d59193e773644474da29e0068e4b204e169973d921f7a2f3b85b10c666850c5422f61d685
-  data.tar.gz: d9497503468a4d3ce800156293299f857ec5766bbae85887e20699aa62ed71af7d4a5aded239ca56c49c07cad272b778576086dc505013fe30c1c7941ff6cb17
+  metadata.gz: 39d62d81dfbb69a9e213efb8df7f02486906849e9218572d3a48a76758715e3f598c14bbbcce9abfd483edc96f32f8d9dcb2167c44085774bd3e71573ac552a9
+  data.tar.gz: c32264166f4d1ba98cd73606541671718308de01b47e38c9b08cc59d9aa4496f1ea9a33385807e1aabe9a129e2c920e0433266107588028ad038c98b83e595af

data/default.yml

@@ -12,6 +12,7 @@ require:
   - ./lib/gl_rubocop/gl_cops/consolidate_request_system_specs.rb
   - ./lib/gl_rubocop/gl_cops/interactor_inherits_from_interactor_base.rb
   - ./lib/gl_rubocop/gl_cops/limit_flash_options.rb
+  - ./lib/gl_rubocop/gl_cops/no_stubbing_env.rb
   - ./lib/gl_rubocop/gl_cops/no_stubbing_perform_async.rb
   - ./lib/gl_rubocop/gl_cops/prevent_haml_files.rb
   - ./lib/gl_rubocop/gl_cops/rails_cache.rb
@@ -62,6 +63,11 @@ GLCops/InteractorInheritsFromInteractorBase:
 GLCops/LimitFlashOptions:
   Enabled: true
 
+GLCops/NoStubbingEnv:
+  Enabled: true
+  Include:
+    - "**/*spec.rb"
+
 GLCops/NoStubbingPerformAsync:
   Enabled: true
   Include:

data/lib/gl_rubocop/gl_cops/no_stubbing_env.rb

@@ -0,0 +1,42 @@
+module GLRubocop
+  module GLCops
+    # This cop ensures that you don't stub ENV with `allow`/`expect` + `receive`.
+    #
+    # Stubbing ENV this way (even with `and_call_original`) only stubs the env vars the
+    # spec explicitly knows about. Any var that is read but not stubbed - for example a
+    # var that is present on CI but not locally, or vice versa - behaves inconsistently
+    # and causes flaky failures.
+    #
+    # Instead, replace ENV wholesale with `stub_const`, building the hash from the real
+    # ENV so unrelated vars keep working everywhere:
+    #
+    # Good:
+    #   stub_const('ENV', ENV.to_hash.except('VAR_TO_UNSET').merge('VAR_TO_SET' => 'value'))
+    #
+    # Bad:
+    #   allow(ENV).to receive(:[]).and_call_original
+    #   allow(ENV).to receive(:[]).with('VAR_TO_SET').and_return('value')
+    #   expect(ENV).to receive(:fetch)
+    class NoStubbingEnv < RuboCop::Cop::Base
+      MSG = "Don't stub ENV with allow/expect + receive. Use " \
+            "stub_const('ENV', ENV.to_hash.except('VAR_TO_UNSET')." \
+            "merge('VAR_TO_SET' => 'value')) instead, so unrelated ENV vars " \
+            '(e.g. on CI) keep working.'.freeze
+
+      # Match `allow(ENV).to receive(...)` / `expect(ENV).not_to receive(...)`, including
+      # chained forms like `.and_call_original`, `.and_return(...)`, `.with(...)`.
+      def_node_matcher :stubbing_env?, <<~PATTERN
+        (send
+          (send nil? {:allow :expect} (const {nil? cbase} :ENV))
+          {:to :not_to :to_not}
+          `(send nil? :receive ...))
+      PATTERN
+
+      def on_send(node)
+        return unless stubbing_env?(node)
+
+        add_offense(node)
+      end
+    end
+  end
+end

data/lib/gl_rubocop/version.rb

@@ -1,3 +1,3 @@
 module GLRubocop
-  VERSION = '0.5.2'.freeze
+  VERSION = '0.5.3'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gl_rubocop
 version: !ruby/object:Gem::Version
-  version: 0.5.2
+  version: 0.5.3
 platform: ruby
 authors:
 - Give Lively
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-05-06 00:00:00.000000000 Z
+date: 2026-05-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -166,6 +166,7 @@ files:
 - lib/gl_rubocop/gl_cops/consolidate_request_system_specs.rb
 - lib/gl_rubocop/gl_cops/interactor_inherits_from_interactor_base.rb
 - lib/gl_rubocop/gl_cops/limit_flash_options.rb
+- lib/gl_rubocop/gl_cops/no_stubbing_env.rb
 - lib/gl_rubocop/gl_cops/no_stubbing_perform_async.rb
 - lib/gl_rubocop/gl_cops/prevent_haml_files.rb
 - lib/gl_rubocop/gl_cops/rails_cache.rb
@@ -201,7 +202,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.26
+rubygems_version: 3.5.22
 signing_key:
 specification_version: 4
 summary: A shareable configuration of Give Lively's rubocop rules.