使用 new 运算符重构代码以使其更具可测试性

Question

我有以下代码并尝试对其进行单元测试： <代码>
公共重写 IRenderable GetRenderable()
{
var val = SomeCalculationUsingClassMemberVariables();
返回新的 EquationRenderable(val);
} 。

看来我想在这里使用一个工厂，这样我就可以将 IRenderable 的创建与此类分开 问题是我有许多这样的类，它们创建了以不同方式构造的不同 IRenderable，因此我需要为每个类实现一个新的工厂方法。解决这个问题的最佳方法是什么？

Answer 1

好问题。

首先，由于 DI 容器没有以“正确”的方式使用或者设计可以改进，所以在任何地方都想使用 AbstractFactories 的感觉有点难闻。

但有时我也遇到过这个问题。我看到以下内容：

1. 使用 AbstractFactory/Factory 并注入它。对于 C#，您必须具有可以传递委托的优势，充当创建实例的接口。
2. 'new' 就可以了，只需测试'new' 的输出即可。
3. 在提取的方法中存根“新”的调用（hacky！！）

注入已经被提及，所以我不会重复。我更喜欢 Java，所以请原谅一些语法错误。

测试“新”的输出

如果“新”创建的实例是域对象而不是服务，我经常使用它。因为它是直接在方法中返回的，所以我可以用我的测试来测试直接输出。

Prod-Code:
...
public override IRenderable GetRenderable()
{
  var val = SomeCalculationUsingClassMemberVariables();
  return new EquationRenderable(val);
} 

Test Case:
...
[Test]
public void test_new()
{
  SUT sut = ...;
  IRenderable r = sut.GetRenderable();
  assertTrue(r instanceof EquationRenderable);
}

存根“new”本身的调用

只有当您以某种方式将其作为返回值时，才可以测试上面的直接输出。如果代码的“副作用”是间接输出（您无法通过返回值直接感知），事情会变得更加复杂。如果是这样，我经常提取新创造的方法，然后在我的测试中控制它。这很糟糕，我更多地使用它来保证测试的安全，并在以后开始更多的重构（DI 和工厂）。我有时会在遗留代码中执行此操作，其中直接使用“新”创建服务，并且在没有测试的情况下重构为 DI 风险太大。

Prod-Code:
...
public override IRenderable GetRenderable()
{
  var val = SomeCalculationUsingClassMemberVariables();
  return createEquationRenderable();
} 

public IRenderable createEquationRenderable() 
{
   return new EquationRenderable(val);
}


Test Case:
...

class Stubbed : SUT 
{
   boolean called = false;

   public override EquationRenderable createEquationRenderable() 
   {
      called=true;
      return MyMock();
   }
}

[Test]
public void test_new()
{
  Stubbed sut = new Stubbed();
  sut.GetRenderable();
  assertTrue(sut.called);
  // do further stuff on MyMock
}

我知道，这个例子有点矫枉过正，有点毫无意义，它只是为了描述这个想法。我确信上面的内容可以通过 C# 的模拟框架来简化。无论如何，在这里测试返回值直接输出是更简单、更好的方法。

也许你有更详细的例子？

Answer 2

根据具体 IRenderable 构造函数的一致性，您可以使用以下模式进行工厂创建

public IRenderable CreateInstance<T>(object calculation) where T : IRenderable 
{
   Activator.CreateInstance<T>(new[] { calculation });
}

，或者如果您有许多不同的构造函数，您可以使用 params 关键字传递任意数量的参数

public IRenderable CreateInstance<T>(params object[] args) where T : IRenderable 
{
   Activator.CreateInstance<T>(args);
}

以便能够对参数进行某种运行时检查您在调用 Activator.CreateInstance 之前使用此代码

var types = args.Select(o => o.GetType()).ToArray();

var c = typeof(T).GetConstructor(types);
if (c == null)
{
   throw new InvalidOperationException("No matched constructor")
}

Answer 3

更好的方法可能是简单地按原样对代码进行单元测试，而不是重构它。从技术上讲，这可以通过使用合适的模拟工具来完成，例如 TypeMock Isolator 或 Microsoft Moles（还有第三个，我现在不记得了）。