You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2013/06/02 00:05:27 UTC
svn commit: r1488620 -
/commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl2/ArithmeticTest.java
Author: sebb
Date: Sat Jun 1 22:05:27 2013
New Revision: 1488620
URL: http://svn.apache.org/r1488620
Log:
There's no reason to change the default leniency here; just change our engine
This should fix issues with leaving a stale value for other tests
Modified:
commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl2/ArithmeticTest.java
Modified: commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl2/ArithmeticTest.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl2/ArithmeticTest.java?rev=1488620&r1=1488619&r2=1488620&view=diff
==============================================================================
--- commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl2/ArithmeticTest.java (original)
+++ commons/proper/jexl/branches/2.0/src/test/java/org/apache/commons/jexl2/ArithmeticTest.java Sat Jun 1 22:05:27 2013
@@ -312,7 +312,8 @@ public class ArithmeticTest extends Jexl
jexl.setSilent(false);
// for non-silent, silent...
for (int s = 0; s < 2; ++s) {
- JexlThreadedArithmetic.setLenient(Boolean.valueOf(s == 0));
+ final boolean lenient = s == 0;
+ jexl.setLenient(s == 0);
int zthrow = 0;
int zeval = 0;
// for vars of all types...
@@ -337,7 +338,7 @@ public class ArithmeticTest extends Jexl
}
}
}
- if (!jexl.isLenient()) {
+ if (!lenient) {
assertTrue("All expressions should have thrown " + zthrow + "/" + PERMS,
zthrow == PERMS);
} else {