7

我有一个关于如何设计适合单元测试的应用程序的问题。

我正在尝试实施 SRP（单一职责原则），据我了解，这涉及将大多数功能拆分为单独的专用类，以使代码更有条理。例如，我有这个特定的场景。

一个类RecurringProfile，它有一个方法.ActivateProfile()。此方法的作用是将状态标记为已激活，并为下一个到期日创建下一个（第一次）定期付款。例如，我将拆分功能以在单独的类中创建下一次定期付款RecurringProfileNextPaymentCreator。我的即时想法是让这个类'RecurringProfile'在它的构造函数中作为参数：

RecurringProfileNextPaymentCreator(IRecurringProfile profile)

但是，我认为这对于单元测试来说是有问题的。我想创建一个测试ActivateProfile功能的单元测试。此方法将通过依赖注入 (Ninject) 获取一个实例IRecurringProfileNextPaymentCreator，并调用该方法.CreateNextPayment。

我创建单元测试的想法是创建一个 , 的模拟IRecurringProfileNextPaymentCreator并替换它，以便我可以验证.ActivateProfile()实际调用的方法。但是，由于构造函数参数，这不适合作为 NInject 的默认构造函数。必须为这种情况创建一个自定义的 NInject 提供程序（我可以在整个解决方案中拥有许多这样的类）会有点矫枉过正。

任何想法/最佳实践将如何去做？

-- 下面是关于上述示例的示例代码：（请注意，代码是手写的，在语法上并非 100% 正确）

public class RecurringProfile
{
    public void ActivateProfile()
    {
        this.Status = Enums.ProfileStatus.Activated;
        //now it should create the first recurring payment
        IRecurringProfileNextPaymentCreator creator = NInject.Get<IRecurringProfileNextPaymentCreator>();
        creator.CreateNextPayment(this); //this is what I'm having an issue about 
    }
}

和一个示例单元测试：

public void TestActivateProfile()
{   
    var mockPaymentCreator = new Mock<IRecurringProfileNextPaymentCreator>();
    NInject.Bind<IRecurringProfileNextPaymentCreator>(mockPaymentCreator.Object);

    RecurringProfile profile = new RecurringProfile();
    profile.ActivateProfile();
    Assert.That(profile.Status == Enums.ProfileStatus.Activated);
    mockPayment.Verify(x => x.CreateNextPayment(), Times.Once());

}

转到示例代码，我的问题是，将 RecurringProfile 作为参数传递给creator.CreateNextPayment()方法是否是一种好习惯，或者RecurringProfile在获取IRecurringProfileNextPaymentCreator，考虑到IRecurringProfileNextPaymentCreator将始终对 anIRecurringProfile进行操作以创建下一笔付款。希望这能让问题更清楚一些。

4

2 回答 2

6

在此类单元测试期间，您不应该使用您的 DI 容器（Ninject）。在更新被测类时，您将手动注入模拟对象。然后验证调用是在模拟上进行的。

于 2012-10-16T07:22:58.200 回答
4

因为你没有显示任何代码我猜你想做这样的事情

public class RecurringProfile
{
  private readonly DateTime _dueDate;
  private readonly TimeSpan _interval;
  public RecurringProfile(DateTime dueDate, TimeSpan interval)
  {
    _dueDate = dueDate;
    _interval = interval;
  }
  public bool IsActive { get; private set; }
  public DateTime DueDate
  {
    get { return _dueDate; }
  }
  public TimeSpan Interval
  {
    get { return _interval; }
  }
  public RecurringProfile ActivateProfile()
  {
    this.IsActive = true;
    return new RecurringProfile(this.DueDate + this.Interval, this.Interval);
  }
}

这还不够简单吗？

更新

不要滥用 DI 容器作为 ServiceLocator。您将支付创建者作为 ctor 参数注入的想法是正确的方法。ServiceLocator 被认为是现代应用程序架构中的反模式。类似下面的代码应该可以正常工作。

[TestClass]
public class UnitTest1
{
  [TestMethod]
  public void TestMethod1()
  {
    var mock = new Mock<INextPaymentCreator>();
    DateTime dt = DateTime.Now;
    var current = new RecurringProfile(mock.Object, dt, TimeSpan.FromDays(30));
    current.ActivateProfile();
    mock.Verify(c => c.CreateNextPayment(current), Times.Once());
  }
}
public class RecurringProfile
{
  private readonly INextPaymentCreator _creator;
  private readonly DateTime _dueDate;
  private readonly TimeSpan _interval;
  public RecurringProfile(INextPaymentCreator creator, DateTime dueDate, TimeSpan interval)
  {
    _creator = creator;
    _dueDate = dueDate;
    _interval = interval;
  }
  public bool IsActive { get; private set; }
  public DateTime DueDate
  {
    get { return _dueDate; }
  }
  public TimeSpan Interval
  {
    get { return _interval; }
  }
  public RecurringProfile ActivateProfile()
  {
    this.IsActive = true;
    var next = this._creator.CreateNextPayment(this);
    return next;
  }
}

public interface INextPaymentCreator
{
  RecurringProfile CreateNextPayment(RecurringProfile current);
}
于 2012-10-16T11:53:55.847 回答