运行Minitest时出现ActiveRecord::RecordInvalid邮箱重复错误求助

执行rake test时遇到如下错误：

Error: ShareTest#test_valid_setup: ActiveRecord::RecordInvalid: Validation failed: Email has already been taken test/models/share_test.rb:7:in `setup'

我对此感到困惑——即使在setup中覆盖email参数仍会触发该错误，其他测试均正常通过。

模型关联关系 #

• User has_many Items、Trades和Shares
• Trade belongs_to User和Share
• Share belongs_to User和Item
• Item belongs_to User且has_many Trades和Shares

相关FactoryBot工厂代码 #

FactoryBot.define do 
  factory :user, aliases: [:owner, :trade_requester, :trade_recipient, :share_owner] do 
    email Faker::Internet.email 
    firstname Faker::Name.first_name 
    lastname Faker::Name.last_name 
    username Faker::Pokemon.name 
    website Faker::Internet.domain_name 
  end 

  factory :item do 
    owner 
    image { Rack::Test::UploadedFile.new(Rails.root.join('test', 'lilac.jpg'), 'image/jpeg') } 
    title Faker::Book.title 
    artist Faker::Book.author 
    medium Faker::Book.genre 
    year Faker::Number.number(4) 
    price Faker::Number.between(1, 5) 
    length Faker::Number.between(1, 2) 
    height Faker::Number.between(1, 2) 
    width Faker::Number.between(1, 2) 
    agreement true 
  end 

  factory :share, aliases: [:wanted_share, :collateral_share] do 
    user 
    item 
    amount { Item.price } 
    active true 
  end 
end

触发错误的测试代码 #

require 'test_helper' 
require 'support/database_cleaner' 

class ShareTest < Minitest::Test 
  def setup 
    DatabaseCleaner.start 
    @share = FactoryBot.create(:share) 
  end 

  def test_valid_setup 
    assert_kind_of Integer, share.amount 
  end 

  def teardown 
    DatabaseCleaner.clean 
  end 
end

问题原因及解决方法 #

我来帮你拆解这个问题：这个错误的核心是User模型的邮箱唯一性验证被触发了，具体有两个可能的诱因：

1. Faker生成的邮箱重复了 #

Faker::Internet.email虽然是随机生成，但在多次测试运行中，有可能碰巧生成相同的邮箱地址。而你的User模型肯定设置了validates :email, uniqueness: true的验证规则，这就导致创建新User时，系统提示邮箱已被占用。

2. DatabaseCleaner没有彻底清理数据库 #

虽然你在ShareTest里手动加了DatabaseCleaner的setup和teardown，但如果全局配置有漏洞，或者其他测试类没有正确清理数据库，会导致数据库中残留之前测试创建的User记录。当你创建Share时，关联生成的新User邮箱刚好和残留记录的邮箱重复，就会触发错误。

靠谱的解决方法： #

方法一：给User邮箱添加序列（最彻底）

修改User工厂，用FactoryBot的sequence方法生成绝对唯一的邮箱，从根源上避免重复问题：

FactoryBot.define do
  factory :user, aliases: [:owner, :trade_requester, :trade_recipient, :share_owner] do
    # 用sequence生成唯一邮箱
    sequence(:email) { |n| "test_user_#{n}@example.com" }
    firstname Faker::Name.first_name
    lastname Faker::Name.last_name
    username Faker::Pokemon.name
    website Faker::Internet.domain_name
  end
  # 其他工厂代码保持不变
end

方法二：检查DatabaseCleaner全局配置

确保你的test_helper.rb里有正确的全局配置，让所有测试自动在前后清理数据库，不用在单个测试类里重复写代码：

require 'database_cleaner'

# 用事务策略，速度快且干净
DatabaseCleaner.strategy = :transaction

class ActiveSupport::TestCase
  setup do
    DatabaseCleaner.start
  end

  teardown do
    DatabaseCleaner.clean
  end
end

另外顺便提个小细节：你的Share工厂里amount { Item.price }这里的Item.price是调用类方法，应该改成关联的item实例的price，也就是amount { item.price }，否则可能拿到nil或者默认值——不过这和当前的邮箱错误无关，只是个小优化~

内容的提问来源于stack exchange，提问作者calyxofheld