尝试 Catch 块有效但测试 assertThrows 失败(Junit 5)

Try Catch block works but test assertThrows fail (Junit 5)

我正在尝试学习本教程

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

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

当我 运行 测试异常消息被打印出来 "Undefined Math Expression" 但我的 IDE 说 "Expected java.lang.Throwable to be thrown, but nothing was thrown." 我认为我的代码范围存在一些问题?我是新手,请多多关照。我在下面提供了代码和测试:

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);
        });
    }
}

Throwabletry catch 块捕获,因此 Junit 无法访问它。尝试删除 try catch 块。

你实际上并没有抛出异常,你正在捕获它。为此,您应该删除 try catch 块。

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

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

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

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

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

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

Throwable 很少是正确的捕捉对象。 ThrowableExceptionError 的超类型,因此您可能会无意中捕捉到 Error,例如 OutOfMemoryError,它不应该被捕捉,因为有除了让你的程序崩溃之外,没有什么可以做的。尽可能抓住最具体的类型;这也意味着您应该尽可能抛出最具体的类型(或者至少是适合抽象的类型)。


(*) 这是 "never",如 "ok, there are a limited number of circumstances where it may be appropriate"。但是除非你明白这些是什么,否则不要。