This is an automated email from the ASF dual-hosted git repository. ggregory pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-jexl.git
commit 77c73fb7462c09a6c6a5880ba65f98d252f64971 Author: Gary Gregory <[email protected]> AuthorDate: Sat Jun 1 15:58:50 2024 -0400 Use assertThrows() --- src/test/java/org/apache/commons/jexl3/Issues300Test.java | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/src/test/java/org/apache/commons/jexl3/Issues300Test.java b/src/test/java/org/apache/commons/jexl3/Issues300Test.java index 3e94c655..2c704aab 100644 --- a/src/test/java/org/apache/commons/jexl3/Issues300Test.java +++ b/src/test/java/org/apache/commons/jexl3/Issues300Test.java @@ -1044,6 +1044,7 @@ public class Issues300Test { @Test public void test393() { + // @formatter:off final String src = "const total = 0;\n" + "if (true) {\n" + " total = 1;\n" + @@ -1053,12 +1054,9 @@ public class Issues300Test { .safe(false) .strict(true) .create(); - try { - final JexlScript script = jexl.createScript(src); - fail("should fail on const total assignment"); - } catch (final JexlException.Parsing xparse) { - assertTrue(xparse.getMessage().contains("total")); - } + // @formatter:on + final JexlException.Parsing xparse = assertThrows(JexlException.Parsing.class, () -> jexl.createScript(src), "should fail on const total assignment"); + assertTrue(xparse.getMessage().contains("total")); } @Test
