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

[tv...@apache.org]

Author: tv
Date: Tue Apr  8 01:48:34 2008
New Revision: 645794

URL: http://svn.apache.org/viewvc?rev=645794&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=645794&r1=645793&r2=645794&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 01:48:34 2008
@@ -571,7 +571,7 @@
     {
         // no param
         BigDecimal result = parser.getBigDecimal("invalid");
-        assertEquals(new BigDecimal(0), result);
+        assertNull(result);
 
         // default
         result = parser.getBigDecimal("default", new BigDecimal(3));