当我模拟 ASP.NET MVC 控制器时，我的 ActionMethod 不返回任何视图。为什么？

Question

在我的简单 Index() ActionMethod 中，我引用了 User.Identity 属性。所以，我想我需要嘲笑这一点。

所以我创建了一些模拟 HomeController 并在我的单元测试中使用它。当我这样做时，ActionMethod 返回 null 作为视图。当我用具体实例替换模拟控制器时（当然还要注释掉对 User.Identity 的任何引用），则会返回正确的视图。

例如。

// Arrange.
var homeController = Mock<HomeController>(..);
homeController.Setup(x => x.User).Returns(new GenericPrincipal(..));

// Act.
var result = homeController.Index();

// Assert.
Assert.IsNotNull(result); // This fails here. result is NULL.

但是当我这样做时（并注释掉任何 User 引用），它起作用了......

// Arrange.
var homeController = new HomeController(..);

// Act.
var result = homeController.Index();

// Assert.
Assert.IsNotNull(result); // Tick!

有什么想法吗？

Answer 1

您的单元测试中有一些奇怪的事情。您正在对控制器进行单元测试，但正在模拟被测对象的创建： var homeController = Mock(..); 这是不正确的。

以下是将模拟用户注入到您愿意进行单元测试的控制器中的正确方法：

// arrange
var sut = new HomeController();
var user = new GenericPrincipal(new GenericIdentity("foo"), null);
var httpContext = new Mock<HttpContextBase>();
httpContext.Setup(x => x.User).Returns(user);
var context = new ControllerContext(new RequestContext(httpContext.Object, new RouteData()), sut);
sut.ControllerContext = context;

// act
var actual = sut.Index();

// assert
...

Answer 2

我认为你应该模拟 HttpContext 供控制器使用。 我提供了一个您可以使用的另一个答案。作为 Steve Rowbotham 说，你应该模拟被测系统的依赖关系（即控制器的依赖关系），而不是模拟被测系统本身；您想要测试控制器的真实版本而不是模拟版本:)

使用链接中的 HttpContextBase 类，您只需在测试中执行以下操作

var controllerToTest = new HomeController();
var context = new MockHttpContextBase(controllerToTest);

// do stuff that you want to test e.g. something goes into session

Assert.IsTrue(context.HttpContext.Session.Count > 0); 

您可以更进一步，创建 Setup 和 TearDown使用模拟上下文设置控制器的方法。

Answer 3

老实说，这看起来是一个非常奇怪的测试，因为您正在模拟被测系统 (SUT)，即 HomeController。通常，人们会模拟 SUT 的依赖项，设置对模拟的期望，并将模拟注入到 SUT 中，以确认它与其依赖项正确交互。

当您创建 HomeController 的模拟时，Moq 将创建一个继承自 HomeController 的类并覆盖虚拟方法 Index。因此，当您在模拟上调用 Index 时，您并不是调用在 HomeController 类中定义的 Index 的实现，而是调用被重写的实现。由于您没有显式Setup模拟上的方法，它将返回默认值，在本例中为null。

在第二个测试中，您将调用 Index 的实际实现，因为您正在构造 HomeController 类的实际实例。如果您在模拟对象的实例上调用 GetType() ，那么您将看到它是派生自 HomeController 的代理实例，它拦截对公共定义的调用，基类上的可重写方法（这是模拟对象的很大一部分目的）。

Answer 4

我认为您的 Index 方法可能是虚拟的，这会导致 Moq 用模拟函数替换该函数。为了防止这种情况，您需要在模拟上设置 CallBase 属性。

但是，我同意其他回复，即您不应该嘲笑您的控制器，但您应该嘲笑您的依赖关系。

（一个更简单的方法是创建一个专门的模型绑定器，它可以从 HttpContext 获取主体，然后您可以将主体作为输入参数传递给您的方法）