5

我在 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

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

这是我的 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 的属性？

4

2 回答 2

3

我不得不不同意 w/cpuguy。我同意 demeter 定律是一件好事，但你的对象图似乎只是因为你的关系数据库和你存储的分层数据之间的阻抗不匹配而违反了它。

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

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

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

除此之外，如果有更好的方法来做到这一点，我还没有找到。

于 2013-08-27T12:49:37.413 回答
1

我想说尝试重构它，以便您测试一件事。如果您这样做，则不需要创建所有这些对象（反过来会减慢您的测试速度）。

如果您没有违反 LoD，https://www.google.com/search? q=law+of+demeter&ie=UTF-8&oe=UTF-8&hl=en&client=safari#itp=open0 ，那么这会变得容易得多。

您应该能够根据需要存根您在相关对象上调用的方法，而不是创建实际的关系。

于 2013-05-11T02:11:21.637 回答