missing_non_null_constraint: ignore database views

Author: fatkodima

lib/active_record_doctor/detectors/missing_non_null_constraint.rb

@@ -24,9 +24,11 @@ def message(column:, table:)
 
       def detect
         table_models = models.select(&:table_exists?).group_by(&:table_name)
+        views = connection.views
 
         table_models.each do |table, models|
           next if ignored?(table, config(:ignore_tables))
+          next if views.include?(table)
 
           concrete_models = models.reject do |model|
             model.abstract_class? || sti_base_model?(model)

test/active_record_doctor/detectors/missing_non_null_constraint_test.rb

@@ -259,6 +259,26 @@ def test_not_null_check_constraint
     refute_problems
   end
 
+  def test_views_are_ignored
+    connection = ActiveRecord::Base.connection
+    connection.create_table(:old_comments, force: true) do |t|
+      t.integer :user_id
+    end
+
+    connection.execute(<<-SQL)
+      CREATE VIEW comments AS SELECT * FROM old_comments
+    SQL
+
+    Context.define_model(:Comment) do
+      belongs_to :user, required: true
+    end
+
+    refute_problems
+  ensure
+    connection.execute("DROP VIEW comments")
+    connection.drop_table(:old_comments)
+  end
+
   def test_config_ignore_tables
     Context.create_table(:users) do |t|
       t.string :name, null: true