c# – 我应该如何对返回模型的控制器进行单元测试？

2023年2月3日 210次阅读

我有一个名为PostsController的控制器

public class PostsController : Controller
{
    private readonly IPostsRepository repository;

    public PostsController(IPostsRepository repository)
    {
        this.repository = repository;
    }

    public ViewResult Index()
    {
        var posts =
            repository.All()
                      .OrderBy(post => post.PublishedAt);

        return View("Index", posts.MapTo<PostViewModel>());
    }
}

以及一个名为PostsControllerTest的相应测试夹具

[TestFixture]
public class PostsControllerTest
{
    private PostsController controller;
    private Mock<IPostsRepository> repository;

    [SetUp]
    public void SetUp()
    {
        AutoMapperConfig.Configure();
        repository = new Mock<IPostsRepository>();
        controller = new PostsController(repository.Object);
    }

    [Test]
    public void Index_ReturnsCorrectModel()
    {
        var actual = controller.Index().Model;

        Assert.IsAssignableFrom<IEnumerable<PostViewModel>>(actual);
    }
}

目前我只测试控制器返回正确的模型类型.我是否还应该存储存储库并测试返回正确的数据,如下所示：

[Test]
public void Index_ReturnsCorrectModel()
{
    var post = new Post
    {
        Slug = "continuing-to-an-outer-loop",
        Title = "Continuing to an outer loop",
        Summary = "When you have a nested loop, sometimes",
        Content = "When you have a nested loop, sometimes",
        PublishedAt = DateTime.Now.AddDays(7),
        Tags = new Collection<Tag> { new Tag { Name = "Programming" } }
    };
    repository.Setup(repo => repo.All()).Returns(new[] { post });

    var actual = controller.Index().Model as IEnumerable<PostViewModel>;
    Assert.NotNull(actual);
    Assert.AreEqual(1, actual.Count());
    Assert.AreEqual(post.Title, actual.First().Title);
}

我不知道自己是否正确进行单元测试,感到非常沮丧.我明白我应该采取的明确解释为什么会非常有帮助.

最佳答案 我不认为您需要在此单元测试中测试IPostRepository的功能.您应该为它创建一个单独的单元测试类.

类似地,MapTo< T>的功能也是如此.应单独测试.

单元测试应该只测试SUT(被测系统)的功能,在这种情况下,它是PostsController类的Index方法.

这是一个简单的方法,因此您希望在此单元测试中验证的两件事是：

1- repository.All()方法被调用一次

2-您的视图模型已正确映射(您已经在进行)

这就是我对这种方法进行单元测试的方法：

[Test]
    public void Index_ReturnsCorrectModel()
    {
        // Arrange
        repository.Setup(repo => repo.All()).Returns(Enumerable.Empty<Post>());

        // Act
        var actual = controller.Index().Model;

        // Assert
        Assert.IsAssignableFrom<IEnumerable<PostViewModel>>(actual);
        repository.Verify(repo => repo.All(), Times.Once);
    }

此外,为了最大限度地减少安排单元测试所需的工作量,可以使用像AutoFixture这样的库,它会自动为您创建post对象.

点赞