如何避免使用allow_any_instance_of？

Question

想象一下,我们有以下代码:

class A
  def create_server
    options = {
      name: NameBuilder.new.build_name
    }
    do_some_operations(options)
  end
end

为了测试这样的方法,我曾经使用过allow_any_instance_of:

it 'does operations' do
  allow_any_instance_of(NameBuilder).to receive(:build_name)
  # test body
end

但是文档建议我们不要使用它,原因有几个.怎么回避allow_any_instance_of？我只有一个解决方案:

class A
  def create_server
    options = {
      name: builder.build_name
    }
    do_some_operations
  end

  private

  def builder
    NameBuilder.new
  end
end

但是使用这样的方法代码很快就会变得几乎无用的方法(特别是当你在所描述的类中主动使用不同对象的组合时).

Answer 1

根据乌兹别克斯坦的回答（我同意），在没有依赖项注入的情况下，您也可以考虑取消对的调用，NameBuilder.new以便您可以直接控制NameBuilder被测实例：

class NameBuilder
  def build_name
    # builds name...
  end
end

class A
  def create_server
    options = {
      name: NameBuilder.new.build_name
    }
    do_some_operations(options)
  end

  def do_some_operations(options)
    # does some operations
  end
end

RSpec.describe A do
  let(:a) { described_class.new }

  describe '#create_server' do
    let(:name_builder) { instance_double(NameBuilder) }

    before do
      allow(NameBuilder).to receive(:new).and_return(name_builder)
    end

    it 'does operations' do
      # the first expectation isn't really part of what you seem
      # to want to test, but it shows that this way of testing can work
      expect(name_builder).to receive(:build_name)
      expect(a).to receive(:do_some_operations)
      a.create_server
    end
  end
end

Answer 2

如果很难测试，说明你的类设计有问题。在您的情况下，当您在类中的特定类上测试特定方法调用时，您正在测试如下：

allow_any_instance_of(NameBuilder).to receive(:build_name)

您的测试确切地知道该方法的内部实现方式。您的类应该封装逻辑并隐藏它。你所做的恰恰相反。

您不应该测试任何内部方法逻辑。只需测试行为即可。给出输入并测试输出的正确性。

如果您确实想在NameBuilder类上测试该方法调用，请注入该依赖项并使您的类更具可测试性。这也遵循OOP原则。

class A
  def create_server(builder)
    do_some_operations(name: builder.build_name)
  end
end