哪个更好，ExpectedException 或 @Test(expected= [关闭]

Question

我有代码可以检查 jUnit 中的异常。我想知道以下哪一项是好的 jUnit 实践？

第一

@Rule
public ExpectedException exception = ExpectedException.none();

@Test
public void checkNullObject() throws CustomException {
    exception.expect(CustomException.class);
    MyClass myClass= null;
    MyCustomClass.get(null);
}

第二

@Test(expected=CustomException.class)
public void checkNullObject() throws CustomException {
    MyClass myClass= null;
    MyCustomClass.get(null);    
}

编辑：请注意 CustomException 是未经检查的自定义异常。 （虽然不会对这个问题产生任何影响）。

Answer 1

这取决于您要检查异常的内容。如果您所做的只是检查是否引发了异常，那么使用@Test(expected=...) 可能是最简单的方法：

@Test(expected=CustomException.class)
public void checkNullObject() throws CustomException {
  MyClass myClass= null;
  MyCustomClass.get(null);
}

但是，@Rule ExpectedException 有更多选项，包括检查消息，来自javadoc：

// These tests all pass.
public static class HasExpectedException {
    @Rule
    public ExpectedException thrown= ExpectedException.none();

    @Test
    public void throwsNothing() {
        // no exception expected, none thrown: passes.
    }

    @Test
    public void throwsNullPointerException() {
        thrown.expect(NullPointerException.class);
        throw new NullPointerException();
    }

    @Test
    public void throwsNullPointerExceptionWithMessage() {
        thrown.expect(NullPointerException.class);
        thrown.expectMessage("happened?");
        thrown.expectMessage(startsWith("What"));
        throw new NullPointerException("What happened?");
    }

    @Test
    public void throwsIllegalArgumentExceptionWithMessageAndCause() {
        NullPointerException expectedCause = new NullPointerException();
        thrown.expect(IllegalArgumentException.class);
        thrown.expectMessage("What");
        thrown.expectCause(is(expectedCause));
        throw new IllegalArgumentException("What happened?", cause);
    }
}

因此您可以检查消息，即异常的原始原因。检查消息，你可以使用匹配器，所以你可以检查startsWith()和类似的。

使用旧式 (Junit 3) throw/catch 的一个原因是如果您有特定要求。这些并不多，但它可能会发生：

@Test
public void testMe() {
    try {
        Integer.parseInt("foobar");
        fail("expected Exception here");
    } catch (Exception e) {
        // OK
    }
}

Answer 2

第二个版本绝对是标准的做法。在 Junit 4 之前，老式的做法是这样的：

try {
    codeThatShouldThrowException();
    fail("should throw exception");
} catch (ExpectedException expected) {
    //Expected
}

有时您可能希望恢复到这种方式，例如，如果您想对异常中的消息进行断言。