Question

我有一个名为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);
}

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

Answer 1

我认为您不需要在此单元测试中测试IPostRepository的功能。你应该为它创建一个单独的单元测试类。

同样，MapTo<T>的功能应单独测试。

单元测试应该只测试SUT（被测系统）的功能，在这种情况下，这是Index类的PostsController方法。

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

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对象。

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

1 个答案: