2

好的,我有一个奇怪的问题。

我在 C# 测试项目中使用 NUnit 和 Moq。我认为您可以忽略模拟对象,因为真正的问题在于 UserManager.AddUser() 没有正确执行。

以下代码将永远不会到达 UserManager.AddUser() 开始处的断点,因此由于 NullPointerException 将导致 Assertion 失败。

[Test]
enter code here
public void AddUser_NoPassword_GeneratesPassword()
    {
        //assert
        var userRepositoryFake = new Mock<IUserRepository>();
        userRepositoryFake.Setup(x => x.GetUser(It.IsAny<string>(), It.IsAny<bool>())).Returns((User)null);
        userRepositoryFake.Setup(x => x.SaveUser(It.IsAny<User>())).Returns(new Mock<User>().Object);

        var userManager = new UserManager(userRepositoryStub.Object);
        var createUserViewModel = new CreateUserViewModel { Username = "username" };

        //act
        var validationErrors = userManager.AddUser(createUserViewModel);

        //assert
        Assert.IsNotNullOrEmpty(createUserViewModel.Password);
    }

但是,如果我添加这个断言:

        //assert
        Assert.IsEmpty(validationErrors); //new assertion
        Assert.IsNotNullOrEmpty(createUserViewModel.Password);

到达 AddUser() 中的断点,测试成功。似乎测试根本不执行 AddUser() ,除非它的返回值用于某些事情。

4

1 回答 1

5

我的猜测是这样AddUser实现的:

public IEnumerable<Error> AddUser(Model model)
{
    // Do some stuff
    if (foo)
    {
        yield return ...;
    }
    if (bar)
    {
        yield return ...;
    } 
    // More stuff
}

换句话说,使用迭代器块。迭代器块被延迟执行 - 代码仅在请求第一个元素时开始执行,即使在返回该元素时它也会“暂停”。

List<Error>鉴于这不是幂等查询或类似的查询,我建议使用您添加到的内部然后在最后返回的方法来实现该方法会更有意义。这样,该方法将只执行一次,但返回值会被迭代多次。我怀疑这是预期的执行模式。

于 2013-04-26T15:11:40.280 回答