表达<Func<T, bool> &gt; 在模拟设置中没有按预期工作

Question

这是我要测试的方法：

public async Task<List<Lesson>> GetProfessionalLessonsByTutorIdAsync(long tutorId)
{
     return await _unitOfWork.Repository<Lesson>().GetEntities(l => l.TeacherId == tutorId && l.LessonTypeId == 1)
                .AsNoTracking().ToListAsync();
}

这里的GetEntities方法如下：

public IQueryable<TEntity> GetEntities(Expression<Func<TEntity, bool>> condition = null,
        Func<IQueryable<TEntity>, IIncludableQueryable<TEntity, object>> include = null)
{
        IQueryable<TEntity> query = _dbSet;
        if (condition != null)
        {
            query = query.Where(condition);
        }

        if (include != null)
        {
            query = include(query);
        }

        return query;
}

我的测试方法：

[Fact]
public async Task GetProfessionalLessonsByTutorIdAsync_WithTutorIdInputParam_ReturnsListOfLesson()
    {
        // Arrange
        private readonly Mock<IUnitOfWork> _mockUnitOfWork = new Mock<IUnitOfWork>();
        private readonly Mock<IHttpContextAccessor> _mockContextAccessor = new Mock<IHttpContextAccessor>();
        private readonly Mock<IUserService> _mockUserService = new Mock<IUserService>();


        var fakeLessonList = new List<Lesson>
        {
            new Lesson() { LessonId = 1, LessonTypeId = 1,LanguageId = 1, TeacherId = 1, LessonName = "Professional Lesson"},
            new Lesson() { LessonId = 2,LessonTypeId = 2, LanguageId = 2, TeacherId = 2, LessonName = "Professional Lesson"}
        }.AsQueryable().BuildMock();

        _mockUnitOfWork.Setup(uow => uow.Repository<Lesson>().GetEntities(It.IsAny<Expression<Func<Lesson, bool>>>() ,
            It.IsAny<Func<IQueryable<Lesson>, IIncludableQueryable<Lesson, object>>>())).Returns(fakeLessonList.Object);

        LessonService lessonService = new LessonService(_mockUnitOfWork.Object, _mockContextAccessor.Object, _mockUserService.Object);

        // Act
        var exceptedValue = 1;
        List<Lesson> lessons = await lessonService .GetProfessionalLessonsByTutorIdAsync(1);
        var actualValue = lessons.Count; // Here count should be 1 but its getting 2

        //Assert
        Assert.Equal(exceptedValue, actualValue);
 }

问题是当await lessonService.GetProfessionalLessonsByTutorIdAsync(1); 在测试方法中调用它返回 2 个项目，实际上它应该返回 1 与匹配的条件。

我猜问题出在以下模拟设置代码中：

_mockUnitOfWork.Setup(uow => uow.Repository<Lesson>().GetEntities(It.IsAny<Expression<Func<Lesson, bool>>>() ,
                It.IsAny<Func<IQueryable<Lesson>, IIncludableQueryable<Lesson, object>>>())).Returns(fakeLessonList.Object);

可能是我错过了什么！ 请高手帮忙解答！

注意：如果我按如下方式修改原始方法，则它可以工作。

public async Task<List<Lesson>> GetProfessionalLessonsByTutorIdAsync(long tutorId)
{
     return await _unitOfWork.Repository<Lesson>().GetEntities().Where(l => l.TeacherId == tutorId && l.LessonTypeId == 1)
                .AsNoTracking().ToListAsync();
}

现在另一个问题是为什么测试方法适用于GetEntities().Where(l => l.TeacherId == tutorId && l.LessonTypeId == 1)但不适用于.GetEntities(l => l.TeacherId == tutorId && l.LessonTypeId == 1) 。

Answer 1

您设置的问题在于您将GetEntities设置为始终返回完整的fakeLessonList列表。 您永远不会遇到作为参数提供的查询。

为此，您可以使用 Moq Returns()方法的另一个重载，该方法提供调用者通过 lambda 方法传递的参数。 此外，如果您想根据给定条件实际过滤掉列表，即实际运行查询，则无需模拟列表。

    var fakeLessonList = new List<Lesson>
    {
        new Lesson() { LessonId = 1, LessonTypeId = 1,LanguageId = 1, TeacherId = 1, LessonName = "Professional Lesson"},
        new Lesson() { LessonId = 2,LessonTypeId = 2, LanguageId = 2, TeacherId = 2, LessonName = "Professional Lesson"}
    }.AsQueryable(); // .BuildMock(); - no mock, just a real list

    _mockUnitOfWork.Setup(uow => uow.Repository<Lesson>().GetEntities(It.IsAny<Expression<Func<Lesson, bool>>>(),
        It.IsAny<Func<IQueryable<Lesson>, IIncludableQueryable<Lesson, object>>>()))
            .Returns(
                (Expression<Func<Lesson, bool>> condition,
                 Func<IQueryable<Lesson>, IIncludableQueryable<Lesson, object>> include) =>
                // Run the queries against the list
                // Need to add some checks in case any of those are null
                fakeLessonList.Where(condition)
            );

我没有测试代码，但我希望它能让您了解需要调整的内容。

编辑：这是您的测试中发生的事情。

• 您设置了一个模拟列表、一个GetEntities()模拟和其他一些...
• 你调用真正的await lessonService .GetProfessionalLessonsByTutorIdAsync(1); 方法。
• 在方法内部有一个对GetEntities(condition)模拟的调用，但它只是忽略了condition因为你的模拟（你设置它的方式）不评估任何条件，它总是返回完整列表。

换句话说，无论您的GetProfessionalLessonsByTutorIdAsync方法是调用GetEntities(condition)还是GetEntities()没有区别，因为模拟设置为始终返回完整列表并忽略传递给它的任何条件。

如果您更改GetProfessionalLessonsByTutorIdAsync的代码以运行GetEntities().Where(condition)那么您最终将根据GetEntities()返回的列表评估condition 。 这样做的问题是您不再能够控制condition发生的事情。