svn commit: r922755 - in /ofbiz/trunk/framework/base/src/org/ofbiz/base: conversion/test/MiscTests.java json/test/JSONTests.java

[doogie-3]

Author: doogie
Date: Sun Mar 14 03:54:53 2010
New Revision: 922755

URL: http://svn.apache.org/viewvc?rev=922755&view=rev
Log:
Fixed coverage of the test cases.

Modified:
    ofbiz/trunk/framework/base/src/org/ofbiz/base/conversion/test/MiscTests.java
    ofbiz/trunk/framework/base/src/org/ofbiz/base/json/test/JSONTests.java

Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/conversion/test/MiscTests.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/conversion/test/MiscTests.java?rev=922755&r1=922754&r2=922755&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/org/ofbiz/base/conversion/test/MiscTests.java (original)
+++ ofbiz/trunk/framework/base/src/org/ofbiz/base/conversion/test/MiscTests.java Sun Mar 14 03:54:53 2010
@@ -76,7 +76,7 @@ public class MiscTests extends GenericTe
         Converter<S, ? super S> converter = Converters.getConverter(sourceClass, targetClass);
         Object result = converter.convert(UtilGenerics.<S>cast(wanted));
         assertEquals("pass thru convert", wanted, result);
-        assertTrue("pass thru exact equals", wanted == result);
+        assertSame("pass thru exact equals", wanted, result);
         assertTrue("pass thru can convert wanted", converter.canConvert(wanted.getClass(), targetClass));
         assertTrue("pass thru can convert source", converter.canConvert(sourceClass, targetClass));
         assertEquals("pass thru source class", wanted.getClass(), converter.getSourceClass());

Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/json/test/JSONTests.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/json/test/JSONTests.java?rev=922755&r1=922754&r2=922755&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/org/ofbiz/base/json/test/JSONTests.java (original)
+++ ofbiz/trunk/framework/base/src/org/ofbiz/base/json/test/JSONTests.java Sun Mar 14 03:54:53 2010
@@ -186,54 +186,35 @@ public class JSONTests extends GenericTe
     }
 
     public void testParseErrors() throws Exception {
-        for (char c = 1; c < 1024; c++) {
-            switch (c) {
-                case '[':
-                    doParseExceptionTest("[:", JSONConstants.KEY_SEP);
-                    break;
-                case ']':
-                    doParseExceptionTest("]", JSONConstants.ARRAY_END);
-                    break;
-                case '{':
-                    doParseExceptionTest("{:", JSONConstants.KEY_SEP);
-                    break;
-                case '}':
-                    doParseExceptionTest("}", JSONConstants.OBJECT_END);
-                    break;
-                case ':':
-                    doParseExceptionTest(":", JSONConstants.KEY_SEP);
-                    break;
-                case ',':
-                    doParseExceptionTest(",", JSONConstants.ITEM_SEP);
-                    break;
-                case '"':
-                    doParseExceptionTest("\"", JSONConstants.EOF);
-                    break;
-                case 't':
-                    doParseExceptionTest("true:", JSONConstants.KEY_SEP);
-                    break;
-                case 'f':
-                    doParseExceptionTest("false:", JSONConstants.KEY_SEP);
-                    break;
-                case 'n':
-                    doParseExceptionTest("null:", JSONConstants.KEY_SEP);
-                    break;
-                case '-':   // numbers
-                case '.':
-                case '0': case '1': case '2': case '3': case '4':
-                case '5': case '6': case '7': case '8': case '9':
-                    break;
-                case '\t':
-                    doWhitespaceExceptionTest(Character.toString(c), 8);
-                    break;
-                case '\n':
-                case '\r':
-                case ' ':
-                    doWhitespaceExceptionTest(Character.toString(c), 1);
-                    break;
-                default:
-                    doTokenMgrErrorTest(c);
-                    break;
+        for (char c = 1; c < 128; c++) {
+            if (c == '\t') {
+                doWhitespaceExceptionTest(Character.toString(c), 8);
+            } else if (c == '\n' || c == '\r' || c == ' ') {
+                doWhitespaceExceptionTest(Character.toString(c), 1);
+            } else if (c == '"') {
+                doParseExceptionTest("\"", JSONConstants.EOF);
+            } else if (c == ',') {
+                doParseExceptionTest(",", JSONConstants.ITEM_SEP);
+            } else if (c == '-' || c == '.' || (c >= '0' && c <= '9')) {
+                // numbers
+            } else if (c == ':') {
+                doParseExceptionTest(":", JSONConstants.KEY_SEP);
+            } else if (c == '[') {
+                doParseExceptionTest("[:", JSONConstants.KEY_SEP);
+            } else if (c == ']') {
+                doParseExceptionTest("]", JSONConstants.ARRAY_END);
+            } else if (c == 't') {
+                doParseExceptionTest("true:", JSONConstants.KEY_SEP);
+            } else if (c == 'f') {
+                doParseExceptionTest("false:", JSONConstants.KEY_SEP);
+            } else if (c == 'n') {
+                doParseExceptionTest("null:", JSONConstants.KEY_SEP);
+            } else if (c == '{') {
+                doParseExceptionTest("{:", JSONConstants.KEY_SEP);
+            } else if (c == '}') {
+                doParseExceptionTest("}", JSONConstants.OBJECT_END);
+            } else {
+                doTokenMgrErrorTest(c);
             }
         }
     }