ruby-on-rails - 单元测试隔离 : 20 lines of test for 1 line of code?

Question

据我了解，在单元测试中methods should be isolated from their dependencies , 这样他们就不会受到环境变化的影响。

尽管如此，删除所有依赖项让我觉得我是在测试实现而不是行为。

换句话说，通过隔离依赖关系，我将测试与实现细节相结合。因此，任何 code refactoring会导致测试失败，即使行为(期望的结果)没有改变。

例如，这是一个简单的 (Ruby) 方法:

  def send_request
    update_attributes(response.page_params) if active?
  end

这是我对这一行代码的两个独立测试:

  let(:page) { Page.new }

  describe '#send_request' do
    context 'when a page is active' do
      it 'updates page with the response parameters' do
        page.active = true
        response = double('response')
        page_params = double('page_params')
        response.stub(:page_params).and_return(page_params)
        page.stub(:response).and_return(response)
        page.stub(:update_attributes).and_return(nil)
        page.should_receive(:update_attributes).with(page_params)
        page.send_request
      end
    end
    context 'when a page is inactive' do
      it 'does NOT send a request' do
        page.active = false
        page.should_not_receive(:response)
        page.send_request
      end
    end
  end

测试通过了，但我发现了一些严重的问题:

• 如果稍后我决定使用除 update_attributes() 之外的任何其他方法将更改保存到数据库中，我的测试将失败，即使数据将按预期保存
• 如果 response.page_params 的实现发生变化，我的软件将在生产环境中失败，但测试仍会通过

我一定是做错了什么。

单元测试的正确写法是什么？

Answer 1

正如 AlistairIsrael 所说，我不认为您在这里完全偏离了目标。

您可以进行一些优化以使其更加简洁。一个好的测试应该清楚地表明您对代码的期望。

let(:page) { Page.new }

describe '#send_request' do
  context 'when a page is active' do
    it 'updates page with the response parameters' do
      page.active = true
      response = double('response',
        :page_params => page_params = mock('page_params')
      )

      # not needed as .should_receive creates a nil stub by default.
      # page.stub(:update_attributes).and_return(nil)
      page.should_receive(:update_attributes).with(page_params)
      page.send_request
    end
  end
  context 'when a page is inactive' do
    it 'does NOT send a request' do
      page.active = false

      subject.should_not_receive(:update_attributes)
      page.send_request
    end
  end
end

从上面的一些变化你可以看出 rspec 的双重助手非常非常强大，你可以构造复杂的对象并使用一些赋值的东西你可以访问链中最后计算的方法。

我对负面情况做了一个假设，但你应该明白了。测试 update_attributes 的方法调用可能更容易，也更清晰，因为您知道 page_params 如果处于事件状态，将永远不会被调用？不满足条件。

HTH