You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ra...@apache.org on 2009/08/08 05:29:53 UTC

svn commit: r802292 - in /commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl: AssignTest.java UnifiedJEXLTest.java

Author: rahul
Date: Sat Aug  8 03:29:42 2009
New Revision: 802292

URL: http://svn.apache.org/viewvc?rev=802292&view=rev
Log:
Four unused variables in tests -- remove three, use one.

Modified:
    commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/AssignTest.java
    commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/UnifiedJEXLTest.java

Modified: commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/AssignTest.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/AssignTest.java?rev=802292&r1=802291&r2=802292&view=diff
==============================================================================
--- commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/AssignTest.java (original)
+++ commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/AssignTest.java Sat Aug  8 03:29:42 2009
@@ -109,6 +109,7 @@
         }
         catch(RuntimeException xrt) {
             String str = xrt.toString();
+            assertTrue(str.contains("nosuchbean"));
             return;
         }
         finally {

Modified: commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/UnifiedJEXLTest.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/UnifiedJEXLTest.java?rev=802292&r1=802291&r2=802292&view=diff
==============================================================================
--- commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/UnifiedJEXLTest.java (original)
+++ commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl/UnifiedJEXLTest.java Sat Aug  8 03:29:42 2009
@@ -174,7 +174,7 @@
         try {
             UnifiedJEXL.Expression expr = EL.parse("${'world'");
             JexlContext none = null;
-            Object o = expr.evaluate(none);
+            expr.evaluate(none);
             fail("should be malformed");
         }
         catch(UnifiedJEXL.Exception xjexl) {
@@ -188,7 +188,7 @@
         try {
             UnifiedJEXL.Expression expr = EL.parse("#{${hi} world}");
             JexlContext none = null;
-            Object o = expr.evaluate(none);
+            expr.evaluate(none);
             fail("should be malformed");
         }
         catch(UnifiedJEXL.Exception xjexl) {
@@ -202,7 +202,7 @@
         try {
             UnifiedJEXL.Expression expr = EL.parse("#{${hi}+'.world'}");
             JexlContext none = null;
-            Object o = expr.evaluate(none);
+            expr.evaluate(none);
             fail("should be malformed");
         }
         catch(UnifiedJEXL.Exception xjexl) {