如何在 ASP.NET MVC 中对私有控制器方法进行单元测试？

Question

在 ASP.NET MVC 控制器内对私有方法进行单元测试的最佳实践是什么？目前，如果我想对控制器内的私有方法进行单元测试，我必须将其设置为公共，但是当我这样做时，该方法会暴露在网络上。

将方法移至辅助类的最佳方法是？

Answer 1

您应该将该方法移至您提取接口的辅助类中。这样，可以更轻松地执行依赖项注入并切换底层实现或在需要时模拟它。
测试私有方法是一种代码味道（或测试味道）。
仅在必要时才使用InternalsVisibleTo 技巧（即，您使用的是不可测试的类，并且必须引发通过受保护的函数调用对您隐藏的事件）。

Answer 2

您应该使用私有对象，如下所示：

var privateObject = new PrivateObject(this.controller);
var result = (ActionResult)privateObject.Invoke("RegisterToEvent", shippingAddressID, creditCardID);  

Answer 3

您可以将该方法设置为内部方法而不是私有方法，然后将InternalsVisibleTo属性设置为测试程序集的名称。

在装配信息中：

[assembly: InternalsVisibleTo("MyTestAssembly")]

Answer 4

您可以为这样的测试方法创建一个操作过滤器...

public class TestActionFilter : ActionFilterAttribute
{
    public override void OnActionExecuting(ActionExecutingContext filterContext)
    {
         filterContext.HttpContext.Response.Redirect("loginurl");
         //or throw exception or some other action
    }
}

或者您可以使该方法只能由另一个程序集公开访问。

[assembly: InternalsVisibleTo("TestAssembly")]

Answer 5

您可以通过将测试限制为可见方法来解决该问题。任何私有方法都将至少被一个可见方法使用（否则：可以安全地删除它）。

因此，我建议开发调用具有实用价值的可见方法的测试，以便测试私有算法。如果代码无法通过可见方法访问 - 删除它。

Answer 6

我使用了 PrivateObject() 方法，但我喜欢 Charlino 对 NonActionAttribute 的建议（请参阅对上面原始问题的评论）也是如此。