c# - 如果发生对静态记录器的调用，如何进行单元测试

Question

考虑一个带有 try catch 的方法，如下所示:

try
{
    person = this.personRepo.GetPerson(name);
}
catch (PersonException)
{
    LogHelper.LogDebug("PersonService", "No Person found!");
}

在单元测试中，personRepo 是用 FakeItEasy 伪造的:

A.CallTo(() => this.personRepository.GetPerson(personName)).Throws(new PersonException("Person not found"));

问题:

如何检查是否调用了静态记录器？

Answer 1

这里的假设是 personRepo 被注入(inject)到被测对象中。因此，我还假设您正在使用 DI。

当前显示的代码与静态实现问题紧密耦合，这使得很难单独测试主题。

为了使主题更灵活、更易于维护和单独测试，需要重构主题以依赖于抽象而不是具体化。

将静态 LogHelper 封装在一个抽象之后。

public interface ILogger {
    void LogDebug(string category, string msg);
    //...other members
}

这将包装所需的行为/功能

public class LogHelperWrapper : ILogger {
    public void LogDebug(string source, string msg) {
        LogHelper.LogDebug(source, msg);
    }

    //...other members
}

主题将使用抽象，这将通过构造函数注入(inject)重构以遵循显式依赖原则。

private ILogger log; //set via constructor injection
try {
    person = this.personRepo.GetPerson(name);
} catch (PersonException) {
    this.log.LogDebug("PersonService", "No Person found!");
}

这将允许接受测试的受试者在隔离测试时接受替换。

//Arrange
var logger = A.Fake<ILogger>();
//...other arrangements

//Act
//...exercise test

//Assertion
A.CallTo(() => logger.LogDebug("PersonService", "No Person found!"))
.MustHaveHappened(Repeated.Exactly.Once);
//...other assertions.