单元测试，使用Moq的工作单元和通用仓储模式框架

Question

我正在对使用Moq的工作单元和通用存储库的服务进行单元测试。问题是，在调试模式下运行测试时，服务类中的_subsiteRepository始终为null。

我正在模拟的服务类的设置。

private readonly IRepository<Subsite> _subsiteRepository;

public PlatformService(IUnitOfWork<PlatformContext> unitOfWork)
{
    _subsiteRepository = unitOfWork.GetRepository<Subsite>();
}

我正在测试的这个类中的方法。问题是_subsiteRepository始终为null。该方法做了更多的事情，但这是相关的部分。

public async Task<IEnumerable<Subsite>> GetSubsites()
{
    // Get Subsites
    var subsites = await _subsiteRepository
        .GetAll()
        .ToListAsync();
}

最后，这是我正在运行的测试：

private readonly Mock<IRepository<Subsite>> _subsiteRepository;
private readonly Mock<IUnitOfWork<PlatformContext>> _unitOfWork;
private readonly PlatformService _platformService;

_subsiteRepository = new Mock<IRepository<Subsite>>();
_unitOfWork = new Mock<IUnitOfWork<PlatformContext>>();
_platformService = new PlatformService(_unitOfWork.Object);

// Arrange
var fakeSubsites = new List<Subsite>
{
    new Subsite {IDSubsite = new Guid(), Title = "Subsite One"}
}.AsQueryable();

_unitOfWork.Setup(x => x.GetRepository<Subsite>()).Returns(_subsiteRepository.Object);
_unitOfWork.Setup(x => x.GetRepository<Subsite>().GetAll()).Returns(fakeSubsites);

// Act
var subsites = await _platformService.GetSubsites(null, null);

// Assert
Assert.NotNull(subsites);

- Three Value Logic

---

在创建PlatformService之后，您设置了unitOfWork.GetRepository。因此，在PlatformService的构造函数中，您实际上永远不会得到subsiteRepository.Object。我建议将PlatformService的构造放在需要调用对象之前，即在Act之前。 - Stephen Ross

不是很相关，但你为什么不直接使用 Mock<ISubsite> 呢？我猜存储库有一个接口。 - MZawg

Answer 1

将 _platformService 的创建移动到排列步骤之后。因为在设置 unitOfWork 模拟之前，调用了 PlatformService 构造函数。

_subsiteRepository = new Mock<IRepository<Subsite>>();
_unitOfWork = new Mock<IUnitOfWork<PlatformContext>>();

// Arrange
var fakeSubsites = new List<Subsite>
{
    new Subsite {IDSubsite = new Guid(), Title = "Subsite One"}
}.AsQueryable();

_unitOfWork.Setup(x => x.GetRepository<Subsite>()).Returns(_subsiteRepository.Object);
_unitOfWork.Setup(x => x.GetRepository<Subsite>().GetAll()).Returns(fakeSubsites);

// Act
_platformService = new PlatformService(_unitOfWork.Object);
var subsites = await _platformService.GetSubsites(null, null);

// Assert
Assert.NotNull(subsites);

- Valentin

---

我完全同意你回答的第一部分，但是我不同意你的第二部分。是的，他们正在尝试同时测试两个不同的部分，但单元测试不应直接影响您的代码，因此您不应该需要添加业务逻辑仅供测试使用。我建议他们考虑一种方法，即提供一个“子站点”而不是使用“IUnitOfWork<T>”作为服务定位器反模式，从而涵盖您的第二点，即消除对模拟“IUnitOfWork<T>”的依赖。 - Stephen Ross

@StephenRoss 我删除了第二部分，因为我认为它是基于个人观点的，而且没有回答用户提出的问题。感谢您的评论。 - Valentin

非常感谢！我对moq的工作原理有了根本性的误解。感谢您对拆分单元测试的评论。我一定会研究如何清晰地分离它并逐个测试。 - Three Value Logic