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 d63664ad474572c4c7f91ab220093bc95a99e9c4 Author: Gary Gregory <[email protected]> AuthorDate: Sat Jun 1 13:48:38 2024 -0400 Use assertThrows() --- .../java/org/apache/commons/jexl3/introspection/SandboxTest.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/test/java/org/apache/commons/jexl3/introspection/SandboxTest.java b/src/test/java/org/apache/commons/jexl3/introspection/SandboxTest.java index 288e8006..0dba946f 100644 --- a/src/test/java/org/apache/commons/jexl3/introspection/SandboxTest.java +++ b/src/test/java/org/apache/commons/jexl3/introspection/SandboxTest.java @@ -411,7 +411,7 @@ public class SandboxTest extends JexlTestCase { "foo.callMeNot()", "foo.NONO", "new('org.apache.commons.jexl3.SandboxTest$Foo', 'one', 'two')" }; - // @formatter:off + // @formatter:on final JexlEngine sjexl = new JexlBuilder().strict(true).safe(false).create(); for (final String expr : exprs) { final JexlScript script = sjexl.createScript(expr, "foo"); @@ -598,7 +598,8 @@ public class SandboxTest extends JexlTestCase { // can not write quux, null for (final String k : Arrays.asList("'quux'", "null")) { final JexlExpression expression2 = jexl.createExpression("{" + k + " : 'foo'}[" + k + "] = '42'"); - assertTrue(assertThrows(JexlException.Property.class, () -> expression2.evaluate(null), "should have blocked " + k).getMessage().contains("undefined")); + assertTrue(assertThrows(JexlException.Property.class, () -> expression2.evaluate(null), "should have blocked " + k).getMessage() + .contains("undefined")); } } }
