You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@turbine.apache.org by tv...@apache.org on 2008/04/08 11:08:06 UTC

svn commit: r645801 - /turbine/core/branches/TURBINE_2_3_BRANCH/src/test/org/apache/turbine/util/BaseValueParserTest.java

Author: tv
Date: Tue Apr  8 02:08:05 2008
New Revision: 645801

URL: http://svn.apache.org/viewvc?rev=645801&view=rev
Log:
Important change: The BaseValueParser now consistently returns null for 
all get-methods that return an Object, if the parameter name does not exist. This changes the behaviour of getBigDecimal which used to return 0 in this case. Changed the test to reflect this new behaviour.

Modified:
    turbine/core/branches/TURBINE_2_3_BRANCH/src/test/org/apache/turbine/util/BaseValueParserTest.java

Modified: turbine/core/branches/TURBINE_2_3_BRANCH/src/test/org/apache/turbine/util/BaseValueParserTest.java
URL: http://svn.apache.org/viewvc/turbine/core/branches/TURBINE_2_3_BRANCH/src/test/org/apache/turbine/util/BaseValueParserTest.java?rev=645801&r1=645800&r2=645801&view=diff
==============================================================================
--- turbine/core/branches/TURBINE_2_3_BRANCH/src/test/org/apache/turbine/util/BaseValueParserTest.java (original)
+++ turbine/core/branches/TURBINE_2_3_BRANCH/src/test/org/apache/turbine/util/BaseValueParserTest.java Tue Apr  8 02:08:05 2008
@@ -585,7 +585,7 @@
         // unparsable value
         parser.add("unparsable", "a");
         result = parser.getBigDecimal("unparsable");
-        assertEquals(new BigDecimal(0), result);
+        assertNull(result);
 
         // array
         parser.add("array", "1");