JUnit4 @Test(expected=MyException.class) VS try/catch

Question

我正在考虑异常处理和单元测试最佳实践，因为我们正在尝试制定一些代码最佳实践。

在我们公司的 wiki 上找到的一篇关于最佳实践的文章指出“不要使用 try/catch，而是使用 Junit4 @Test(expect=MyException.class)”，但没有进一步的信息。我不相信。

我们的许多自定义异常都有一个枚举，以便识别失败原因。 因此，我宁愿看到像 : 这样的测试

@Test
public void testDoSomethingFailsBecauseZzz() {
try{
   doSomething();
} catch(OurCustomException e){
   assertEquals("Omg it failed, but not like we planned", FailureEnum.ZZZ, e.getFailure());
}
}

，而不是 :

@Test(expected = OurCustomException.class)
public void testDoSomethingFailsBecauseZzz() {
   doSomething();
}

当 doSomethig() 看起来像 :

public void doSomething throws OurCustomException {
  if(Aaa) {
     throw OurCustomException(FailureEnum.AAA);
  } 
  if(Zzz) {
     throw OurCustomException(FailureEnum.ZZZ);
  }
  // ...
}

时，我非常确信在某些情况下 @Test(expected=blabla.class) 是最好的选择（对于例如，当异常是精确的并且毫无疑问导致它的原因时）。

我在这里遗漏了一些东西还是应该在必要时使用 try/catch ？

Answer 1

听起来您的枚举被用作异常层次结构的替代品？也许如果你有一个异常层次结构，@Test(expected=XYZ.class)会变得更有用？

Answer 2

• 如果您只想检查是否引发了某种类型的异常，请使用注释的 expected 属性。
• 如果您想检查抛出的异常的属性（例如消息或自定义成员值），请在测试中捕获它并进行断言。

就您而言，您似乎想要后者（断言异常具有特定的 FailureEnum 值）； 使用try/catch没有任何问题。

你应该“不使用 try/catch”（解释为“从不”）的概括是废话。

杰夫是对的；您的异常层次结构的组织是可疑的。不过，你似乎也认识到这一点。 :)

Answer 3

如果您想检查原始异常类型，那么 expected 方法是合适的。否则，如果您需要测试有关异常的某些内容（并且不管测试消息内容的enum怪异情况如何），您可以执行 try catch，但这有点老派。新的 JUnit 方法是使用 MethodRule。 API 中的异常 (ExpectedException) 专门用于测试消息，但您可以轻松查看代码并调整该实现来检查失败的 enum。

Answer 4

在您的特殊情况下，您需要测试 (1) 是否引发预期的异常 type 以及 (2) 错误号是否正确，因为该方法可能会引发不同类型的相同异常。

这需要检查异常对象。但是，您可以坚持建议并验证是否引发了正确的异常：

@Test(expected = OurCustomException.class)
public void testDoSomethingFailsBecauseZzz() {
   try {
      doSomething();
   } catch (OurCustomException e) {
      if (e.getFailureEnum.equals(FailureEnum.ZZZ))  // use *your* method here
         throw e;

      fail("Catched OurCostomException with unexpected failure number: " 
        + e.getFailureEnum().getValue());  // again: your enum method here
   }
}

此模式将吃掉意外的异常并使测试失败。 Strike>

编辑

更改它是因为我错过了显而易见的事情：我们可以使测试用例失败并捕获消息。现在：如果抛出预期的异常以及预期的错误代码，则测试通过。如果测试因意外错误而失败，那么我们可以读取错误代码。

Answer 5

我在搜索如何处理异常时遇到了这个。

正如 @Yishai 提到的，预期异常的首选方法是使用 JUnit 规则和 ExpectedException。

使用 @Test(expected=SomeException.class) 时，如果在方法中的任何位置抛出异常，则测试方法将通过。

当您使用ExpectedException时：

@Test
public void testException()
{
    // If SomeException is thrown here, the test will fail.
    expectedException.expect(SomeException.class);
    // If SomeException is thrown here, the test will pass.
}

您还可以测试：

• 预期消息：ExpectedException.expectMessage();
• 预期原因：expectedException.expectCause()。

附带说明：我不认为使用枚举来表示异常消息/原因是一个好的做法。 （如果我错了，请纠正我。）

Answer 6

我做了 catch-exception 因为我遇到了和你一样的问题，Stph。
使用 catch-exception 您的代码可能如下所示：

@Test
public void testDoSomethingFailsBecauseZzz() {
   verifyException(myObj, OurCustomException.class).doSomething();
   assertEquals("Omg it failed, but not like we planned", FailureEnum.ZZZ,    
               ((OurCustomException)caughtException()).getFailure() ;
}