单元测试异常已被捕获和处理

Question

我写了一个使用BufferedReader和FileReader的方法，结果就是需要处理FileNotFoundException和IOException。

现在我在我的类中使用了 try 和 catch 语句来处理异常，所以那里没有问题。

我遇到的问题是编写一个测试来验证异常是否已被捕获。

我无法编写如下测试，因为我的方法已经捕获了异常，因此它不会传播到测试中进行处理：

@Test(expected = FileNotFoundException.class)
public void testFileNotFound() {
    ...
};

如何测试是否已捕获异常？我是否需要创建我的类的模拟对象并验证是否使用 Mockitio 之类的东西捕获了异常？还是不创建一个 mock 并使用 Mockitio 来监视实际对象以验证是否捕获了异常？

编辑：这是我要测试的方法...

public List<String[]> parse(String fileToRead) {

    BufferedReader fileReader = null;
    List<String[]> parsedData = new ArrayList<String[]>();
    String currentLine;

    try {
        fileReader = new BufferedReader(new FileReader(fileToRead));

        while ((currentLine = fileReader.readLine()) != null) {
            String[] parsedLine = currentLine.split(",");
            parsedData.add(parsedLine);
        }
    }
    catch (FileNotFoundException e) {
        logger.error("File does not exist", e);
    }
    return parsedData;
}

Answer 1

我认为这是尝试与语言对抗而不是使用它的经典示例。它也可能是设计上不太可测试的代码示例。

首先，了解受检异常和未受检异常之间的概念区别很重要。像NullPointerException 和ArrayIndexOutOfBoundsException 这样的未经检查的异常是“未经检查的”（意味着编译器不关心它们），因为当它们被抛出时，通常是程序员的错。这些类型的异常通常可以在方法调用之前使用 if 语句轻松防范。另一方面，IOException 和 FileNotFoundException 被“检查”（意味着编译器会抱怨缺少 catch/throw），因为即使您在尝试读取文件之前使用 File#exists() 方法检查文件是否存在，同时可能会发生外部事件导致文件丢失，或者在您读取文件时可能会从机器中取出硬盘驱动器。检查的异常通常表示一种崩溃状态，您必须具体决定如何从中恢复，并且只有您（作为程序员）才能做出该决定。

考虑你的方法。您如何通过方法的输出判断读取操作是失败还是使用空文件成功？你不能，因为即使这两个事件完全不同，你处理它们的方式完全相同。这可能是您想要的，也可能不是您想要的，但与您调用该方法的位置不同，处理这两种情况可能更合适。例如，您可能希望向用户显示“文件为空”或“文件不存在”的消息，而不是“无数据”。

其次，可测试性问题（这可能是也可能不是问题，具体取决于您的设置）。 catch-clause 中唯一发生的事情（因此是引发和捕获异常的唯一迹象）是臭名昭著的“记录并忘记”策略。您正在记录错误（而不是在适当的情况下对其进行处理/反应），因此要验证是否捕获了异常，您唯一真正的选择是模拟记录器（如果您的设计允许）并验证其 error() 方法是叫。如果您的目标是验证在给定特定输入的情况下是否引发异常，您不能只检查输出，因为这只是验证输出 - 一个完全不同的测试。您必须检查是否采用了引发异常的代码路径，在这种情况下，一种方法是模拟记录器。