Merge pull request #1398 from viralpraxis/fix-rails-file-path-cop-error-on-join-method-implicit-receiver

Fix `Rails/FilePath` cop error on `join` method with implicit receiver

Author: koic

changelog/fix_merge_pull_request_1392_from.md

@@ -0,0 +1 @@
+* [#1397](https://github.com/rubocop/rubocop-rails/pull/1397): Fix `Rails/FilePath` cop error on `join` method with implicit receiver. ([@viralpraxis][])

lib/rubocop/cop/rails/file_path.rb

@@ -66,6 +66,8 @@ def on_dstr(node)
 
         def on_send(node)
           check_for_file_join_with_rails_root(node)
+          return unless node.receiver
+
           check_for_rails_root_join_with_slash_separated_path(node)
           check_for_rails_root_join_with_string_arguments(node)
         end

spec/rubocop/cop/rails/file_path_spec.rb

@@ -236,6 +236,14 @@
         RUBY
       end
     end
+
+    context 'with `join` method with implicit receiver' do
+      it 'does not register an offense' do
+        expect_no_offenses(<<~RUBY)
+          join(Rails.root, path)
+        RUBY
+      end
+    end
   end
 
   context 'when EnforcedStyle is `arguments`' do
@@ -420,5 +428,13 @@
         RUBY
       end
     end
+
+    context 'with `join` method with implicit receiver' do
+      it 'does not register an offense' do
+        expect_no_offenses(<<~RUBY)
+          join(Rails.root, path)
+        RUBY
+      end
+    end
   end
 end