构造函数使用模拟对象，如何单独测试方法？

Question

我有一个如下所示的类：

class MyClass {

  private IDependency dep;

  public MyClass(IDependency dep) {
    this.dep = dep;
    this.dep.Reset();
  }

  public void Clear() {
    this.dep.Reset();
  }
}

如何测试 Reset 方法是否在 Clear 方法中正确调用，同时忽略构造函数的作用？

我的 Moq 测试如下所示：

MockRepository mocks = new MockRepository(MockBehavior.Default);
var dep = mocks.Create<IDependency>();

dep.Setup(s => s.Reset());

MyClass myclass = new MyClass(dep.Object);
myclass.Clear():

state.Verify(s => s.Reset(), Times.Exactly(1));

它失败了，因为 Reset 已被调用两次（一次在构造函数中，一次在 Clear 方法中）。

Answer 1

我希望有更好的方法来做到这一点，但是模拟将记录对 Reset 的所有调用，因此使用标准 Verify 调用将始终返回 2。以下维护单独的柜台，不太优雅。如果有一种内置的方式可以用 Moq 来做到这一点，我很想知道。

int clearResetCount = 0;

Mock<IDependency> dep = new Mock<IDependency>();

MyClass myclass = new MyClass(dep.Object);

dep.Setup(s => s.Reset()).Callback(() => clearResetCount++);

Assert.AreEqual(0, clearResetCount, "Expected clear reset count - before.");

myclass.Clear();

Assert.AreEqual(1, clearResetCount, "Expected clear reset count - after.");

Answer 2

正如其他人所建议的，您可以推出自己的模拟，也可以对依赖项设置一些期望。

例如，您可以验证您的方法是否被调用：

var mock = new Mock<IDependency>();
var subject = new MyClass(mock.Object);

subject.Clear();

mock.Verify( dep => dep.Reset(), Times.AtMost(2));

但是值得指出的是，可以工作构造函数内有一种已知的代码气味，当您尝试编写测试时，这种气味会加剧。

事实上，您的构造函数需要在依赖项上调用此方法，这表明该对象知道有关依赖项的实现细节的太多信息。这违反了开闭原则，并且使您无法在初始化时调用 Reset 方法。

还要考虑到任何使用 MyClass 具体对象作为虚拟参数的类或测试都需要初始化 Mock，否则您将得到 NullReferenceException。这会增加编写测试的相当大的开销，并增加一定程度的脆弱性，相当于测试中的长期维护和漏报。解决这个问题的唯一方法是让一切都成为一个界面，虽然有效，但也不是最好的长期策略。

根据 http://googletesting.blogspot.com/2009/07/separation-anxiety .html，使用工厂会减少一些这种耦合，并使您能够更好地重用该对象。

Answer 3

我遇到了同样的问题。

执行以下操作以使模拟仅记录 Clear 方法的行为：

MockRepository mocks = new MockRepository(MockBehavior.Default);
var dep = mocks.Create<IDependency>();

MyClass myclass = new MyClass(dep.Object);

// setting up the mock just before calling the method under test
// will ignore any prior call to IDependency.Reset
int resetTimes = 0;
dep.Setup(s => s.Reset()).Callback(() => resetTimes++);

myclass.Clear();

mocks.VerifyAll();
Assert.That(resetTimes, Is.EqualTo(1));

Answer 4

您可以编写一个间谍，而不是使用模拟对象。需要更多的编码，但测试更容易阅读。

class DependencySpy : IDependency {
    public int ResetCallCount { get; private set; }
    public void Reset() { ResetCallCount++; }
    public void ClearResetCallCount() { ResetCallCount = 0; }
}

测试可以写成

// Setup
var spy = new DependencySpy;
MyClass myclass = new MyClass(spy);
spy.ClearResetCallCount();
// Exercise
myclass.Clear();
// Validate
Assert.AreEqual(1, spy.ResetCallCount);

Answer 5

您可以使用反射将私有字段 dep 设置为您的模拟对象。然后只需调用 Clear 方法并测试依赖项调用即可。