如何正确地进行单元测试

Question

我正在尝试使用测试驱动设计方法编写一个应用程序--我对单元测试很陌生，所以我只是想知道测试正确输入和异常的正确方法是什么。

我有一个用于加载配置文件的类：

class Config
{
    private XmlDocument configfile;

    public Config()
    {
        configfile = new XmlDocument();
    }

    public void LoadConfigFile(string filename)
    {
        if(string.IsNullOrEmpty(filename)) 
            throw new System.ArgumentException("You must specify a filename");

        try
        {
            configfile.Load(filename);
        }
        catch (Exception ex)
        {
            throw new System.IO.FileNotFoundException("File could not be loaded");
        }
    }
}

null

    /// <summary>
    ///A test for LoadConfigFile
    ///</summary>
    [TestMethod()]
    public void LoadConfigFileTest()
    {
        Config target = new Config(); // TODO: Initialize to an appropriate value
        string filename = "config.xml"; // TODO: Initialize to an appropriate value
        target.LoadConfigFile(filename);
        Assert.Inconclusive("A method that does not return a value cannot be verified.");
    }

    /// <summary>
    ///A test for LoadConfigFile
    ///</summary>
    [TestMethod()]
    [ExpectedException(typeof(System.ArgumentException))]
    public void LoadConfigFileTest1()
    {
        Config target = new Config(); // TODO: Initialize to an appropriate value
        string filename = ""; // TODO: Initialize to an appropriate value
        target.LoadConfigFile(filename);
        Assert.Inconclusive("A method that does not return a value cannot be verified.");
    }

    /// <summary>
    ///A test for LoadConfigFile
    ///</summary>
    [TestMethod()]
    [ExpectedException(typeof(System.IO.FileNotFoundException))]
    public void LoadConfigFileTest2()
    {
        Config target = new Config(); // TODO: Initialize to an appropriate value
        string filename = "blah.xml"; // TODO: Initialize to an appropriate value
        target.LoadConfigFile(filename);
        Assert.Inconclusive("A method that does not return a value cannot be verified.");
    }

另外，这3个测试是否都有try{}catch（）{}语句？在第一个测试中，正确性是隐含的，在第二个和第三个测试中，我无论如何都在检查异常，所以异常对测试没有影响。

Answer 1

你在正确的道路上，但还没有完全实现。

必须调用assert.incolution的情况非常罕见，在您的情况下，这是不必要的：当您预期出现异常并且该异常被抛出时，它将按它应该的方式工作（也就是：它应该显示为绿色结果）。当您期望出现异常但没有抛出异常时，它将显示为失败（又名：红色结果）。更多关于这一点。

事实上，可以测试返回void的方法。它可能会改变某物的状态，而不是返回值。在您的例子中，变量configfile。测试的方法是通过检索值（例如提供getter）和/或使用依赖项注入并在测试中用变量替换假的/mock/stub（选择您的行话）。

关于评论的扩展：

我不希望开发器直接干扰configfile属性，那么我是否应该为测试将其公开，然后将其改回private？

这是一个值得关注的问题，每个开发人员在进行测试时都要面对它。重要的是要意识到一个单元的内部工作不是你通常应该测试的东西。实现细节正是它们的本来面目：实现细节。然而，有时替代方案甚至更不需要，所以你必须进行比较，是否你想做到底。

这个过程--依赖注入--是单元测试的支柱，也将有助于您的第一个评论。