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

球队

我正在尝试遵循本教程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;
    }
}

考试:

class CalculatorTest {
    @Test
    void denominatorContainsZero() {
        assertThrows(Throwable.class, () -> {
            Calculator.calculate(0,0,0,0);
        });
    }
}
安迪·特纳(Andy Turner)

这里的误解似乎是JUnit实际可以看到的。

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

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

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

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

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

Throwable很少是正确的事情。Throwable是两者的超ExceptionError,所以你可能会无意中捕获一个Error,像OutOfMemoryError,这不应该被抓,因为没有什么合理的做,除了你的程序崩溃。赶上最具体的类型;这也意味着您应该抛出可能的最具体类型(或者至少是适合抽象的类型)。


(*)这是“从不”,如“好,在有限的情况下可能是适当的”中所述。但是除非您了解这些是什么,否则请不要。

本文收集自互联网,转载请注明来源。

如有侵权,请联系 [email protected] 删除。

编辑于
0

我来说两句

0 条评论
登录 后参与评论

相关文章