单元测试;正确的方法？

Question

在工作中，我被要求增加我们在其中一款软件产品中的代码覆盖率。我以前从未做过单元测试，并且在网上阅读了许多教程；这些作为起点很有帮助，但都遵循相同的模式——它们正在测试非常简单的方法/类，如计算器或银行账户。

我在我们的代码中找到了一个简单的开始方法，但问题是它仍然比我读过的示例复杂得多，我不确定从哪里开始。方法如下：

public static void moveFiles {
    string rootDir = ConfigurationManager.AppSettings["RootLoc"];
    string dropboxLoc = ConfigurationManager.AppSettings["DropBoxLocation"];
    DirectoryInfo dropbox = new DirectoryInfo(dropboxLoc);
    folders = dropbox.GetDirectories("*", SearchOption.AllDirectories);

    string path = "";
    string prevPath = "";

    foreach (DirectoryInfo di in folders)
    {
        FileInfo[] files = di.GetFiles();
        foreach (FileInfo fi in files)
        {
            prevPath = fi.FullName;
            string[] p = prevPath.Split(new string[] { dropbox.Name }, StringSplitOptions.None);
            path = rootDir + p[1];
            fi.MoveTo(path);
        }

    }
}

我为此创建了一个小测试方法：

[TestMethod]
public void GetDirectories_ValidLocation_SetsDropboxLocation()
{
        string dropboxLoc = ConfigurationManager.AppSettings["DropBoxLocation"];
        DirectoryInfo dropbox = new DirectoryInfo(dropboxLoc);
        Assert.IsTrue(dropbox.Exists);
}

这是否符合应测试的内容？还是我看错了？

Answer 1

您的单元测试应该测试该方法是否完成了它应该做的事情。你的moveFiles方法应该是移动文件，所以你需要测试它是否移动文件。

这是可以做到的：

1. 在测试项目的 app.config 中，将 RootLoc 和 DropBoxLocation 设置为测试目录。

2. 在你的测试方法中：

   一个。通过在 RootLoc 中创建一些测试文件来设置您的测试环境。确保 DropBoxLocation 为空。

   b.调用你的 moveFiles 方法。

   c。断言 DropBoxLocation 包含您期望的文件。

   d。清理（​​即删除测试文件）。

理想情况下，步骤 a 和 d 可以分别在 TestInitialize 和 TestCleanup 方法中完成。此外，您可能希望将您的方法拆分为两种方法：

public static void moveFiles(string source, string destination)
{
    ...
}

public static void moveFiles()
{
    moveFiles(ConfigurationManager.AppSettings["RootLoc"],
              ConfigurationManager.AppSettings["DropBoxLocation"]);
}

这将允许您单独测试 moveFiles 方法，而无需依赖外部配置文件。这是一个非常简单的依赖注入示例 - 如果你的方法难以测试，你可能需要阅读它。