在 JUnit 测试中尝试 catch

Question

我正在为一个已经存在很长时间的应用程序编写单元测试。我需要测试的一些方法是这样构建的：

public void someMethod() throws Exception { 
   //do something 
}

如果我想测试这些方法，我必须在我的单元测试中写这样的东西：

@Test
public void someTest() {
   try {
      someMethod();
   }
   catch (Exception e) {
      e.printStackTrace();
   }
}

这样做是个好习惯吗？还是有其他方法可以测试这些方法？

我在互联网上进行了一些研究，发现了一些带有@Rule 注释和@Test(expected=Exception.class) 的解决方案，但这不起作用（Eclipse 一直在测试中将someMethod() 行显示为错误）。 我不知道这些是否是好的解决方案，因为我对整个单元测试故事还很陌生。

如果有对此了解很多的人可以帮助我，我将非常感激。

Answer 1

由于Exception 是一个检查异常，您可以：

• 必须在try...catch 语句中捕获异常，或者
• 声明要在方法本身中抛出的异常。

你在那里工作正常，但我个人的偏好是声明要抛出的异常。这样，如果在测试运行期间抛出了我不期望的异常，测试将失败。

@Test
public void someTest() throws Exception {
    // dodgy code here
}

如果我们需要查看是否抛出了特定的异常，那么您可以选择使用@Rule 或直接将值添加到@Test 注释中。

@Test(expected = FileNotFoundException.class)
public void someTest() throws Exception {
    // dodgy code here
}

在 JUnit 5 中，您可以利用 Assertions.assertThrows 来完成同样的事情。我对这个整体不太熟悉，因为在编辑时它还不是 GA，但它似乎接受来自 JUnit 5 的 Executable。

@Test
public void someTest() {
    assertThrows(FileNotFoundException.class, () ->
         { dodgyService.breakableMethod() };
}

Answer 2

@Test
public void someTest() {
   try {
     someMethod();
   }
   catch (Exception e) {
     Assert.fail("Exception " + e);
   }
}

如果不应该发生异常，您可以做什么。另一种方法是在签名中抛出异常，如下所示：

@Test
public void someTest() throws Exception {
     someMethod();
}

不同之处在于，在一种情况下，测试会因断言异常而失败，而在另一种情况下，测试会因为测试崩溃而失败。 （就像在你的代码中的某个地方你会得到一个 NPE 并且测试会因此而发生）

之所以必须这样做，是因为 Exception 是一个已检查的异常。见Checked versus unchecked exception

@Test(expected=Exception.class) 用于测试，用于测试是否会抛出异常。

@Test(expected=ArrayIndexOutOfBounds.class)
public void testIndex() {
   int[] array = new int[0];
   int var = array[0]; //exception will be thrown here, but test will be green, because we expect this exception

}

Answer 3

不要在测试代码中捕获应用程序的异常。相反，声明它向上抛出。

因为，当 JUnit 的 TestRunner 发现抛出异常时，它会自动将其记录为 error 用于测试用例。

仅当您 testcase 期望该方法应抛出 Exception 时，您才应使用 @Test(expected=Exception.class) 或捕获异常。

在其他情况下，只需将其向上扔，

public void someTest() throws Exception {

Answer 4

您可以在测试方法签名中添加异常。然后，如果你正在测试是否抛出异常，你必须使用@Test(expected=Exception.class)。在没有抛出异常的测试用例中，测试将成功通过。

@Test
public void testCaseWhereExceptionWontBeThrown() throws Exception {
    someMethod(); //Test pass
}

@Test(expected = Exception.class)
public void testCaseWhereExceptionWillBeThrown() throws Exception {
    someMethod(); //Test pass
}

Answer 5

关于如何在 Junit 测试人员处处理异常有两个主要规则：

1. 如果异常源自测试代码：

   • 如果需要，请在Test 注释的expected 属性中声明它。或者，如果应该对异常对象本身进行进一步检查，则捕获它并忽略它。 （在这种情况下，还必须在 try 块的末尾调用 Assert.fail，以表明没有产生预期的异常。
   • 如果不是预期的，捕获它并执行 Assert.fail。 （之前拨打Exception.printStackTrace 也很有用）。
1234563 em>)，在throws 子句处重新抛出它。

为什么您应该期待测试器中出现异常？提醒：您应该为测试代码上的每个可能结果编写一种测试方法（以实现高代码覆盖率）：在您的情况下，一种方法必须成功返回，并且至少另一种方法必须产生异常。

Answer 6

关于JUnit的三点：

• 测试应该是精确的，它们应该完全根据测试输入的设置方式明确地通过或失败。

• 测试应该将失败报告回框架。

• 测试不应依赖于读取其输出。

您的示例在所有三个方面都失败了。无论是否抛出异常，测试仍然通过。如果抛出异常，JUnit 永远不会发现它并且不能将其包含在测试结果中。知道出了什么问题的唯一方法是读取测试写入标准输出的内容，这使得错误太容易被忽略。这不是编写测试的有用方法。

JUnit 旨在让做正确的事情变得容易，并为开发人员提供有用的反馈。如果从测试方法中抛出异常，它会被框架捕获。如果测试带有异常注释，表明该异常是预期的，则框架将测试标记为通过。否则，框架无法通过测试并记录堆栈跟踪以进行报告。该框架会报告哪些断言失败以及发生了哪些意外异常，以便每个人都知道测试是否有效。

如果您希望测试成功而不会引发异常，那么如果测试中的任何内容都可以引发已检查异常，请将throws Exception 添加到测试方法签名中。在签名中添加throws 并不是说​​方法必须抛出任何东西，它只是让任何发生的异常都被抛出，以便测试框架可以捕获它们。

您在测试中实际捕获异常的唯一实例是您要测试有关异常的断言；例如，您可以测试有关异常的消息是否符合您的预期，或者异常是否设置了原因。在这种情况下，您可以在 try-block 的末尾添加 Assert.fail()，这样如果不抛出异常就会导致测试失败。

不是 try-catch 块那么糟糕，而是缺少任何会导致测试失败的东西。

当你一开始写一个测试时，让它失败。这样你就可以向自己证明你知道测试在做什么，并且你确认，当失败时，你会意识到它。

Answer 7

这是什么异常？是吗

1. 使用单元测试中不会发生的流之类的异常或
2. 可能由于某种错误输入而发生的异常？

如果是 1。我会把它放在方法签名级别，因为 try-catch 除了仪式之外没有任何实际用途。

@Test
public void testFoo() throws Exception {
    // ...
}

如果是 2. 它会变得有点复杂。如果抛出异常，您需要问自己应该发生什么。测试应该失败吗？是预期的吗？这无关紧要吗？以下示例说明了如何处理所有这些。 注意：我只使用了异常，因为你使用了。我希望它真的不是，因为如果有可能抛出一些 other 异常而不是预期的那么这些将非常不稳定。如果可能的话不要使用Exception，使用更具体的东西（在junit 和代码中）。

// The below code assumes you've imported the org.junit.Assert class.

@Test
public void thisShouldFailIfExceptionCaught() {
    //Given...
    try {
        // When...
    } catch (Exception e) {
        Assert.fail();
    }
    // Then...
}

@Test
public void thisShouldPassOnlyIfTheExceptionIsCaught() {
    //Given...
    try {
        // When...
        Assert.fail();
    } catch (Exception expected) {}
    // No "then" needed, the fact that it didn't fail is enough.
}

@Test
public void irrelevantExceptionThatCouldBeThrown() {
    //Given...
    try {
        // When...
    } catch (Exception e) {}
    // Then...
}