You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2024/06/01 19:23:00 UTC
(commons-jexl) 04/07: Let JUnit handle exception
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 7924558a6b1c956be1a8f73d66469aa14de493af
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sat Jun 1 15:15:58 2024 -0400
Let JUnit handle exception
---
.../java/org/apache/commons/jexl3/Issues200Test.java | 19 +++++--------------
1 file changed, 5 insertions(+), 14 deletions(-)
diff --git a/src/test/java/org/apache/commons/jexl3/Issues200Test.java b/src/test/java/org/apache/commons/jexl3/Issues200Test.java
index 645167be..cb0fb59b 100644
--- a/src/test/java/org/apache/commons/jexl3/Issues200Test.java
+++ b/src/test/java/org/apache/commons/jexl3/Issues200Test.java
@@ -610,18 +610,14 @@ public class Issues200Test extends JexlTestCase {
@Test
public void test275c() throws Exception {
final JexlContext ctxt = new MapContext();
- //ctxt.set("out", System.out);
+ // ctxt.set("out", System.out);
final JexlEngine jexl = new JexlBuilder().strict(true).safe(true).silent(true).create();
JexlScript e;
Object r;
e = jexl.createScript("(s, v)->{ var x = y ; 42; }");
// wont make an error
- try {
- r = e.execute(ctxt, false, true);
- assertEquals(42, r);
- } catch (final JexlException.Variable xjexl) {
- fail("should not have thrown");
- }
+ r = e.execute(ctxt, false, true);
+ assertEquals(42, r);
}
@Test
@@ -629,14 +625,9 @@ public class Issues200Test extends JexlTestCase {
final JexlContext ctxt = new MapContext();
ctxt.set("out", System.out);
final JexlEngine jexl = new JexlBuilder().strict(true).safe(true).create();
-
final JexlScript e = jexl.createScript("{ var xyz = 42 } out.println(xyz)");
- try {
- final Object o = e.execute(ctxt);
- assertNull(o);
- } catch (final JexlException.Variable xvar) {
- fail("should not have thrown" + xvar);
- }
+ final Object o = e.execute(ctxt);
+ assertNull(o);
}
@Test