Rails RSpec入门问题：测试为何时而通过时而失败？

问题背景 #

我维护一个书籍数据库，书籍支持多种装帧格式（精装、平装等），使用factory_bot编写测试工厂。测试时偶尔会出现错误，但第二次运行又恢复正常，不知道该从哪里入手排查。

错误信息 #

1) BookFormat displays the proper book format for a book with that format
   Failure/Error: expect(@book.book_format.name).to eq('Hardcover')
   
     expected: "Hardcover"
          got: "Not defined"

相关代码 #

测试代码 #

require 'rails_helper'

RSpec.describe BookFormat, type: :model do
  before(:all) do
    @book = create(:hobbit)
    @book_format_default = create(:not_defined)
  end

  it 'displays the proper book format for a book with that format' do
    expect(@book.book_format.name).to eq('Hardcover')
  end

  it 'should reassign to the fallback book_format if their book_format is deleted' do
    format = @book.book_format
    format.destroy
    expect(@book.reload.book_format.id).to eq(@book_format_default.id)
  end

  it 'should not let the fallback format be deleted' do
    format = @book_format_default
    format.destroy
    expect(format).to be_truthy
  end

end

Hobbit书籍工厂 #

factory :hobbit, class: Book do
  title { 'The Hobbit' }
  year { 1937 }
  rating { 5 }
  condition { 4 }
  synopsis {  "<p>#{Faker::Lorem.paragraphs(number: 30).join(' ')}</p>" }
  association :book_format, factory: :hardcover
  association :user, factory: :me
  genres { [ create(:fiction) ] }

  after(:build) do |hobbit|
    hobbit.cover.attach(
      # rubocop:disable Rails/FilePath
      io: File.open(Rails.root.join('db', 'sample', 'images', 'cover-1.jpg')),
      # rubocop:enable Rails/FilePath
      filename: 'cover.jpg',
      content_type: 'image/jpeg'
    )
  end
end

BookFormat工厂 #

FactoryBot.define do
  factory :not_defined, class: BookFormat  do
    name { 'Not defined'}
    fallback { true }
  end
  factory :hardcover, class: BookFormat do
    name { 'Hardcover' }
  end

  factory :softcover, class: BookFormat do
    name { 'Softcover' }
  end
end

排查方向与解决方案 #

1. 核心问题：测试数据共享与污染 #

你用了before(:all)初始化测试数据，这个钩子会在所有测试用例执行前只运行一次，创建的数据会在三个测试用例之间共享。第二个测试用例删除了@book关联的精装格式，触发了回退到默认格式的逻辑——这直接修改了@book的关联关系，若测试用例执行顺序随机，就会导致第一个测试用例断言失败，出现偶发错误。

解决方案：把before(:all)改成before(:each)，让每个测试用例都使用全新的测试数据，彻底避免数据污染：

require 'rails_helper'

RSpec.describe BookFormat, type: :model do
  before(:each) do
    @book = create(:hobbit)
    @book_format_default = create(:not_defined)
  end

  # 测试用例保持不变
end

2. 默认格式的唯一性问题 #

如果BookFormat的fallback字段应该是唯一的（只能有一个默认格式），多次调用create(:not_defined)会导致重复创建，或factory_bot返回已存在的记录，干扰关联逻辑。

解决方案：修改默认格式的工厂，使用find_or_create_by确保只存在一个默认格式：

FactoryBot.define do
  factory :not_defined, class: BookFormat  do
    name { 'Not defined'}
    fallback { true }

    # 确保只创建一个fallback为true的格式
    initialize_with { BookFormat.find_or_create_by(fallback: true) }
  end
  # 其他工厂代码不变
end

3. 数据库清理策略 #

如果测试套件未配置自动清理数据库，之前测试残留的数据可能和当前测试数据冲突，引发偶发错误。

解决方案：启用数据库清理机制，比如使用database_cleaner gem，配置为每个测试用例后截断数据库表，保证测试环境干净。

4. 关联回调逻辑检查 #

检查BookFormat模型的destroy回调逻辑，确保只有当格式被删除时，才会将关联书籍切换到默认格式，且不会误修改其他未关联该格式的书籍。示例：

# 在BookFormat模型中
before_destroy :reassign_books_to_fallback

private

def reassign_books_to_fallback
  return if fallback? # 阻止默认格式被删除

  fallback_format = BookFormat.find_by(fallback: true)
  return unless fallback_format

  books.update_all(book_format_id: fallback_format.id)
end

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