Moqing 方法，其中 Expression>作为参数传入

Question

我对单元测试和模拟非常陌生！我正在尝试编写一些单元测试，其中涵盖一些与数据存储交互的代码。数据访问由 IRepository 封装：

interface IRepository<T> {
    ....
    IEnumerable<T> FindBy(Expression<Func<T, bool>> predicate);
    ....
}

我尝试使用 IRepository 的具体 IoC 实现来测试的代码如下所示：

public class SignupLogic {
    private Repository<Company> repo = new Repository<Company>();

    public void AddNewCompany(Company toAdd) {
        Company existingCompany = this.repo.FindBy(c => c.Name == toAdd.Name).FirstOrDefault();

        if(existingCompany != null) {
            throw new ArgumentException("Company already exists");
        }

        repo.Add(Company);
        repo.Save();
    }
}

因此，我正在测试 SignupLogic.AddNewCompany() 本身的逻辑，而不是逻辑和具体的存储库，我正在模拟 IRepository 并将其传递到 SignupLogic 中。模拟的存储库如下所示：

Mock<Repository> repoMock = new Mock<Repository>();
repoMock.Setup(moq => moq.FindBy(c => c.Name == "Company Inc")....

它返回一个内存中的 IEnumberable，其中包含名称设置为“Company Inc”的 Company 对象。调用 SignupLogic.AddNewCompany 的单元测试设置了一家具有重复详细信息的公司，并尝试将其传入，我断言会引发 ArgumentException 并显示消息“公司已存在”。这个测试没有通过。

在运行时通过单元测试和 AddNewCompany() 进行调试，结果显示，existingCompany 始终为 null。在绝望中，我发现如果我更新 SignupLogic.AddNewCompany() ，以便对 FindBy 的调用如下所示：

Company existingCompany = this.repo.FindBy(c => c.Name == "Company Inc").FirstOrDefault();

测试通过，这表明 Moq 仅响应完全 与我在测试装置中设置的相同。显然，这在测试任何重复的公司是否被 SignupLogic.AddNewCompany 拒绝时并不是特别有用。

我尝试将 moq.FindBy(...) 设置为使用“Is.ItAny”，但这也不会导致测试通过。

从我读到的所有内容来看，似乎用 Moq 来测试我正在尝试的表达式实际上是不可能的。是否可以？请帮忙！

Answer 1

只有具有完全相同的结构（和文字值）的 Expression 才会匹配，这可能是正确的。我建议您使用 Returns() 的重载，它允许您使用调用模拟的参数：

repoMock.Setup(moq => moq.FindBy(It.IsAny<Expression<Func<Company, bool>>>())
        .Returns((Expression<Func<Company, bool>> predicate) => ...);

在 ... 中，您可以使用 predicate< /code> 返回匹配的公司（如果匹配的公司不是您所期望的，甚至可能抛出异常）。不是很漂亮，但我认为它会起作用。

Answer 2

您应该能够使用 It.IsAny<>() 来完成您想要做的事情。通过使用 It.IsAny<>()，您可以简单地调整设置的返回类型来测试代码的每个分支。

It.IsAny<Expression<Func<Company, bool>>>()

第一个测试，返回一个公司，无论谓词如何，这将导致抛出异常：

var repoMock = new Mock<IRepository<Company>>();
repoMock.Setup(moq => moq.FindBy(It.IsAny<Expression<Func<Company, bool>>>())).Returns(new List<Company>{new Company{Name = "Company Inc"}});
var signupLogic = new SignupLogic(repoMock.Object);
signupLogic.AddNewCompany(new Company {Name = "Company Inc"});
//Assert the exception was thrown.

第二个测试，使返回类型为空列表，这将导致 add 被调用：

var repoMock = new Mock<IRepository<Company>>();
repoMock.Setup(moq => moq.FindBy(It.IsAny<Expression<Func<Company, bool>>>())).Returns(new List<Company>());
var signupLogic = new SignupLogic(repoMock.Object);
signupLogic.AddNewCompany(new Company {Name = "Company Inc"});
repoMock.Verify(r => r.Add(It.IsAny<Company>()), Times.Once());

Answer 3

您通常只嘲笑您拥有的类型。那些不属于你的，真的不应该因为种种困难而被嘲笑。因此，模拟表达式 - 正如你的问题的名称所暗示的那样 - 不是正确的方法。

在起订量框架中。重要的是要为函数添加 .Returns() ，否则将不匹配。所以，如果你还没有这样做，那就是你的问题了。

repoMock.Setup(moq => moq.FindBy(c => c.Name == "Company Inc").Returns(....