active_record_doctor 1.10.0 → 1.11.0

data/test/active_record_doctor/detectors/missing_non_null_constraint_test.rb

@@ -4,7 +4,7 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
   def test_optional_columns_with_presence_validator_are_disallowed
     create_table(:users) do |t|
       t.string :name, null: true
-    end.create_model do
+    end.define_model do
       validates :name, presence: true
     end
 
@@ -17,7 +17,7 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
     create_table(:companies)
     create_table(:users) do |t|
       t.references :company, null: true
-    end.create_model do
+    end.define_model do
       belongs_to :company, required: true
     end
 
@@ -26,10 +26,33 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
     OUTPUT
   end
 
+  def test_optional_columns_with_required_polymorphic_association_are_disallowed
+    create_table(:comments) do |t|
+      t.references :commentable, polymorphic: true, null: true
+    end.define_model do
+      belongs_to :commentable, polymorphic: true, required: true
+    end
+
+    assert_problems(<<~OUTPUT)
+      add `NOT NULL` to comments.commentable_id - models validates its presence but it's not non-NULL in the database
+      add `NOT NULL` to comments.commentable_type - models validates its presence but it's not non-NULL in the database
+    OUTPUT
+  end
+
+  def test_required_columns_with_required_polymorphic_association_are_allowed
+    create_table(:comments) do |t|
+      t.references :commentable, polymorphic: true, null: false
+    end.define_model do
+      belongs_to :commentable, polymorphic: true, required: true
+    end
+
+    refute_problems
+  end
+
   def test_required_columns_with_presence_validators_are_allowed
     create_table(:users) do |t|
       t.string :name, null: false
-    end.create_model do
+    end.define_model do
       validates :name, presence: true
     end
 
@@ -39,7 +62,7 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
   def test_optional_columns_without_presence_validator_are_allowed
     create_table(:users) do |t|
       t.string :name, null: false
-    end.create_model do
+    end.define_model do
       validates :name, presence: false
     end
 
@@ -49,7 +72,7 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
   def test_validators_matched_to_correct_columns
     create_table(:users) do |t|
       t.string :name, null: true
-    end.create_model do
+    end.define_model do
       # The age validator is a form of regression test against a bug that
       # caused false positives. In this test case, name is NOT validated
       # for presence so it does NOT need be marked non-NULL. However, the
@@ -65,7 +88,7 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
   def test_validators_with_if_on_optional_columns_are_allowed
     create_table(:users) do |t|
       t.string :name, null: true
-    end.create_model do
+    end.define_model do
       validates :name, presence: true, if: -> { false }
     end
 
@@ -75,7 +98,7 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
   def test_validators_with_unless_on_optional_columns_are_allowed
     create_table(:users) do |t|
       t.string :name, null: true
-    end.create_model do
+    end.define_model do
       validates :name, presence: true, unless: -> { false }
     end
 
@@ -85,7 +108,7 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
   def test_validators_allowing_nil_on_optional_columns_are_allowed
     create_table(:users) do |t|
       t.string :name, null: true
-    end.create_model do
+    end.define_model do
       validates :name, presence: true, allow_nil: true
     end
 
@@ -93,7 +116,7 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
   end
 
   def test_models_with_non_existent_tables_are_skipped
-    create_model(:User)
+    define_model(:User)
 
     refute_problems
   end
@@ -102,9 +125,9 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
     create_table(:users) do |t|
       t.string :type, null: false
       t.string :email, null: true
-    end.create_model
+    end.define_model
 
-    create_model(:Client, ModelFactory::Models::User) do
+    define_model(:Client, TransientRecord::Models::User) do
       validates :email, presence: true
     end
 
@@ -117,13 +140,13 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
     create_table(:users) do |t|
       t.string :type, null: false
       t.string :email, null: true
-    end.create_model
+    end.define_model
 
-    create_model(:Client, ModelFactory::Models::User) do
+    define_model(:Client, TransientRecord::Models::User) do
       validates :email, presence: true
     end
 
-    create_model(:Admin, ModelFactory::Models::User) do
+    define_model(:Admin, TransientRecord::Models::User) do
       validates :email, presence: false
     end
 
@@ -133,11 +156,11 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
   def test_optional_columns_validated_by_all_non_sti_models_are_disallowed
     create_table(:users) do |t|
       t.string :email, null: true
-    end.create_model do
+    end.define_model do
       validates :email, presence: true
     end
 
-    create_model(:Client) do
+    define_model(:Client) do
       self.table_name = :users
 
       validates :email, presence: true
@@ -151,11 +174,11 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
   def test_optional_columns_validated_by_some_non_sti_models_are_allowed
     create_table(:users) do |t|
       t.string :email, null: true
-    end.create_model do
+    end.define_model do
       validates :email, presence: true
     end
 
-    create_model(:Client) do
+    define_model(:Client) do
       self.table_name = :users
 
       validates :email, presence: false
@@ -169,7 +192,7 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
 
     create_table(:users) do |t|
       t.string :email
-    end.create_model do
+    end.define_model do
       validates :email, presence: true
     end
 
@@ -185,7 +208,7 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
 
     create_table(:users) do |t|
       t.string :email
-    end.create_model do
+    end.define_model do
       validates :email, presence: true
     end
 
@@ -201,7 +224,7 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
   def test_config_ignore_tables
     create_table(:users) do |t|
       t.string :name, null: true
-    end.create_model do
+    end.define_model do
       validates :name, presence: true
     end
 
@@ -218,7 +241,7 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
   def test_global_ignore_tables
     create_table(:users) do |t|
       t.string :name, null: true
-    end.create_model do
+    end.define_model do
       validates :name, presence: true
     end
 
@@ -234,7 +257,7 @@ class ActiveRecordDoctor::Detectors::MissingNonNullConstraintTest < Minitest::Te
   def test_config_ignore_columns
     create_table(:users) do |t|
       t.string :name, null: true
-    end.create_model do
+    end.define_model do
       validates :name, presence: true
     end
 

data/test/active_record_doctor/detectors/missing_presence_validation_test.rb

@@ -4,7 +4,7 @@ class ActiveRecordDoctor::Detectors::MissingPresenceValidationTest < Minitest::T
   def test_null_column_is_not_reported_if_validation_absent
     create_table(:users) do |t|
       t.string :name
-    end.create_model do
+    end.define_model do
     end
 
     refute_problems
@@ -13,18 +13,18 @@ class ActiveRecordDoctor::Detectors::MissingPresenceValidationTest < Minitest::T
   def test_non_null_column_is_reported_if_validation_absent
     create_table(:users) do |t|
       t.string :name, null: false
-    end.create_model do
+    end.define_model do
     end
 
     assert_problems(<<~OUTPUT)
-      add a `presence` validator to ModelFactory::Models::User.name - it's NOT NULL but lacks a validator
+      add a `presence` validator to TransientRecord::Models::User.name - it's NOT NULL but lacks a validator
     OUTPUT
   end
 
   def test_non_null_column_is_not_reported_if_validation_present
     create_table(:users) do |t|
       t.string :name, null: false
-    end.create_model do
+    end.define_model do
       validates :name, presence: true
     end
 
@@ -32,10 +32,10 @@ class ActiveRecordDoctor::Detectors::MissingPresenceValidationTest < Minitest::T
   end
 
   def test_non_null_column_is_not_reported_if_association_validation_present
-    create_table(:companies).create_model
+    create_table(:companies).define_model
     create_table(:users) do |t|
       t.references :company, null: false
-    end.create_model do
+    end.define_model do
       belongs_to :company, required: true
     end
 
@@ -43,8 +43,8 @@ class ActiveRecordDoctor::Detectors::MissingPresenceValidationTest < Minitest::T
   end
 
   def test_not_null_column_is_not_reported_if_habtm_association
-    create_table(:users).create_model do
-      has_and_belongs_to_many :projects, class_name: "ModelFactory::Models::Project"
+    create_table(:users).define_model do
+      has_and_belongs_to_many :projects, class_name: "TransientRecord::Models::Project"
     end
 
     create_table(:projects_users) do |t|
@@ -52,8 +52,8 @@ class ActiveRecordDoctor::Detectors::MissingPresenceValidationTest < Minitest::T
       t.bigint :user_id, null: false
     end
 
-    create_table(:projects).create_model do
-      has_and_belongs_to_many :users, class_name: "ModelFactory::Models::User"
+    create_table(:projects).define_model do
+      has_and_belongs_to_many :users, class_name: "TransientRecord::Models::User"
     end
 
     refute_problems
@@ -62,19 +62,19 @@ class ActiveRecordDoctor::Detectors::MissingPresenceValidationTest < Minitest::T
   def test_non_null_boolean_is_reported_if_nil_included
     create_table(:users) do |t|
       t.boolean :active, null: false
-    end.create_model do
+    end.define_model do
       validates :active, inclusion: { in: [nil, true, false] }
     end
 
     assert_problems(<<~OUTPUT)
-      add a `presence` validator to ModelFactory::Models::User.active - it's NOT NULL but lacks a validator
+      add a `presence` validator to TransientRecord::Models::User.active - it's NOT NULL but lacks a validator
     OUTPUT
   end
 
   def test_non_null_boolean_is_not_reported_if_nil_not_included
     create_table(:users) do |t|
       t.boolean :active, null: false
-    end.create_model do
+    end.define_model do
       validates :active, inclusion: { in: [true, false] }
     end
 
@@ -84,22 +84,42 @@ class ActiveRecordDoctor::Detectors::MissingPresenceValidationTest < Minitest::T
   def test_non_null_boolean_is_not_reported_if_nil_excluded
     create_table(:users) do |t|
       t.boolean :active, null: false
-    end.create_model do
+    end.define_model do
       validates :active, exclusion: { in: [nil] }
     end
 
     refute_problems
   end
 
+  def test_non_null_boolean_is_not_reported_if_exclusion_is_proc
+    create_table(:users) do |t|
+      t.boolean :active, null: false
+    end.define_model do
+      validates :active, exclusion: { in: ->(_user) { [nil] } }
+    end
+
+    refute_problems
+  end
+
+  def test_non_null_boolean_is_not_reported_if_inclusion_is_proc
+    create_table(:users) do |t|
+      t.boolean :active, null: false
+    end.define_model do
+      validates :active, inclusion: { in: ->(_user) { [true, false] } }
+    end
+
+    refute_problems
+  end
+
   def test_non_null_boolean_is_reported_if_nil_not_excluded
     create_table(:users) do |t|
       t.boolean :active, null: false
-    end.create_model do
+    end.define_model do
       validates :active, exclusion: { in: [false] }
     end
 
     assert_problems(<<~OUTPUT)
-      add a `presence` validator to ModelFactory::Models::User.active - it's NOT NULL but lacks a validator
+      add a `presence` validator to TransientRecord::Models::User.active - it's NOT NULL but lacks a validator
     OUTPUT
   end
 
@@ -113,14 +133,14 @@ class ActiveRecordDoctor::Detectors::MissingPresenceValidationTest < Minitest::T
       # errors in some MySQL versions when using t.timestamp.
       t.datetime :created_on, null: false
       t.datetime :updated_on, null: false
-    end.create_model do
+    end.define_model do
     end
 
     refute_problems
   end
 
   def test_models_with_non_existent_tables_are_skipped
-    create_model(:User)
+    define_model(:User)
 
     refute_problems
   end
@@ -130,14 +150,14 @@ class ActiveRecordDoctor::Detectors::MissingPresenceValidationTest < Minitest::T
 
     create_table(:users) do |t|
       t.string :name
-    end.create_model
+    end.define_model
 
     ActiveRecord::Base.connection.execute(<<-SQL)
       ALTER TABLE users ADD CONSTRAINT name_not_null CHECK (name IS NOT NULL)
     SQL
 
     assert_problems(<<~OUTPUT)
-      add a `presence` validator to ModelFactory::Models::User.name - it's NOT NULL but lacks a validator
+      add a `presence` validator to TransientRecord::Models::User.name - it's NOT NULL but lacks a validator
     OUTPUT
   end
 
@@ -146,7 +166,7 @@ class ActiveRecordDoctor::Detectors::MissingPresenceValidationTest < Minitest::T
 
     create_table(:users) do |t|
       t.string :name
-    end.create_model
+    end.define_model
 
     ActiveRecord::Base.connection.execute(<<-SQL)
       ALTER TABLE users ADD CONSTRAINT name_not_null CHECK (name IS NOT NULL) NOT VALID
@@ -155,16 +175,24 @@ class ActiveRecordDoctor::Detectors::MissingPresenceValidationTest < Minitest::T
     refute_problems
   end
 
+  def test_abstract_class
+    define_model(:ApplicationRecord) do
+      self.abstract_class = true
+    end
+
+    refute_problems
+  end
+
   def test_config_ignore_models
     create_table(:users) do |t|
       t.string :name, null: false
-    end.create_model do
+    end.define_model do
     end
 
     config_file(<<-CONFIG)
       ActiveRecordDoctor.configure do |config|
         config.detector :missing_presence_validation,
-          ignore_models: ["ModelFactory::Models::User"]
+          ignore_models: ["TransientRecord::Models::User"]
       end
     CONFIG
 
@@ -174,12 +202,12 @@ class ActiveRecordDoctor::Detectors::MissingPresenceValidationTest < Minitest::T
   def test_global_ignore_models
     create_table(:users) do |t|
       t.string :name, null: false
-    end.create_model do
+    end.define_model do
     end
 
     config_file(<<-CONFIG)
       ActiveRecordDoctor.configure do |config|
-        config.global :ignore_models, ["ModelFactory::Models::User"]
+        config.global :ignore_models, ["TransientRecord::Models::User"]
       end
     CONFIG
 
@@ -189,13 +217,13 @@ class ActiveRecordDoctor::Detectors::MissingPresenceValidationTest < Minitest::T
   def test_config_ignore_attributes
     create_table(:users) do |t|
       t.string :name, null: false
-    end.create_model do
+    end.define_model do
     end
 
     config_file(<<-CONFIG)
       ActiveRecordDoctor.configure do |config|
         config.detector :missing_presence_validation,
-          ignore_attributes: ["ModelFactory::Models::User.name"]
+          ignore_attributes: ["TransientRecord::Models::User.name"]
       end
     CONFIG
 

data/test/active_record_doctor/detectors/missing_unique_indexes_test.rb

@@ -5,7 +5,7 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
     create_table(:users) do |t|
       t.string :email
       t.index :email
-    end.create_model do
+    end.define_model do
       validates :email, uniqueness: true
     end
 
@@ -20,7 +20,7 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
     create_table(:users) do |t|
       t.string :email
       t.index "lower(email)"
-    end.create_model do
+    end.define_model do
       validates :email, uniqueness: true
     end
 
@@ -34,7 +34,7 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
     create_table(:users) do |t|
       t.string :email
       t.string :ref_token
-    end.create_model do
+    end.define_model do
       validates :email, :ref_token, uniqueness: true
     end
 
@@ -48,7 +48,7 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
     create_table(:users) do |t|
       t.string :email
       t.index :email, unique: true
-    end.create_model do
+    end.define_model do
       validates :email, uniqueness: true
     end
 
@@ -62,7 +62,7 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
       t.string :email
       t.boolean :active
       t.index :email, unique: true, where: "active"
-    end.create_model do
+    end.define_model do
       validates :email, uniqueness: true
     end
 
@@ -77,7 +77,7 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
       t.integer :company_id
       t.integer :department_id
       t.index [:company_id, :department_id, :email]
-    end.create_model do
+    end.define_model do
       validates :email, uniqueness: { scope: [:company_id, :department_id] }
     end
 
@@ -92,7 +92,7 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
       t.integer :company_id
       t.integer :department_id
       t.index [:company_id, :department_id, :email], unique: true
-    end.create_model do
+    end.define_model do
       validates :email, uniqueness: { scope: [:company_id, :department_id] }
     end
 
@@ -105,7 +105,7 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
       t.integer :company_id
       t.integer :department_id
       t.index [:company_id, :department_id], unique: true
-    end.create_model do
+    end.define_model do
       validates :email, uniqueness: { scope: [:company_id, :department_id] }
     end
 
@@ -115,7 +115,7 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
   def test_missing_unique_index_with_association_attribute
     create_table(:users) do |t|
       t.integer :account_id
-    end.create_model do
+    end.define_model do
       belongs_to :account
       validates :account, uniqueness: true
     end
@@ -129,7 +129,7 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
     create_table(:users) do |t|
       t.integer :account_id
       t.index :account_id, unique: true
-    end.create_model do
+    end.define_model do
       belongs_to :account
       validates :account, uniqueness: true
     end
@@ -142,7 +142,7 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
       t.string :title
       t.integer :commentable_id
       t.string :commentable_type
-    end.create_model do
+    end.define_model do
       belongs_to :commentable, polymorphic: true
       validates :title, uniqueness: { scope: :commentable }
     end
@@ -158,7 +158,7 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
       t.integer :commentable_id
       t.string :commentable_type
       t.index [:commentable_id, :commentable_type, :title], unique: true
-    end.create_model do
+    end.define_model do
       belongs_to :commentable, polymorphic: true
       validates :title, uniqueness: { scope: :commentable }
     end
@@ -172,7 +172,7 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
       t.integer :organization_id
 
       t.index [:email, :organization_id], unique: true
-    end.create_model do
+    end.define_model do
       validates :email, uniqueness: { scope: :organization_id }
     end
 
@@ -199,7 +199,7 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
     create_table(:users) do |t|
       t.string :email
       t.index :email
-    end.create_model do
+    end.define_model do
       validates_with DummyValidator
     end
 
@@ -208,51 +208,51 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
 
   def test_has_one_without_index
     create_table(:users)
-      .create_model do
-        has_one :account, class_name: "ModelFactory::Models::Account"
-        has_one :account_history, through: :account, class_name: "ModelFactory::Models::Account"
+      .define_model do
+        has_one :account, class_name: "TransientRecord::Models::Account"
+        has_one :account_history, through: :account, class_name: "TransientRecord::Models::Account"
       end
 
     create_table(:accounts) do |t|
       t.integer :user_id
-    end.create_model do
-      has_one :account_history, class_name: "ModelFactory::Models::AccountHistory"
+    end.define_model do
+      has_one :account_history, class_name: "TransientRecord::Models::AccountHistory"
     end
 
     create_table(:account_histories) do |t|
       t.integer :account_id
-    end.create_model do
-      belongs_to :account,  class_name: "ModelFactory::Models::Account"
+    end.define_model do
+      belongs_to :account,  class_name: "TransientRecord::Models::Account"
     end
 
     assert_problems(<<~OUTPUT)
-      add a unique index on accounts(user_id) - using `has_one` in the ModelFactory::Models::User model without an index can lead to duplicates
-      add a unique index on account_histories(account_id) - using `has_one` in the ModelFactory::Models::Account model without an index can lead to duplicates
+      add a unique index on accounts(user_id) - using `has_one` in the TransientRecord::Models::User model without an index can lead to duplicates
+      add a unique index on account_histories(account_id) - using `has_one` in the TransientRecord::Models::Account model without an index can lead to duplicates
     OUTPUT
   end
 
   def test_has_one_with_scope_and_without_index
     create_table(:users)
-      .create_model do
-        has_one :last_comment, -> { order(created_at: :desc) }, class_name: "ModelFactory::Models::Comment"
+      .define_model do
+        has_one :last_comment, -> { order(created_at: :desc) }, class_name: "TransientRecord::Models::Comment"
       end
 
     create_table(:comments) do |t|
       t.integer :user_id
-    end.create_model
+    end.define_model
 
     refute_problems
   end
 
   def test_has_one_with_index
     create_table(:users)
-      .create_model do
-        has_one :account, class_name: "ModelFactory::Models::Account"
+      .define_model do
+        has_one :account, class_name: "TransientRecord::Models::Account"
       end
 
     create_table(:accounts) do |t|
       t.integer :user_id, index: { unique: true }
-    end.create_model
+    end.define_model
 
     refute_problems
   end
@@ -260,14 +260,14 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
   def test_config_ignore_models
     create_table(:users) do |t|
       t.string :email
-    end.create_model do
+    end.define_model do
       validates :email, uniqueness: true
     end
 
     config_file(<<-CONFIG)
       ActiveRecordDoctor.configure do |config|
         config.detector :missing_unique_indexes,
-          ignore_models: ["ModelFactory::Models::User"]
+          ignore_models: ["TransientRecord::Models::User"]
       end
     CONFIG
 
@@ -277,13 +277,13 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
   def test_global_ignore_models
     create_table(:users) do |t|
       t.string :email
-    end.create_model do
+    end.define_model do
       validates :email, uniqueness: true
     end
 
     config_file(<<-CONFIG)
       ActiveRecordDoctor.configure do |config|
-        config.global :ignore_models, ["ModelFactory::Models::User"]
+        config.global :ignore_models, ["TransientRecord::Models::User"]
       end
     CONFIG
 
@@ -294,14 +294,14 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
     create_table(:users) do |t|
       t.string :email
       t.integer :role
-    end.create_model do
+    end.define_model do
       validates :email, :role, uniqueness: { scope: :organization_id }
     end
 
     config_file(<<-CONFIG)
       ActiveRecordDoctor.configure do |config|
         config.detector :missing_unique_indexes,
-          ignore_columns: ["ModelFactory::Models::User(organization_id, email)", "ModelFactory::Models::User(organization_id, role)"]
+          ignore_columns: ["TransientRecord::Models::User(organization_id, email)", "TransientRecord::Models::User(organization_id, role)"]
       end
     CONFIG
 
@@ -318,7 +318,7 @@ class ActiveRecordDoctor::Detectors::MissingUniqueIndexesTest < Minitest::Test
   def assert_skipped(options)
     create_table(:users) do |t|
       t.string :email
-    end.create_model do
+    end.define_model do
       validates :email, uniqueness: options
     end