Question

我有一个用ASP.NET Core编写的基本CRUD应用程序，其示例控制器操作如下所示：

public IActionResult Sheet(Guid characterId)
{
    var character = _repository.GetCharacter(characterId);
    // omit validation for brevity
    return View("Sheet", new ViewModel(character, true, true));
}

一个单元测试示例：

[Fact]
public void TestSheet()
{
    // Arrange stuff and mock setup
    // Act
    var result = _controller.Sheet(characterId) as ViewResult;

    // Assert
    Assert.NotNull(result);

    // assert ViewModel constructor called with x arguments
}

其他研究似乎建议对该对象进行模拟，但这不适合这种情况。我可以简单地让ViewModel的构造函数设置几个getter属性，但这似乎是钻研测试ViewModel而不是控制器的领域。控制器不应该关心ViewModel对传递给它的值做什么，在我看来测试也不应该 - 测试是测试控制器的行为是否正确，并且应该只测试它是否传递了正确的值。一些吸气剂属性肯定会实现这一点，但它在概念上似乎是错误的。

Answer 1

该动作与视图模型的创建紧密相关，可以反转

public interface IViewModelFactory {
    T CreateNew<T>(params object[] args) where T : ViewModelBase;
}

public class ViewModelFactory : IViewModelFactory {
    public T CreateNew<T>(params object[] args) where T : ViewModelBase {
        return (T)Activator.CreateInstance(typeof(T), args);
    }
}

public class ViewModelBase { }

因此创建/初始化视图模型不是控制器的责任。这将是一个显式依赖注入控制器并用于创建视图模型。

public IActionResult Sheet(Guid characterId) {
    var character = _repository.GetCharacter(characterId);
    // omit validation for brevity
    return View("Sheet", _viewModelFactory.CreateNew<ViewModel>(character, true, true));
}

在单元测试中行使期望的行为可以断言

[Fact]
public void TestSheet() {
    // Arrange stuff and mock setup
    var factory = new Mock<IViewModelFactory>();
    factory.Setup(_ => _.CreateNew<ViewModel>(It.IsAny<object[]>()))
           .Returns((object[] args) =>
               new ViewModel((Character)args[0], (bool)args[1], (bool)args[2])
           );      

    // Act
    var result = _controller.Sheet(characterId) as ViewResult;

    // Assert
    Assert.NotNull(result);

    // assert ViewModel constructor called with x arguments
    factory.Verify(_ => _.CreateNew<ViewModel>(It.IsAny<Character>(), true, true), Times.AtLeastOnce());
}

使用Moq测试viewmodel构造函数参数

1 个答案: