Rails项目全局mock外部API调用及FactoryBot使用实践问题

1. 不推荐在spec_helper中全局mock InvoiceServiceSdk，这不属于最佳实践 #

• 全局mock会完全覆盖SDK的真实调用逻辑，后续如果需要编写SDK本身的集成测试、或者测试SDK调用异常的场景时，全局mock会导致这些用例完全失效，排查成本极高
• allow_any_instance_of本身属于RSpec不推荐的写法，和代码实现强耦合，一旦后续修改SDK初始化逻辑、或fetch方法的调用方式，测试很容易出现假阳性，无法及时暴露问题
• 全局mock会掩盖业务代码中的意外SDK调用，无法区分合理调用和Bug导致的多余调用

更推荐的替代方案 #

优先用依赖注入降低耦合，把Invoice的generate方法改造成支持传入服务实例，不影响原有线上逻辑：

# app/models/invoice.rb
def generate(service = InvoiceServiceSdk.new)
  self.external_id = service.fetch
end

测试需要mock的场景时直接传入mock对象即可，不需要全局覆盖：

let(:mock_service) { double('InvoiceServiceSdk', fetch: 'test_external_id') }

it '生成正确的invoice外部ID' do
  invoice = Invoice.new
  invoice.generate(mock_service)
  expect(invoice.external_id).to eq 'test_external_id'
end

如果不想修改业务代码，也可以只在需要用到SDK mock的测试分组上添加before块，不要做全局配置。

2. 完全可以同时引入Invoice工厂，这属于FactoryBot的标准用法 #

你可以直接在FactoryBot中定义两个模型的关联关系，创建Order工厂时自动生成带合法external_id的Invoice，大部分不涉及Invoice生成逻辑的业务测试完全不需要手动mock SDK。

第一步：定义Invoice工厂 #

# spec/factories/invoices.rb
FactoryBot.define do
  factory :invoice do
    order
    external_id { "inv_#{SecureRandom.alphanumeric(8)}" }
    # 跳过生成回调，避免触发真实SDK请求
    after(:build) do |invoice|
      invoice.class.skip_callback(:create, :before, :generate, raise: false)
    end
  end
end

第二步：在Order工厂中关联Invoice #

# spec/factories/orders.rb
FactoryBot.define do
  factory :order do
    # 保留你原本的Order字段定义
    invoice # 自动关联创建合法的invoice实例
  end
end

配置完成后，测试中直接调用create(:order)就能得到已经绑定了合法Invoice的Order实例，绝大多数普通业务场景的测试都不需要额外处理Invoice的mock逻辑，只有当你要专门测试Invoice生成、SDK调用相关的逻辑时，再单独添加mock即可。

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