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 446cd367fb3d25e7a4eac72bd4dc5b38b254375c Author: Gary Gregory <[email protected]> AuthorDate: Sat Jun 1 15:36:04 2024 -0400 Use assertThrows() --- src/test/java/org/apache/commons/jexl3/Issues300Test.java | 8 ++------ 1 file changed, 2 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 39b2b3d0..59f59706 100644 --- a/src/test/java/org/apache/commons/jexl3/Issues300Test.java +++ b/src/test/java/org/apache/commons/jexl3/Issues300Test.java @@ -662,12 +662,8 @@ public class Issues300Test { assertEquals(42, expr0.evaluate(null)); final String parsed = expr0.getParsedText(); assertEquals(src42, parsed); - try { - final JexlExpression expr = jexl.createExpression("new()"); - fail("should not parse"); - } catch (final JexlException.Parsing xparse) { - assertTrue(xparse.toString().contains(")")); - } + final JexlException.Parsing xparse = assertThrows(JexlException.Parsing.class, () -> jexl.createExpression("new()"), "should not parse"); + assertTrue(xparse.toString().contains(")")); } @Test
