单元测试大脑冻结答案

【问题标题】：Unit Testing Brain Freeze单元测试大脑冻结
【发布时间】：2012-02-22 16:41:08
【问题描述】：

我有一个如下所示的类方法：

private List<string> DataStoreContents = new List<string>(new[] { "", "", "", "" });

public void InputDataStore(int DataStore, string Data)
{
  DataStoreContents[DataStore - 1] = Data;
}

我想确保 DataStore >=1 和

如何编写单元测试来确保这一点？

【问题讨论】：

不确定我是否理解这个问题。标题说单元测试，但我在示例中没有看到单元测试。你能详细说明一下吗？
我想编写一个单元测试，以确保我优雅地退出方法或者如果参数不是 >= 1 和
你不是说你想要一个前置条件，比如代码合约？
我想我可以编写一个单元测试来检查我是否抛出异常，如果这些参数不是我想要的但我不想抛出异常
虽然我已经在下面回答了 - 我也很困惑。您是说要在此处放置某种屏障检查以检查传入的 DataStore 值是否在 1-4 范围内，还是说您要进行外部测试以检查 DataStore 时是否发生某种行为/不在这个范围内？

标签： c# .net unit-testing nunit

【解决方案1】：

要么

Assert.IsTrue(DataStore >= 1 && DataStore <= 4);

或者，如果您更喜欢流畅的界面

Assert.That(DataStore, Is.GreaterThanOrEqualTo(1).And.LessThanOrEqualTo(4));

[编辑 - 回应您上述说明]

听起来您想要进行某种障碍检查以检查提供的值是否在范围内。

在这种情况下，您有几个选择：

Philip Fourie 给出了一个涉及代码合约的答案。

另一种简单的方法是自己编写障碍检查：

public void InputDataStore(int DataStore, string Data)
{
    if (DataStore < 1 || DataStore > 4)
    {
        throw new ArgumentOutOfRangeException("DataStore", "Must be in the range 1-4 inc.");
    }
    DataStoreContents[DataStore - 1] = Data;
}

如果您不想抛出异常，但可能想记录并干净退出：

public void InputDataStore(int DataStore, string Data)
{
    if (DataStore < 1 || DataStore > 4)
    {
        // log something here and then return
        return;
    }
    DataStoreContents[DataStore - 1] = Data;
}

链接回单元测试。例如，单元测试可以是您编写的测试，以检查当使用超出范围的值调用 InputDataStore 时，它是否会引发异常。另一种情况是，当使用范围内的值调用它时，它不会抛出异常，并且会正确更新DataStoreContents。

【讨论】：

因为我看不到更完整的画面，你能不能扩展一下？
@Jon 假设超出范围变量的期望行为是将某些内容写入日志。您的测试会将一个超出范围的变量传递给该方法，然后检查是否有任何内容写入日志。执行此操作的典型方法是使用模拟记录器对象（模拟对象通常是实现与真实对象相同接口的专用测试对象）。测试运行后，您将询问模拟记录器以查看是否进行了所需的记录调用。

【解决方案2】：

Assert.IsTrue(DataStore >= 1 && DataStore <= 4);

【讨论】：

因为我看不到更完整的画面，你能不能扩展一下？
@Jon 利用测试框架，您正在为单元测试调用IsTrue() 方法。基本上，如果DataStore >= 1 && DataStore <= 4) 不正确，那么您将通过 (Assert.IsTrue() failed) 获得例外。否则，测试将通过。

【解决方案3】：

也许是这个？（在你修复之前都应该失败）

[Test]
[TestCase(5)]
[TestCase(0)]
[TestCase(int.MaxValue)]
[TestCase(int.MinValue)]
public void InvalidIndices(int index)
{
    Assert.DoesNotThrow(() => yourObj.InputDataStore(index, "don't care"));
}

或者（应该都通过）

[Test]
[TestCase(5)]
[TestCase(0)]
[TestCase(int.MaxValue)]
[TestCase(int.MinValue)]
public void InvalidIndices(int index)
{
    Assert.Throws<IndexOutOfRangeException>(() => yourObj.InputDataStore(index, "don't care"));
}

【讨论】：

我显然需要修改方法以在这些情况下抛出异常
@Jon：您需要修改代码以不抛出异常（默认情况下会出现 IndexOutOfRangeException）。这些只是你为定义类契约而编写的测试——索引错误，做/不抛出异常。

【解决方案4】：

您还可以使用具有许多其他好处的代码合同，例如静态代码检查。这意味着您将在“代码时间”期间收到有关错误使用该方法的警告。

public void InputDataStore(int DataStore, string Data)
{
  Contract.Requires(DataStore >= 1 && DataStore <= 4);
  DataStoreContents[DataStore - 1] = Data;
}

在这里阅读：http://devjourney.com/blog/code-contracts-part-1-introduction/

【讨论】：

@Jon，我建议进一步阅读。我已经用一篇描述代码契约的好文章更新了我的答案，它们真的值得花时间去理解它们。

【解决方案5】：

我认为你不能在这里真正“测试”。

您可以插入一个检查，该检查将在运行时执行。所述检查可能会有所帮助，但它不会比您得到的 ArrayOutOfBoundsException 更有帮助...

另外，插入检查与测试不同。

您应该查看 InputDataStore 函数的调用者。

您可以测试这些：创建一些不同的情况，执行调用者并检查它们是否将正确的值传递给 InputDataStore。

【讨论】：