Minor refactoring wrt java.nio.file.Path handling

Author: cowtowncoder

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

@@ -268,7 +268,7 @@ public void writeField(String fieldName, Object value, int type) throws IOExcept
             writeStringLikeField(fieldName, value.toString(), type);
             return;
         case SER_PATH:
-            writeStringLikeField(fieldName, ((Path) value).toUri().toString(), type);
+            writeStringLikeField(fieldName, pathToString((Path) value), type);
             return;
 
         // Others
@@ -398,7 +398,7 @@ protected void _writeValue(Object value, int type) throws IOException
             writeStringLikeValue(value.toString(), type);
             return;
         case SER_PATH:
-            writeStringLikeValue(((Path) value).toUri().toString(), type);
+            writeStringLikeValue(pathToString((Path) value), type);
             return;
 
         case SER_ITERABLE:
@@ -785,6 +785,13 @@ protected String dateToString(Date v) {
         return v.toString();
     }
 
+    /**
+     * @since 2.17
+     */
+    protected String pathToString(Path value) {
+        return value.toUri().toString();
+    }
+
     /*
     /**********************************************************************
     /* Other internal methods

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

@@ -6,6 +6,7 @@
 import java.io.IOException;
 import java.net.URI;
 import java.net.URL;
+import java.nio.file.Path;
 import java.nio.file.Paths;
 import java.util.*;
 
@@ -241,15 +242,7 @@ public Object read(JSONReader reader, JsonParser p) throws IOException
             }
             return URI.create(p.getValueAsString());
         case SER_PATH:
-            if (p.hasToken(JsonToken.VALUE_NULL)) {
-                return null;
-            }
-            String v = p.getValueAsString();
-            try {
-                return Paths.get(new URI(v));
-            } catch (Exception e) {
-                throw new JSONObjectException("Failed to bind `java.nio.file.Path` from value '"+v+"'");
-            }
+            return _readPath(p);
 
 //        case SER_MAP:
 //        case SER_LIST:
@@ -278,6 +271,19 @@ protected byte[] _readBinary(JsonParser p) throws IOException {
         return p.getBinaryValue();
     }
 
+    // @since 2.17
+    protected Path _readPath(JsonParser p) throws IOException {
+        if (p.hasToken(JsonToken.VALUE_NULL)) {
+            return null;
+        }
+        String v = p.getValueAsString();
+        try {
+            return Paths.get(new URI(v));
+        } catch (Exception e) {
+            throw new JSONObjectException("Failed to bind `java.nio.file.Path` from value '"+v+"'");
+        }
+    }
+
     protected int[] _readIntArray(JsonParser p) throws IOException
     {
         // !!! TODO

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

@@ -186,8 +186,9 @@ public void testMiscUriTypes() throws Exception
         assertEquals(url, JSON.std.beanFrom(URL.class, q(URL_STR)));
 
         Path p = Paths.get(new URI("file:///foo/bar.txt"));
+        String json = JSON.std.asString(p);
         assertEquals(p,
-                JSON.std.beanFrom(Path.class, q("file:///foo/bar.txt")));
+                JSON.std.beanFrom(Path.class, json));
     }
 
     public void testMiscScalarFail() throws Exception {