c# - 单元测试方法的有效输入

Question

问题是，在您的应用程序项目中使用一种方法，该方法除了为您的测试项目生成数据外没有其他目的，这是否是一种不好的做法。

我有一个单元测试，我正在使用它来进行粗略检查，以确保所有有效输入都将通过我的应用程序的主要方法运行而不会出现任何错误。我本质上是运行一个方法来从数据库中提取每个有效输入，然后通过应用程序的主要方法运行每个输入。如果失败，则将 bool 值设置为 false。

下面是我用来执行此操作的代码示例。问题是是否有更好的方法不需要我向应用程序代码添加任何内容来执行此操作。下面的方法要求我在应用程序项目中有一个方法 (TestMethod) 提取所有有效参数，以便通过测试项目中的主要方法 (CheckAvailability) 运行它们。

public void SomeUnitTest()
{
    Availability Availability = new Availability();

    List<TestParam> paramList = new List<TestParam>();
    bool success = true;
    bool expected = true;

    //This method pulls every valid param from my database.
    paramList = Availability.TestMethod();

    //This foreach loop runs each one of those valid params through another method. If there is an error, 
    //success is set to false otherwise it remains true.
    foreach (TestParam s in paramList)
    {
        try
        {
            InputWrapper Wrapper = new InputWrapper();
            Wrapper.ApplicationName = s.APPname;
            Wrapper.Location = s.APPLocation;
            Availability.CheckAvailability(Wrapper);  
        }         
        catch(Exception)
        {
            success = false;
        }

        //I then assert that success remains true. If it is false, it means that
        //the method failed.
        Assert.AreEqual(expected, success);    
    }
} 

Answer 1

您似乎认为用于测试的数据源是一件值得怀疑的事情。

首先让我说这根本不是这种情况，但是，(总有一个但是)你应该记住一个好的单元测试很容易编写(并且 -也许更重要的是 - 阅读)所以理想情况下，您的单元测试中的层数应该尽可能少。

这让您陷入困境:我是要确保阅读我的测试的任何人都可以只看测试方法并知道将要运行的所有内容，还是我要添加一些层以保持测试干净但同时增加复杂性？

很多事情都是这样:你将不得不妥协。我认为你可以在你的单元测试中增加 2 或 3 层的复杂性，但绝对不应该超过这个。

在您的示例中，这意味着我们可以通过提取测试数据以使其与实际测试分开来增加一层复杂性。
这不会对理解测试造成太大的负担，但会使编写和维护测试变得清晰。

你的问题的另一个方面引起了一些人的注意:你说的是将测试数据保存在数据库中。

这不是您的生产数据库，是吗？如果是:停止对实时数据进行测试。您需要对测试进行绝对控制，以确保没有测试数据被更改，并且环境不会在您不知情的情况下发生变化(除了实际生产数据的潜在中断)。

也不需要使用 bool 变量:抛出的异常将自动导致错误测试。

我会更详细地介绍这些事情 here和 here ，请通读它们并随时提出任何后续问题。