Merge pull request #119 from FasterXML/tatu/2.17/116-chagen-path-serialization

Change post #116 to potentially change serialization of `java.nio.file.Path`

Author: cowtowncoder

jr-objects/src/main/java/com/fasterxml/jackson/jr/ob/impl/JSONWriter.java

@@ -789,7 +789,7 @@ protected String dateToString(Date v) {
      * @since 2.17
      */
     protected String pathToString(Path value) {
-        return value.toUri().toString();
+        return value.toString();
     }
 
     /*

jr-objects/src/main/java/com/fasterxml/jackson/jr/ob/impl/SimpleValueReader.java

@@ -152,9 +152,15 @@ public Object read(JSONReader reader, JsonParser p) throws IOException
             return p.getValueAsLong();
 
         case SER_NUMBER_BIG_DECIMAL:
+            if (p.hasToken(JsonToken.VALUE_NULL)) {
+                return null;
+            }
             return p.getDecimalValue();
 
         case SER_NUMBER_BIG_INTEGER:
+            if (p.hasToken(JsonToken.VALUE_NULL)) {
+                return null;
+            }
             return p.getBigIntegerValue();
 
         // Other scalar types:
@@ -278,7 +284,7 @@ protected Path _readPath(JsonParser p) throws IOException {
         }
         String v = p.getValueAsString();
         try {
-            return Paths.get(new URI(v));
+            return Paths.get(v);
         } catch (Exception e) {
             throw new JSONObjectException("Failed to bind `java.nio.file.Path` from value '"+v+"'");
         }

jr-objects/src/test/java/com/fasterxml/jackson/jr/ob/ReadSimpleTest.java

@@ -208,9 +208,6 @@ public void testMiscScalarFail() throws Exception {
     /**********************************************************************
      */
 
-    // 07-Jul-2020, tatu: Should be able to check but as of 2.11 same reader used
-    //    for wrapper and primitives.
-    /*
     public void testNullForMiscNumbers() throws Exception {
         assertNull(JSON.std.beanFrom(Integer.class," null "));
         assertNull(JSON.std.beanFrom(Long.class," null "));
@@ -219,7 +216,6 @@ public void testNullForMiscNumbers() throws Exception {
         assertNull(JSON.std.beanFrom(BigInteger.class," null "));
         assertNull(JSON.std.beanFrom(BigDecimal.class," null "));
     }
-    */
 
     public void testNullForMiscScalars() throws Exception {
         assertNull(JSON.std.beanFrom(Date.class," null "));

jr-objects/src/test/java/com/fasterxml/jackson/jr/ob/WriteSimpleTest.java

@@ -127,9 +127,9 @@ public void testKnownSimpleTypeFile() throws Exception
     public void testKnownSimpleTypePath() throws Exception
     {
         Path p = Paths.get(new URI("file:///foo/bar.txt"));
-        assertEquals(q("file:///foo/bar.txt"), JSON.std.asString(p));
+        assertEquals(q("/foo/bar.txt"), JSON.std.asString(p));
 
-        assertEquals(a2q("{'path':'file:///foo/bar.txt'}"), JSON.std.asString(new PathWrapper(p)));
+        assertEquals(a2q("{'path':'/foo/bar.txt'}"), JSON.std.asString(new PathWrapper(p)));
     }
 
     public void testSimpleEnumTypes() throws Exception