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 7e62b06bdcf387b53db6d8953fb5c7450f8595f2 Author: Gary Gregory <[email protected]> AuthorDate: Sat Jun 1 15:25:13 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 801c934c..2b30bab5 100644 --- a/src/test/java/org/apache/commons/jexl3/Issues300Test.java +++ b/src/test/java/org/apache/commons/jexl3/Issues300Test.java @@ -432,21 +432,19 @@ public class Issues300Test { @Test public void test309b() { + // @formatter:off final String src = "<html lang=\"en\">\n" + " <body>\n" + " <h1>Hello World!</h1>\n" + "$$ var i = a b c;\n" + " </body>\n" + "</html>"; + // @formatter:on final JexlEngine jexl = new JexlBuilder().safe(true).create(); final JxltEngine jxlt = jexl.createJxltEngine(); final JexlInfo info = new JexlInfo("template", 1, 1); - try { - final JxltEngine.Template tmplt = jxlt.createTemplate(info, src); - fail("shoud have thrown exception"); - } catch (final JexlException.Parsing xerror) { - assertEquals(4, xerror.getInfo().getLine()); - } + final JexlException.Parsing xerror = assertThrows(JexlException.Parsing.class, () -> jxlt.createTemplate(info, src)); + assertEquals(4, xerror.getInfo().getLine()); } @Test
