Question

我对单元测试和嘲笑非常新！我正在尝试编写一些单元测试，其中包含一些与数据存储交互的代码。数据访问由IRepository封装：

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

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

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")....

返回包含名称设置为“Company Inc”的Company对象的内存中IEnumberable。调用SignupLogic.AddNewCompany的单元测试会设置一个具有重复详细信息和trys的公司来传递它，并且我断言抛出了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”，但这不会导致测试通过。

从我正在阅读的所有内容来看，测试Expressions正如我所尝试的那样，实际上并不能在这里使用Moq。可能吗？请帮忙！

Answer 1

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

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

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

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.

第二次测试，使返回类型为空列表，这将导致添加被调用。：

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

您通常只会模仿您拥有的类型。那些你不拥有的人，真的不应该因各种困难而被嘲笑。所以嘲弄表达 - 就像你的问题所暗示的那样 - 不是要走的路。

在Moq框架中。将.Returns()放在函数中非常重要，否则它不匹配。所以，如果你还没有这样做，那就是你的问题。

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

表达式<func <t，bool =“”>＆gt;的Moq'ing方法作为参数</func <t，>传入

3 个答案: