Try Catch block工作但test assertThrows失败(JUnit5)

Question

我正在尝试学习这篇教程JUnit 5：如何断言一个异常被抛出？

我使用Java10、IntelliJ2018和JUnit5。

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。它不能看到你的方法内部来看到它们在做什么。它所能看到的就是任何其他代码在执行方法时所能看到的：返回值和未捕获的异常（以及该方法的任何副作用，如果它们对调用代码可见的话）。

这里的方法不是从调用方的角度抛出异常：在内部，它抛出了异常，但它捕获并处理它。

抛出一个异常然后自己捕获并处理它从来不是(*)正确的做法。有什么意义？您可以简单地做您所做的事情来处理它，而不引发异常。抛出异常的代价很高，因为需要捕获整个堆栈跟踪。

throwable从来不是(*)要抛出的正确异常。它是返回object的异常“等价物”：它不向调用方传递关于异常的类型信息，然后调用方要么必须做大量工作来尝试处理它；或者，更现实地说，应该自己传播。IllegalArgumentException是在这里抛出的正确异常，如果您实际上需要抛出（而不是捕获）一个异常。

throwable很少是正确的捕捉对象。throwable是exception和error的超级类型，因此您可能会无意中捕获到error（如outofmemoryerror)，它不应该被捕获，因为除了使程序崩溃之外，没有任何合理的操作可做。抓住你能抓住的最具体的类型；这也意味着您应该抛出您所能抛出的最具体的类型（或者，至少是一个适合于抽象的类型）。

(*)这与“OK,在有限的几种情况下可能是适当的”中的“绝不”相同。但除非你明白这些是什么，否则不要。