java - 尝试 Catch block 有效但测试 assertThrows 失败(Junit 5)

Question

我正在努力学习本教程 JUnit 5: How to assert an exception is thrown?

我使用 Java 10、IntelliJ 2018 和 Junit 5。

我制作了一个计算器应用程序，可以将 2 个分数相加。它检查输入的分母是否为 0。

当我运行测试时，异常消息打印出“未定义的数学表达式”，但我的 IDE 显示“预期会抛出 java.lang.Throwable，但没有抛出任何东西。”我认为我的代码范围有问题？我是新手，请多多关照。我在下面提供了代码和测试:

public class Calculator {
    public static int[] calculate (int firstNumerator, int firstDenominator, int secondNumerator, int secondDenominator) {

        String exceptionMessage = "Undefined Math Expression";
        int resultNumerator;
        int resultDenominator;
        int[] result = new int[2];

        resultNumerator =  (firstNumerator * secondDenominator) +
                (secondNumerator * firstDenominator);
        resultDenominator = firstDenominator * secondDenominator;

        try {
            if (resultDenominator == 0) {
                  throw (new Throwable(exceptionMessage));
            } else {
                result[0] = resultNumerator;
                result[1] = resultDenominator;
            }
        } catch (Throwable e) {
           System.out.println(e.getMessage());
        }

        return result;
    }
}

测试:

class CalculatorTest {
    @Test
    void denominatorContainsZero() {
        assertThrows(Throwable.class, () -> {
            Calculator.calculate(0,0,0,0);
        });
    }
}

Answer 1

这里的误解似乎在于 JUnit 实际可以看到什么。

JUnit 并不神奇:它只是普通的旧 Java。它看不到你的方法内部，看不到它们在做什么。它所能看到的就是任何其他代码在执行方法时所能看到的:返回值和未捕获的异常(以及该方法的任何副作用，如果它们对调用代码可见的话)。

从调用者的角度来看，您的方法不会抛出异常:在内部，它会抛出异常，但会捕获并处理它。

如果您希望 JUnit 测试是否抛出了异常，则不需要捕获该异常。

抛出异常然后自己捕获并处理它从来都不是 (*) 正确的做法。重点是什么？你可以简单地做你做的事情来处理它，而不会抛出异常。抛出异常代价高昂，因为需要捕获整个堆栈跟踪。

Throwable 永远 (*) 不是正确的抛出异常。它是返回 Object 的异常“等效”:它不向调用者传达有关异常的类型信息，调用者要么必须做大量工作来尝试处理它；要么或者，更现实地说，应该自己传播它。 IllegalArgumentException 是在此处抛出的正确异常，如果您确实需要抛出(而不是捕获)异常。

Throwable 很少是正确的捕捉对象。 Throwable 是 Exception 和 Error 的父类(super class)型，因此您可能会无意中捕获到 Error，例如 OutOfMemoryError，它不应该被捕获，因为除了让你的程序崩溃之外没有什么合理的做法。尽可能捕获最具体的类型；这也意味着您应该尽可能抛出最具体的类型(或者至少是适合抽象的类型)。

---

(*) 这是“从不”，如“好的，在有限的情况下它可能是合适的”。但除非您了解这些是什么，否则不要这样做。