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

Throwable被try catch块捕获，因此Junit无法访问它。尝试删除try catch块。

Answer 2

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

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

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

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

永远不要（*）做一个异常，以引发异常然后自己捕获并处理它。重点是什么？您可以简单地做您要处理的事情，而不会引发异常。抛出异常是昂贵的，因为需要捕获整个堆栈跟踪。

Throwable永远都不（*）引发异常。这是返回Object的异常“等效”：它没有将有关异常的类型信息传达给调用方，调用方然后必须做大量工作来尝试处理该异常。或者，更现实的是，应该自己传播它。如果您确实需要抛出（而不是捕获）异常，那么IllegalArgumentException是正确的例外。

Throwable很少是正确的选择。 Throwable是Exception和Error的超类型，因此您可能无意中捕获了Error之类的OutOfMemoryError，例如find ~/dev/test -type d -mtime +7 -exec rm -rf {} \; find ~/dev/test -type f ( -name ".gz" -o -name ".tar" -o -name "*.zip" ) -mtime +7 -exec rm -rf {} +，因为存在除了使程序崩溃以外，没有其他合理的操作。赶上最具体的类型；这也意味着您应该抛出最具体的类型（或者至少是适合抽象的类型）。

（*）这是“从不”，如“好，在少数情况下，它是适当的”。但是除非您了解这些内容，否则请不要。

Answer 3

您实际上并没有引发异常，而是在捕获异常。为此，您应该删除try catch块。

尝试Catch块工作，但测试assertThrows失败（Junit 5）

3 个答案: