Merge pull request #229 from koic/make_unique_validation_without_inde…

…x_aware_schema

[Fix #227] Make `Rails/UniqueValidationWithoutIndex` aware of updating schema.rb

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## master (unreleased)
 
+### Bug fixes
+
+* [#227](https://github.com/rubocop-hq/rubocop-rails/issues/227): Make `Rails/UniqueValidationWithoutIndex` aware of updating db/schema.rb. ([@koic][])
+
 ## 2.5.1 (2020-04-02)
 
 ### Bug fixes

lib/rubocop/cop/mixin/active_record_helper.rb

@@ -14,6 +14,23 @@ module ActiveRecordHelper
         (send nil? :belongs_to {str sym} ...)
       PATTERN
 
+      def external_dependency_checksum
+        if defined?(@external_dependency_checksum)
+          return @external_dependency_checksum
+        end
+
+        schema_path = RuboCop::Rails::SchemaLoader.db_schema_path
+        return nil if schema_path.nil?
+
+        schema_code = File.read(schema_path)
+
+        @external_dependency_checksum ||= Digest::SHA1.hexdigest(schema_code)
+      end
+
+      def schema
+        RuboCop::Rails::SchemaLoader.load(target_ruby_version)
+      end
+
       def table_name(class_node)
         table_name = find_set_table_name(class_node).to_a.last&.first_argument
         return table_name.value.to_s if table_name

lib/rubocop/cop/rails/unique_validation_without_index.rb

@@ -149,10 +149,6 @@ def array_node_to_array(node)
             end
           end
         end
-
-        def schema
-          RuboCop::Rails::SchemaLoader.load(target_ruby_version)
-        end
       end
     end
   end

lib/rubocop/rails/schema_loader.rb

@@ -24,16 +24,6 @@ def reset!
         remove_instance_variable(:@schema)
       end
 
-      private
-
-      def load!(target_ruby_version)
-        path = db_schema_path
-        return unless path
-
-        ast = parse(path, target_ruby_version)
-        Schema.new(ast)
-      end
-
       def db_schema_path
         path = Pathname.pwd
         until path.root?
@@ -46,6 +36,16 @@ def db_schema_path
         nil
       end
 
+      private
+
+      def load!(target_ruby_version)
+        path = db_schema_path
+        return unless path
+
+        ast = parse(path, target_ruby_version)
+        Schema.new(ast)
+      end
+
       def parse(path, target_ruby_version)
         klass_name = :"Ruby#{target_ruby_version.to_s.sub('.', '')}"
         klass = ::Parser.const_get(klass_name)

spec/rubocop/cop/active_record_helper_spec.rb

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+
+RSpec.describe RuboCop::Cop::ActiveRecordHelper, :isolated_environment do
+  include FileHelper
+
+  module RuboCop
+    module Cop
+      class Example < Cop
+        include ActiveRecordHelper
+      end
+    end
+  end
+
+  let(:cop) do
+    RuboCop::Cop::Example.new
+  end
+
+  let(:schema_path) { 'db/schema.rb' }
+
+  describe '#external_dependency_checksum' do
+    subject { cop.external_dependency_checksum }
+
+    context 'with db/schema.rb' do
+      before do
+        create_file(schema_path, <<~RUBY)
+          ActiveRecord::Schema.define(version: 2020_04_08_082625) do
+            create_table "articles" do |t|
+              t.string "title", null: false
+            end
+          end
+        RUBY
+      end
+
+      it { is_expected.to eq '1f263bed5ada8f2292ce7ceebd3c518bac3d2d1d' }
+    end
+
+    context 'with empty db/schema.rb' do
+      before { create_empty_file(schema_path) }
+
+      it { is_expected.to eq 'adc83b19e793491b1c6ea0fd8b46cd9f32e592fc' }
+    end
+
+    context 'without db/schema.rb' do
+      it { is_expected.to be_nil }
+    end
+  end
+end