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 2d4db523e7d210457b73de21e3b6f6daac3695a9 Author: Gary Gregory <garydgreg...@gmail.com> AuthorDate: Sat Jun 1 18:41:18 2024 -0400 Use assertThrows() --- src/test/java/org/apache/commons/jexl3/LexicalTest.java | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/src/test/java/org/apache/commons/jexl3/LexicalTest.java b/src/test/java/org/apache/commons/jexl3/LexicalTest.java index b8c53877..70b8917b 100644 --- a/src/test/java/org/apache/commons/jexl3/LexicalTest.java +++ b/src/test/java/org/apache/commons/jexl3/LexicalTest.java @@ -572,12 +572,7 @@ public class LexicalTest { f.lexical(true); f.lexicalShade(true); final JexlEngine jexl = createEngine(f); - try { - final JexlScript script = jexl.createScript("for(var x : 1..3) { var c = 0}; return x"); - fail("Should not have been parsed"); - } catch (final JexlException ex) { - // OK - } + assertThrows(JexlException.class, () -> jexl.createScript("for(var x : 1..3) { var c = 0}; return x", "Should not have been parsed")); } @Test