0

我正在将自己从 FactoryGirl 中解放出来（至少在 lib 文件夹中）。所以，我开始写一些奇怪的东西，比如“mock”和“stub”。有人可以帮助新手吗？

我有这个模块

module LogWorker
   extend self
   def check_todo_on_log(log, done)
    if done == "1"
      log.todo.completed = true
      log.todo.save!
    elsif done.nil?
      log.todo.completed = false
      log.todo.save!
    end
  end 
end

log并且todo是具有todo :has_many logs关联的rails模型。但这在使用 stub 和 mock 时真的不重要，对吧？

我尝试了很多事情，但是当我将模拟传递给方法时，什么也没有发生，

describe LogWorker do
  it 'should check_todo_on_log'do
    todo = mock("todo")
    log = mock("log")
    log.stub!(:todo).and_return(todo)
    todo.stub!(:completed).and_return(false)
    LogWorker.check_todo_on_log(log,1)
    log.todo.completed.should eq true
  end
end

Failures:

  1) LogWorker should check_todo_on_log
     Failure/Error: log.todo.completed.should eq true

       expected: true
            got: false

       (compared using ==

我真的很想看到一些规范来测试带有存根和/或模拟的 LogWorker.check_todo_on_log 方法。

4

1 回答 1

1

首先，你的check_todo_on_log方法很糟糕。永远不要使用字符串作为选项，尤其是当字符串为“1”时。此外，如果您通过“2”，则不会发生任何事情。我会假设它只是一个部分方法，并且您的代码并不是那样的：P

查看您的代码，您有三个主要问题。首先，您调用LogWorker.check_todo_on_log(log,1). 这不会做任何事情，因为您的方法仅在第二个参数是字符串"1"或 nil 时才起作用。其次，你存根todo.completed所以它总是返回 false: todo.stub!(:completed).and_return(false)。然后你测试它是否是真的。显然，这将失败。最后，您不要模拟该save!方法。我不知道代码实际上是如何为您运行的（它对我不起作用）。

以下是我将如何编写您的规范（请注意，他们正在测试奇怪的行为，因为该check_todo_on_log方法也很奇怪）。

首先，有一种更简单的方法可以将模拟方法添加到模拟对象。您可以将键和值传递给mock方法，它们将自动创建。

接下来，我将模拟物放入let块中。这使得它们可以轻松地为每个测试重新创建。最后，我为函数的每个可能行为添加了一个测试。

# you won't need these two lines, they just let the code be run by itself
# without a rails app behind it. This is one of the powers of mocks, 
# the Todo and Log classes aren't even defined anywhere, yet I can 
# still test the `LogWorker` class!
require 'rspec'
require 'rspec/mocks/standalone'

module LogWorker
   extend self
   def check_todo_on_log(log, done)
    if done == "1"
      log.todo.completed = true
      log.todo.save!
    elsif done.nil?
      log.todo.completed = false
      log.todo.save!
    end
  end
end

describe LogWorker do
  let(:todo) { mock("Todo", save!: true) }
  let(:log) { mock("Log", todo: todo) }
  describe :check_todo_on_log do
    it 'checks todo when done is "1"'do
      todo.should_receive(:completed=).with(true)
      LogWorker.check_todo_on_log(log,"1")
    end
    it 'unchecks todo when done is nil'do
      todo.should_receive(:completed=).with(false)
      LogWorker.check_todo_on_log(log,nil)
    end

    it "doesn't do anything when done is not '1' or nil" do
      todo.should_not_receive(:completed=)
      LogWorker.check_todo_on_log(log,3)
    end
  end
end

注意我是如何使用基于行为的测试的？我不是在测试模拟上的属性是否具有值，而是在检查是否调用了适当的方法。这是正确使用模拟的关键。

于 2013-06-07T17:45:26.570 回答