Simplified test

cbeust · Dec 17, 2023 · a73c132 · a73c132
1 parent 086d7ba
commit a73c132
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/src/test/java/com/beust/jcommander/MethodSetterTest.java b/src/test/java/com/beust/jcommander/MethodSetterTest.java
@@ -56,22 +56,21 @@ public void setHost(String host) {
     Assert.assertTrue(passed, "Should have thrown an exception");
   }
 
+  @Test(expectedExceptions = ParameterException.class)
   public void setterThatThrowsKeepsOriginalException() {
     class Arg {
       @Parameter(names = "--host")
       public void setHost(String host) {
         throw new IllegalArgumentException("Illegal host");
       }
     }
-    boolean passed = false;
     try {
       JCommander.newBuilder().addObject(new Arg()).build().parse("--host", "host");
-    } catch(ParameterException ex) {
+    } catch (ParameterException ex) {
       Assert.assertEquals(ex.getCause().getClass(), IllegalArgumentException.class);
       Assert.assertEquals(ex.getCause().getMessage(), "Illegal host");
-      passed = true;
+      throw ex;
     }
-    Assert.assertTrue(passed, "Should have thrown an exception");
   }
 
   public void getterReturningNonString() {