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

Tea*_*eam 1 java junit5

我正在尝试遵循本教程JUnit 5:如何断言抛出异常?

我使用 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;
    }
}
Run Code Online (Sandbox Code Playgroud)

考试:

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

And*_*ner 7

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

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

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

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

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

Throwable从来都不是 (*) 要抛出的正确异常。这是返回的异常“等效” Object:它不向调用者传达有关异常的类型信息,然后调用者必须做很多工作来尝试处理它;或者,更现实的是,应该自己传播它。IllegalArgumentException是在此处抛出的正确异常,如果您确实需要抛出(而不是捕获)异常。

Throwable很少是正确的捕捉。ThrowableExceptionand的超类型Error,因此您可能会无意中捕获Error, like OutOfMemoryError,它不应该被捕获,因为除了使程序崩溃之外没有其他合理的做法。抓住最具体的类型;这也意味着您应该抛出最具体的类型(或者至少是适合抽象的类型)。


(*) 这是“从不”,如“好吧,在有限的情况下可能合适”。但除非你明白这些是什么,否则不要。


孙兴斌*_*孙兴斌 6

被块Throwable捕获try catch,因此 Junit 无法访问它。尝试移除该try catch块。