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 20:04:33 UTC
(commons-jexl) 06/09: Use assertThrows()
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 77c73fb7462c09a6c6a5880ba65f98d252f64971
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sat Jun 1 15:58:50 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 3e94c655..2c704aab 100644
--- a/src/test/java/org/apache/commons/jexl3/Issues300Test.java
+++ b/src/test/java/org/apache/commons/jexl3/Issues300Test.java
@@ -1044,6 +1044,7 @@ public class Issues300Test {
@Test
public void test393() {
+ // @formatter:off
final String src = "const total = 0;\n" +
"if (true) {\n" +
" total = 1;\n" +
@@ -1053,12 +1054,9 @@ public class Issues300Test {
.safe(false)
.strict(true)
.create();
- try {
- final JexlScript script = jexl.createScript(src);
- fail("should fail on const total assignment");
- } catch (final JexlException.Parsing xparse) {
- assertTrue(xparse.getMessage().contains("total"));
- }
+ // @formatter:on
+ final JexlException.Parsing xparse = assertThrows(JexlException.Parsing.class, () -> jexl.createScript(src), "should fail on const total assignment");
+ assertTrue(xparse.getMessage().contains("total"));
}
@Test