4

所以我有两个规格，我认为它们正在测试相同的东西，但一个失败而另一个通过。我正在开发一个具有重复计划的应用程序。如果用户创建重复的行程，它将继续并为指定的每一天创建新行程。这是第一个失败的测试：

it "makes future trips" do
  expect{FactoryGirl.create(:recurring_transportation_trip)}.to change(Trip, :count).by(4)
end

recurring_transportation_trip 创建一个行程，它将通过 after_save 回调进行三个未来行程。此测试失败并出现错误“计数应该已更改 4，但已更改 1”。

这是另一个通过的测试：

it "makes future trips" do
  count = Trip.count
  FactoryGirl.create(:recurring_transportation_trip)
  Trip.count == count + 4
end

表明存在正确的功能。

第一个测试当然更具可读性，但实际上并没有测试我认为它做了什么。任何人都可以提供并解释原因吗？

- - - -编辑 - - - -

按要求添加工厂代码：

FactoryGirl.define do
  factory :recurring_transportation_trip, :class => :trip do
    collection_time "09:00"
    estimated_duration "60"
    status "Confirmed"
    mileage "30"
    association :collection, :factory => :location
    association :destination, :factory => :location
    association :call, :factory => :recurring_call
  end
end

对于 recurring_call

FactoryGirl.define do
  factory :recurring_call, :class => "Call" do
    recurring true
    recurring_start_date Date.today
    recurring_end_date Date.today + 1.week
    recurring_config [1, 3, 5]
  end
end

-------EDIT2--------

事实证明Trip.count == count + 4实际上并没有断言任何东西，并且测试Trip.count.should == count + 4确实失败了。感谢@BenediktDeicke 指出这一点。

--------EDIT3--------

最后这是我的应用程序代码中的一个错误，我应该从一开始就信任原始测试。感谢所有看过的人。@boulder 和 @BenediktDeicke 感谢您指出 edit2 中提到的缺少断言。

4

2 回答 2

1

您的第二个测试实际上没有测试任何东西，因为它没有定义任何期望。

该行 Trip.count == count + 4 只是一个计算结果为假的表达式。

你想要做的是：

Trip.count.should == count + 4

无论如何，第一个测试是您应该信任的；它告诉您您的应用程序代码中存在需要调查的错误。

于 2013-02-05T20:17:48.767 回答
0

by 链匹配器是预期应该改变的量。

expect{FactoryGirl.create(:recurring_transportation_trip)}.to change(Trip, :count).by(1)

这是文档。

于 2013-02-05T18:58:35.333 回答