[Fix rubocop#893] Support local as an environment for `Rails/Unknow…

…nEnv` from Rails 7.1 onward. Fixes rubocop#893
ghiculescu · Dec 23, 2022 · 3db5d93 · 3db5d93
1 parent b930670
commit 3db5d93
Show file tree

Hide file tree

Showing 3 changed files with 42 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,8 @@
 
 ## master (unreleased)
 
+* [#893](https://github.com/rubocop/rubocop-rails/issues/893): Support `local` as an environment for `Rails/UnknownEnv` from Rails 7.1 onward. ([@ghiculescu][])
+
 ## 2.17.3 (2022-11-20)
 
 ### Bug fixes

diff --git a/lib/rubocop/cop/rails/unknown_env.rb b/lib/rubocop/cop/rails/unknown_env.rb
@@ -86,7 +86,9 @@ def unknown_env_name?(name)
         end
 
         def environments
-          cop_config['Environments']
+          e = cop_config['Environments']
+          e += ['local'] if target_rails_version >= 7.1
+          e
         end
       end
     end

diff --git a/spec/rubocop/cop/rails/unknown_env_spec.rb b/spec/rubocop/cop/rails/unknown_env_spec.rb
@@ -24,6 +24,8 @@
                     ^^^^^^^^^^^^^ Unknown environment `developpment`. Did you mean `development`?
           Rails.env.something?
                     ^^^^^^^^^^ Unknown environment `something`.
+          Rails.env.local?
+                    ^^^^^^ Unknown environment `local`.
         RUBY
       end
 
@@ -36,6 +38,9 @@
 
           'something' === Rails.env
           ^^^^^^^^^^^ Unknown environment `something`.
+
+          'local' === Rails.env
+          ^^^^^^^ Unknown environment `local`.
         RUBY
       end
     end
@@ -49,6 +54,9 @@
                     ^^^^^^^^^^^^^ Unknown environment `developpment`.
           Rails.env.something?
                     ^^^^^^^^^^ Unknown environment `something`.
+
+          Rails.env.local?
+                    ^^^^^^ Unknown environment `local`.
         RUBY
       end
 
@@ -61,6 +69,9 @@
 
           'something' === Rails.env
           ^^^^^^^^^^^ Unknown environment `something`.
+
+          'local' === Rails.env
+          ^^^^^^^ Unknown environment `local`.
         RUBY
       end
     end
@@ -72,4 +83,30 @@
       Rails.env == 'production'
     RUBY
   end
+
+  context 'Rails 7.1' do
+    let(:config) do
+      RuboCop::Config.new(
+        {
+          'AllCops' => {
+            'TargetRailsVersion' => '7.1'
+          },
+          'Rails/UnknownEnv' => {
+            'Environments' => %w[
+              development
+              production
+              test
+            ]
+          }
+        }
+      )
+    end
+
+    it 'accepts local as an environment name on Rails 7.1' do
+      expect_no_offenses(<<~RUBY)
+        Rails.env.local?
+        Rails.env == 'local'
+      RUBY
+    end
+  end
 end