我正在尝试遵循本教程 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);
});
}
}
这里的误解似乎在于 JUnit 实际上可以看到的内容。
JUnit并不神奇:它只是普通的旧Java。它无法看到您的方法内部以查看它们在做什么。它所能看到的只是任何其他代码在执行方法时可以看到的内容:返回值和未捕获的异常(以及方法的任何副作用,如果它们对调用代码可见(。
从调用方的角度来看,此处的方法不会引发异常:在内部,它会引发异常,但会捕获并处理异常。
如果希望 JUnit 测试是否引发异常,则无需捕获该异常。
抛出异常然后自己捕获并处理它绝不是 (*( 正确的做法。有什么意义?您可以简单地执行处理它的操作,而不会引发异常。由于需要捕获整个堆栈跟踪,因此引发异常的成本很高。
Throwable
从来都不是要抛出的正确异常。它是返回Object
的异常"等效":它不向调用者传达有关异常的类型信息,然后调用者必须做很多工作来尝试处理它;或者,更现实地说,应该自己传播它。IllegalArgumentException
是在这里抛出的正确例外,如果你真的需要抛出(而不是捕获(一个异常。
Throwable
很少是合适的东西。Throwable
是Exception
和Error
的超类型,所以你可能会无意中捕获一个Error
,例如OutOfMemoryError
,不应该捕获,因为除了使程序崩溃之外,没有什么合理的事情可做。尽可能捕捉最具体的类型;这也意味着你应该抛出你能抛出的最具体的类型(或者至少是适合抽象的类型(。
(*( 这是"从不",如"好的,在有限的情况下可能是合适的"。但除非你明白这些是什么,否则不要。
Throwable
被try catch
块捕获,因此 Junit 无法访问它。尝试删除try catch
块。
你实际上并没有抛出异常,你正在捕捉它。为此,您应该删除 try catch 块。