Moq 单元工作单位

Question

我是单元测试的新手，我想为我的搜索功能创建一个测试。我的服务层看起来像这样：

public class EmployeeService: BaseService, IEmployeeService
{
    public EmployeeService(IUnitOfWork unitOfWork) : base(unitOfWork)
    {
        _employeeRepo = unitOfWork.EmployeeRepository;
    }

    public IEnumerable<Employee> Search(Employee advancedSearch, int[] divisionIds, bool showInactive, int pageSize, int? page)
    {
        return _employeeRepo.Search(advancedSearch, divisionIds, showInactive, pageSize, page);
    }
}

单元测试：

[Test]
public void SearchShouldFilterActiveEmployees()
{
    var employees = new List<Employee>
    {
        new Employee { EmployeeId = 105, FirstName = "John", LastName = "Smith", IsActive = true },
        new Employee { EmployeeId = 162, FirstName = "John", LastName = "Doe", IsActive = true },
        new Employee { EmployeeId = 3, FirstName = "Jan", LastName = "Doe", IsActive = true }
    };

    var mockUnitOfWork = new Mock<IUnitOfWork>();

    var sut = new EmployeeService(mockUnitOfWork.Object);

    var employeeSearchCriteria = new Employee
    {
        FirstName = "John"
    };

    var employeesReturned = sut.Search(employeeSearchCriteria, null, false, 25, 1);

    Assert.IsTrue(employeesReturned.Count() == 2);
}

我认为问题出在对UnitOfWork进行模拟上。我遇到了Null Reference Exception异常。如何使用moq来测试从搜索中返回正确数量的员工。

- Andrew

---

你需要模拟EmployeeRepository和Search，特别是...作为副作用，你的测试将会比你期望的添加更少的价值，因为没有真正的过滤提供数据库访问被伪造。 - Claudio Redi

你认为创建这个测试有意义吗？在我的代码库中，我已经创建了一个类似的测试，只不过“被测试系统”是“EmployeeRepository”，而不是“EmployeeService”。我真正想测试的是我的服务层是否可以调用存储库中的“Search”方法。 - Andrew

有意义的测试它，这不是最可能的故障点，因为基本上没有逻辑，但您可以测试是否使用正确的参数调用了“搜索”。创建一个设置，仅当使用正确的参数调用“搜索”时返回2个员工。如果使用不正确的参数，则“搜索”将返回null，并且您的断言将失败。 - Claudio Redi

Answer 1

为了模拟存储库，最简单的方法是为其创建一个接口，例如IEmployeeRepo。然后在EmployeeService类中，字段将是private readonly IEmployeeRepo _employeeRepo。

在测试中，您可以验证是否调用了Search。

bool searchwasCalled = false;
Mock<IEmployeeRepo> repositoryMock = new Mock<IEmployeeRepo>();
repositoryMock.Setup(r => r.Search(
    It.IsAny<Employee>(), It.IsAny<int[]>(), It.IsAny<bool>(), It.IsAny<int>(),
    It.IsAny<int?>()))
    .Callback(() => searchwasCalled = true);

var mockUnitOfWork = new Mock<IUnitOfWork>();
mockUnitOfWork.Setup(uow => uow.EmployeeRepository).Returns(repositoryMock.Object);

var sut = new EmployeeService(mockUnitOfWork.Object);

var employeeSearchCriteria = new Employee
{
    FirstName = "John"
};

sut.Search(employeeSearchCriteria, null, false, 25, 1);

Assert.IsTrue(searchwasCalled, "Search was not called.");  

- Daniel Dušek