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 8495eca6167668e36fac77bee5345053e92e9d8b Author: Gary Gregory <[email protected]> AuthorDate: Sat Jun 1 14:53:29 2024 -0400 Use assertThrows() --- src/test/java/org/apache/commons/jexl3/FeaturesTest.java | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/src/test/java/org/apache/commons/jexl3/FeaturesTest.java b/src/test/java/org/apache/commons/jexl3/FeaturesTest.java index c211d9e3..99760669 100644 --- a/src/test/java/org/apache/commons/jexl3/FeaturesTest.java +++ b/src/test/java/org/apache/commons/jexl3/FeaturesTest.java @@ -63,13 +63,9 @@ public class FeaturesTest extends JexlTestCase { for (final String script : scripts) { final JexlScript ctl = JEXL.createScript(script); assertNotNull(ctl); - try { - final JexlScript e = jexl.createScript(features, null, script); - fail("should fail parse: " + script); - } catch (final JexlException.Parsing xfeature) { - final String msg = xfeature.getMessage(); - assertNotNull(msg); - } + final JexlException.Parsing xfeature = assertThrows(JexlException.Parsing.class, () -> jexl.createScript(features, null, script)); + final String msg = xfeature.getMessage(); + assertNotNull(msg); } }
