2

这是贫血的域模型： 

public partial class Person
{
    public virtual int PersonId { get; internal protected set; }
    public virtual string Title { get; internal protected set; } 
    public virtual string FirstName { get; internal protected set; } 
    public virtual string MiddleName { get; internal protected set; } 
    public virtual string LastName { get; internal protected set; } 
}

这是它的行为：

public static class Services
{

    public static void UpdatePerson(Person p, string firstName, string lastName)
    {
        // validate  firstname and lastname
        // if there's a curse word, throw an exception


        // if valid, continue

        p.FirstName = firstName;
        p.LastName = lastName;


        p.ModifiedDate = DateTime.Now;
    }

}

它几乎是可测试的：

[TestMethod]

public void Is_Person_ModifiedDate_If_Updated()
{
    // Arrange
    var p = new Mock<Person>();

    // Act 
    Services.UpdatePerson(p.Object, "John", "Lennon");

    // Assert            
    p.VerifySet(x => x.ModifiedDate = It.IsAny<DateTime>());
}

但是，我想练习富域模型，其中数据和行为在逻辑上更具凝聚力。所以上面的代码现在转换为：

public partial class Person
{
    public virtual int PersonId { get; internal protected set; }
    public virtual string Title { get; internal protected set; }
    public virtual string FirstName { get; internal protected set; } 
    public virtual string MiddleName { get; internal protected set; }
    public virtual string LastName { get; internal protected set; } 

    public virtual void UpdatePerson(string firstName, string lastName)
    {
        // validate  firstname and lastname
        // if there's a curse word, throw an exception


        // if valid, continue


        this.FirstName = firstName;
        this.LastName = lastName;

        this.ModifiedDate = DateTime.Now;
    }           
}

但是我遇到了测试问题：

[TestMethod]
public void Is_Person_ModifiedDate_If_Updated()
{
    // Arrange
    var p = new Mock<Person>();

    // Act 
    p.Object.UpdatePerson("John", "Lennon");

    // Assert            
    p.VerifySet(x => x.ModifiedDate = It.IsAny<DateTime>());
}

单元测试错误：

Result Message: 

Test method Is_Person_ModifiedDate_If_Updated threw exception: 
Moq.MockException: 
Expected invocation on the mock at least once, but was never performed: x => x.ModifiedDate = It.IsAny<DateTime>()
No setups configured.

Performed invocations:
Person.UpdatePerson("John", "Lennon")
Result StackTrace:  
at Moq.Mock.ThrowVerifyException(MethodCall expected, IEnumerable`1 setups, IEnumerable`1 actualCalls, Expression expression, Times times, Int32 callCount)
   at Moq.Mock.VerifyCalls(Interceptor targetInterceptor, MethodCall expected, Expression expression, Times times)
   at Moq.Mock.VerifySet[T](Mock`1 mock, Action`1 setterExpression, Times times, String failMessage)
   at Moq.Mock`1.VerifySet(Action`1 setterExpression)
   at Is_Person_ModifiedDate_If_Updated()

看到直接从被模拟的对象调用方法，被模拟的对象就无法检测到是否调用了它的任何属性或方法。注意到这一点后，对富域模型进行单元测试的正确方法是什么？

4

2 回答 2

5

首先，不要模拟您正在测试的值对象或类。此外，您没有验证是否向人员提供了正确的修改日期。您检查是否分配了某个日期。但这并不能证明您的代码按预期工作。为了测试这样的代码，你应该模拟DateTime.Now 返回的当前日期，或者创建一些抽象，这将提供当前的服务时间。你的第一个测试应该是这样的（我在这里使用了Fluent Assertions和 NUnit）：

[Test]
public void Should_Update_Person_When_Name_Is_Correct()
{
    // Arrange
    var p = new Person(); // person is a real class
    var timeProviderMock = new Mock<ITimeProvider>();
    var time = DateTime.Now;
    timeProviderMock.Setup(tp => tp.GetCurrentTime()).Returns(time);
    Services.TimeProvider = timeProviderMock.Object;
    // Act 
    Services.UpdatePerson(p, "John", "Lennon");
    // Assert
    p.FirstName.Should().Be("John");
    p.LastName.Should().Be("Lennon");
    p.ModifiedDate.Should().Be(time); // verify that correct date was set
    timeProviderMock.VerifyAll();
}

时间提供者是一个简单的抽象：

public interface ITimeProvider
{
    DateTime GetCurrentTime();
}

我会使用单例服务而不是静态类，因为静态类总是存在问题 - 高耦合，没有抽象，难以对依赖类进行单元测试。但是你可以通过属性注入时间提供者：

public static class Services
{
    public static ITimeProvider TimeProvider { get; set; }

    public static void UpdatePerson(Person p, string firstName, string lastName)
    {
        p.FirstName = firstName;
        p.LastName = lastName;
        p.ModifiedDate = TimeProvider.GetCurrentTime();
    }
}

这与您的第二次测试有关。不要模拟您正在测试的对象。您应该验证您的应用程序将使用的真实代码，而不是测试一些仅用于测试的模拟。使用到达域模型进行测试将如下所示：

[Test]
public void Should_Update_Person_When_Name_Is_Correct()
{
    // Arrange        
    var timeProviderMock = new Mock<ITimeProvider>();
    var time = DateTime.Now;
    timeProviderMock.Setup(tp => tp.GetCurrentTime()).Returns(time);
    var p = new Person(timeProviderMock.Object); // person is a real class
    // Act 
    p.Update("John", "Lennon");
    // Assert
    p.FirstName.Should().Be("John");
    p.LastName.Should().Be("Lennon");
    p.ModifiedDate.Should().Be(time); // verify that correct date was set
    timeProviderMock.VerifyAll();
}
于 2013-12-31T13:40:01.130 回答
2

你的来电：

p.Object.UpdatePerson("John", "Lennon");

在你的模拟上调用一个公共virtual方法。UpdatePerson您的模拟具有行为Loose（也称为Default），而您没有Setup该虚拟方法。

在这种情况下，Moq 的行为是在其实现（覆盖）中什么都不UpdatePerson做。

有几种方法可以改变这一点。

  • 您可以从方法中删除virtual关键字。UpdatePerson然后起订量不会（也不能）覆盖其行为。
  • 或者，您实际上可以Setup在调用之前使用 Moq 的虚拟方法。（在这种情况下没有用，因为它会覆盖您实际要测试的方法。）
  • 或者你可以说p.CallBase = true; 在调用该方法之前。其工作方式如下（具有Loose行为）：如果调用未设置的virtual成员，则 Moq 将调用基类的实现。

这解释了你所看到的。我同意谢尔盖·别列佐夫斯基在回答中给出的建议。

于 2014-01-01T10:58:49.897 回答