0

我和我的同事正在讨论编写单元测试的正确方法，以确保用户在我们的 ASP.NET MVC 2 应用程序的表单中输入错误数据时收到错误。这是我们过去所做的，从模型开始：

public class LoginModel
{
    public string Username { get; set; }
}

这是控制器动作：

[HttpPost]
public ActionResult Login( LoginModel loginModel )
{
    if ( loginModel.Username == null )
    {
        ModelState.AddModelError( "Username", "Username is required!" );

        return View( loginModel );
    }

    LoginService.Login( loginModel );
}

最后，这是测试方法：

[TestMethod]
public void Login_Post_Blank_Username_Displays_Error()
{
    var controller = GetHomeController();

    var loginModel = new LoginModel
    {
        Username = null
    };

    var result = controller.Login( loginModel );

    Assert.IsInstanceOfType( result, typeof( ViewResult ) );

    var view = (ViewResult)result;

    Assert.IsNotNull( view.ViewData.ModelState["Username"].Errors.First().ErrorMessage );
}

他向我指出，这确实不是针对这种情况编写测试的正确方法。出于一个原因，它非常脆弱 - 将 Username 属性更改为其他任何东西都会破坏测试。其次，最好依靠 DataAnnotations 并针对控制器正在做的事情进行测试。因此，我们的新模型如下所示：

public class LoginModel
{
    [Required( ErrorMessage = "Username is required!" )]
    public string Username { get; set; }
}

我们的控制器动作会变成这样：

[HttpPost]
public ActionResult Login( LoginModel loginModel )
{
    if ( !Model.IsValid() )
    {
        return View( loginModel );
    }

    LoginService.Login( loginModel );
}

问题来自单元测试，它完全没有注意到 DataAnnotations，因此测试失败。我的同事说，我们真正应该测试的是没有调用 LoginService，但我不确定如何测试。他建议像这样使用起订量：

[TestMethod]
public void Login_Post_Blank_Username_Displays_Error()
{
    var controller = GetHomeController();

    var loginModel = new LoginModel
    {
        Username = null
    };

    loginServiceMock.Setup( x => x.Login( It.IsAny<LoginModel>() ) )
        .Callback( () => Assert.Fail( "Should not call LoginService if Username is blank!" ) );

    var result = controller.Login( loginModel );

    loginServiceMock.Verify();
}

您对此有何看法？测试未调用服务方法的正确方法是什么？在用户表单中测试不良数据的正确方法是什么？

4

2 回答 2

3

验证时：

loginServiceMock.Verify( x => x.Login( It.IsAny<LoginModel>() ), Times.Never() );

您应该删除模拟中的设置。为了使代码在您的操作中输入 if 语句，您可以在调用控制器中的操作之前在 ModelState 中添加模型错误。

您的代码应如下所示：

[TestMethod]
public void Login_Post_Blank_Username_Displays_Error()
{
    var controller = GetHomeController();

    var loginModel = new LoginModel
    {
        Username = null
    };

    controller.ModelState.AddModelError("a key", "a value");

    var result = controller.Login( loginModel );

    loginServiceMock.Verify( x => x.Login( It.IsAny<LoginModel>() ), Times.Never() );
}
于 2012-06-20T23:56:45.213 回答
1

假设您的 Mocked LoginService 已通过属性注入设置，您可以编写如下测试..

    [TestMethod]
    public void LoginPost_WhenUserNameIsNull_VerifyLoginMethodHasNotBeenCalled()
    {
        //Arrange
        var loginServiceMock = new Mock<ILoginService>();
        var sut = new HomeController { LoginService = loginServiceMock .Object};

        var loginModel = new LoginModel {
            Username = null
        };
        sut.ModelState.AddModelError("fakeKey", "fakeValue");

        //Act
        sut.Login(loginModel);

        //Verify
        loginServiceMock.Verify(x => x.Login(loginModel), Times.Never(), "Login method has been called.");
    }
于 2012-06-21T05:51:10.670 回答