C# 中对同一结果进行多个断言的最佳实践

Question

您认为对结果进行多个断言的最简洁方法是什么？过去我对它们进行了相同的测试，但这开始感觉有点肮脏，我只是在使用设置来尝试另一个想法。

[TestFixture]
public class GridControllerTests
{
    protected readonly string RequestedViewId = "A1";

    protected GridViewModel Result { get; set;}

    [TestFixtureSetUp]
    public void Get_UsingStaticSettings_Assign()
    {
        var dataRepository = new XmlRepository("test.xml");

        var settingsRepository = new StaticViewSettingsRepository();

        var controller = new GridController(dataRepository, settingsRepository);

        this.Result = controller.Get(RequestedViewId);

    }

    [Test]
    public void Get_UsingStaticSettings_NotNull()
    {
        Assert.That(this.Result,Is.Not.Null);
    }

    [Test]
    public void Get_UsingStaticSettings_HasData()
    {
        Assert.That(this.Result.Data,Is.Not.Null);
        Assert.That(this.Result.Data.Count,Is.GreaterThan(0));
    }

    [Test]
    public void Get_UsingStaticSettings_IdMatches()
    {
        Assert.That(this.Result.State.ViewId,Is.EqualTo(RequestedViewId));
    }

    [Test]
    public void Get_UsingStaticSettings_FirstTimePageIsOne()
    {
        Assert.That(this.Result.State.CurrentPage, Is.EqualTo(1));
    }
}

Answer 1

在同一个测试中使用多个断言可能会导致断言轮盘，所以这是你应该做的事情一定要小心。

然而，当断言不相关时，断言轮盘赌通常会出现问题。如果它们在概念上密切相关，则许多断言通常可以被视为单个逻辑断言。

在许多情况下，您可以通过将此类逻辑断言显式封装在自定义类型或方法中来获得两全其美的效果。

Answer 2

您需要遵循的是 Arrange、Act、Assert 的模式（然后结束测试）。在您的情况下，所有安排都在 TestFixtureSetUp 中，就像正在测试的操作一样。我会稍微重新安排一下，当您进行更多测试时，它可能会变得难以操作。正如 Dockers 指出的那样，应避免繁重的测试设置，它们可能会成为问题 - 它们在课堂上的所有测试中都是“一刀切”，因此可能会变得比大多数测试所需的更重。

如果您想继续进行另一个后续操作，然后进行更多断言，请将其放入单独的测试中。

我对将多个断言放在同一个测试中没有任何问题，只要它们有助于测试相同的事物（即是同一“逻辑断言”的一部分）。在这种情况下，对 this.Result.Data 的内容进行任意数量的断言对我来说都是可以的 - 它们都会检查相同的结果值。您的 Get_UsingStaticSettings_HasData 非常清楚地做到了这一点。最好对每个断言使用唯一的失败消息，以便更容易判断哪个断言失败。

或者，您可以将相关断言包装在一个方法中。如果您多次使用它，这对于通常的 DRY 原因很有用，但除此之外我不认为这是一个很大的区别。

总结
* 每次测试执行一项操作
* 操作之后，根据需要使用尽可能多的相关断言来测试一件事
* 在那里结束测试。

Answer 3

您可以使用 Oapt - 用于每个测试运行一个断言的 NUnit 插件：

[TestFixture]
public class GridControllerTests
{
  [TestCase, ForEachAssert]
  public void Get_UsingStaticSettings_Assign()
  {
      var dataRepository = new XmlRepository("test.xml");
      var settingsRepository = new StaticViewSettingsRepository();
      var controller = new GridController(dataRepository, settingsRepository);

      var result = controller.Get("A1");

      AssertOne.From(
        () => Assert.That(this.Result,Is.Not.Null),
        () => Assert.That(this.Result.Data,Is.Not.Null),
        () => Assert.That(this.Result.Data.Count,Is.GreaterThan(0)),
        () => Assert.That(this.Result.State.ViewId,Is.EqualTo(RequestedViewId)),
        () => Assert.That(this.Result.State.CurrentPage, Is.EqualTo(1)));
  }
}

这将创建 5 个不同的测试用例，每个断言一个。

Answer 4

我倾向于仅在断言本身有价值的情况下才单独提出断言。如果我想要一个单独的断言，我会创建一个自定义断言：

AssertThatMyObjectMatches(field1, field2, field3, field4, myObject);

不过，有时我喜欢在测试中使用多个示例。当一半行为没有另一半就没有价值时，我就会这样做。

Assert.True(list.IsEmpty());

list.Add(new Thing());
Assert.False(list.IsEmpty());

其他人，包括 Ruby 社区的大部分人，对此有不同的看法。它主要由 Dave Astels 的博客文章驱动，此处：

http://www. artima.com/weblogs/viewpost.jsp?thread=35578

我发现“每个测试一个断言”方法对于验证之类的事情非常有用，其中每个小方面都很有价值。不然我也不会这么担心。

对您和您的团队有效的方法可能就是正确的方法。当我更好地了解什么是正确的事情时，我倾向于做任何看起来简单且容易改变的事情，然后将其变为正确的事情。我还在更复杂的示例中添加了大量单元级 Given / When / Then 注释，如果类变得太复杂而难以理解，则将其拆分。

以这种方式编写测试的原因并不是为了捕获损坏的东西。它是为了帮助人们理解代码并在不破坏代码的情况下对其进行更改。