Merge pull request #187 from sinsoku/make_excludes_absolute

Fix the issue that excluded files in rubocop-rails did not work

Author: koic

CHANGELOG.md

@@ -6,6 +6,7 @@
 
 * [#184](https://github.com/rubocop-hq/rubocop-rails/issues/184): Fix `Rake/Environment` to allow task with no block. ([@hanachin][])
 * [#122](https://github.com/rubocop-hq/rubocop-rails/issues/122): Fix `Exclude` paths that were not inherited. ([@koic][])
+* [#187](https://github.com/rubocop-hq/rubocop-rails/pull/187): Fix an issue that excluded files in rubocop-rails did not work. ([@sinsoku][])
 
 ## 2.4.1 (2019-12-25)
 

lib/rubocop/rails/inject.rb

@@ -8,7 +8,7 @@ module Inject
       def self.defaults!
         path = CONFIG_DEFAULT.to_s
         hash = ConfigLoader.send(:load_yaml_configuration, path)
-        config = Config.new(hash, path)
+        config = Config.new(hash, path).tap(&:make_excludes_absolute)
         puts "configuration from #{path}" if ConfigLoader.debug?
         config = ConfigLoader.merge_with_default(config, path, unset_nil: false)
         ConfigLoader.instance_variable_set(:@default_configuration, config)

spec/rubocop/rails/inject_spec.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+RSpec.describe RuboCop::Rails::Inject do
+  describe '#defaults!' do
+    before { allow(Dir).to receive(:pwd).and_return('/home/foo/project') }
+
+    it 'makes excludes absolute' do
+      configuration = described_class.defaults!
+      expect(configuration['AllCops']['Exclude'])
+        .to include(
+          '/home/foo/project/bin/*',
+          '/home/foo/project/db/schema.rb'
+        )
+    end
+  end
+end