ruby-on-rails - 具有深度关联链的 FactoryGirl 的最佳实践？

Question

我在 Rails 中为复杂的采购工作流程建模，将申请转换为订单。我正在使用 FactoryGirl 进行测试并且一切正常，直到我尝试测试 OrderLineItem，它依赖于订单和报价单，它们各自依赖于其他对象，依此类推...

有问题的测试检查受产品影响的 OrderLineItem 上的行为，这是链上游的几个关联。

有没有一种设置 FactoryGirl 的好方法，这样我就可以轻松构建 OrderLineItems 并规定链中更高层对象的行为，而无需一次分解每个对象？

这是我的对象图:

class Requisition
  has_many :requisition_line_items
  has_many :orders
end

class RequisitionLineItem
  belongs_to :requisition
  belongs_to :product
  has_many :quotes
end

class Quote
  belongs_to :line_item
  belongs_to :vendor
  has_one :order_line_item
end

class Order
  belongs_to :requisition
  belongs_to :vendor
  has_many :order_line_items
end

class OrderLineItem
  belongs_to :order
  belongs_to :quote
  has_many :assets
end

class Asset
  belongs_to :order_line_item
  belongs_to :product
end

class Product
  has_many :assets
end

class Vendor
  has_many :orders
end

这个看似复杂的模型允许根据供应商的报价将购买“建议”转换为一个或多个实际订单，并且当元素到达时，它们会被赋予 Assets 标签。然后， Assets 本身可以链接回订单和供应商，以便稍后提供支持。

这是我的 OrderLineItem 规范，我有一个相当简洁的设置:

describe '#requires_tag?' do

  let(:product)              { FactoryGirl.create :product, requires_tag: false }
  let(:purchase_requisition) { FactoryGirl.create :purchase_requisition }
  let(:line_item)            { FactoryGirl.create :line_item, 
                                 purchase_requisition: purchase_requisition, 
                                 product: product }
  let(:quote)                { FactoryGirl.create :quote, 
                                 line_item: line_item, unit_price: 0 }

  subject { FactoryGirl.build :order_line_item, quote: quote }

  context 'when neither product nor price require a tag' do
    its(:requires_tag?) { should be_false }
  end

  context 'when product requires a tag' do
    let(:product) { FactoryGirl.create :product, requires_tag: true }
    its(:requires_tag?) { should be_true }
  end

end

我真的需要无数的 let 语句，还是有更好的方法来构建 OrderLineItem 并控制它所依赖的 Product 的属性？

Answer 1

我不同意 cpuguy。我同意得墨忒耳定律是一件好事，但由于关系数据库与存储的分层数据之间的阻抗不匹配，您的对象图似乎只是违反了它。

如果这里有可以重构的东西，它可能是你的模型结构，或者你的模型存储机制。您遇到的 Demeter 问题是您正在使用关系系统为分层数据模型建模这一事实的症状。考虑是否所有的订单信息都只是在一个大散列中。我不认为你会感到同样程度的痛苦。唯一的替代方法是尝试将其中一些字段复制到您使用它们的位置。

我实际上认为您的规范非常好，因为:a) 它们是行为性的，测试听起来像是不太可能改变的业务功能的离散元素b) 如果您的规范需要模拟内部结构，那么规范在重构方面就变得毫无用处，因为您的期望必须随着它们而改变。

主要问题在于如何构建测试环境。您可以将它们抽象到更高级别的工厂中，但要小心不要最终隐藏使您的规范独一无二的东西。不过，您在这方面也做得很好。我可能建议的一个建议是在每个上下文中创建一个 let(:requires_tag) ，一个为真，一个为假。然后将其他所有内容保留在您的设置中。这样一来，每个上下文与主要设置的区别就很清楚了，这可能需要更长的时间才能理解。

除此之外，如果有更好的方法，我还没有找到。