fix: conversion from JSON Pointer to GJSON Path

differ_test.go

@@ -27,6 +27,33 @@ func TestArrayCases(t *testing.T)  { runCasesFromFile(t, "testdata/tests/array.j
 func TestObjectCases(t *testing.T) { runCasesFromFile(t, "testdata/tests/object.json") }
 func TestRootCases(t *testing.T)   { runCasesFromFile(t, "testdata/tests/root.json") }
 
+func TestDiffer_Reset(t *testing.T) {
+	d := &Differ{
+		ptr: pointer{
+			buf: make([]byte, 15, 15),
+			end: 15,
+		},
+		hashmap: map[uint64]jsonNode{
+			1: {},
+		},
+		patch: make([]Operation, 42, 42),
+	}
+	d.Reset()
+
+	if l := len(d.patch); l != 0 {
+		t.Errorf("expected empty patch collection, got length %d", l)
+	}
+	if l := len(d.hashmap); l != 0 {
+		t.Errorf("expected cleared hashmap, got length %d", l)
+	}
+	if d.ptr.end != 0 {
+		t.Errorf("expected reset ptr")
+	}
+	if l := len(d.ptr.buf); l != 0 {
+		t.Errorf("expected empty ptr buf, got length %d", l)
+	}
+}
+
 func TestOptions(t *testing.T) {
 	makeopts := func(opts ...Option) []Option { return opts }
 
@@ -127,30 +154,3 @@ func runTestCase(t *testing.T, tc testcase, pc patchGetter, opts ...Option) {
 		}
 	}
 }
-
-func TestDiffer_Reset(t *testing.T) {
-	d := &Differ{
-		ptr: pointer{
-			buf: make([]byte, 15, 15),
-			end: 15,
-		},
-		hashmap: map[uint64]jsonNode{
-			1: {},
-		},
-		patch: make([]Operation, 42, 42),
-	}
-	d.Reset()
-
-	if l := len(d.patch); l != 0 {
-		t.Errorf("expected empty patch collection, got length %d", l)
-	}
-	if l := len(d.hashmap); l != 0 {
-		t.Errorf("expected cleared hashmap, got length %d", l)
-	}
-	if d.ptr.end != 0 {
-		t.Errorf("expected reset ptr")
-	}
-	if l := len(d.ptr.buf); l != 0 {
-		t.Errorf("expected empty ptr buf, got length %d", l)
-	}
-}

pointer.go

@@ -22,10 +22,9 @@ var (
 	// https://tools.ietf.org/html/rfc6901
 	rfc6901Escaper = strings.NewReplacer("~", "~0", "/", "~1")
 
-	// dotPathReplacer converts a RFC6901 JSON pointer to
-	// a JSON path, while also escaping any existing dot
-	// characters present in the original pointer.
-	dotPathReplacer = strings.NewReplacer(".", "\\.", "/", ".")
+	// pointerToGJSONPath converts a RFC6901 JSON pointer to a GJSON path.
+	// See https://github.com/tidwall/gjson/blob/master/SYNTAX.md
+	pointerToGJSONPath = strings.NewReplacer(".", "\\.", "*", "\\*", "?", "\\?", "/", ".", "~0", "~", "~1", "/")
 )
 
 // pointer represents an RFC 6901 JSON Pointer.
@@ -93,7 +92,7 @@ func (p pointer) reset() pointer {
 
 func toJSONPath(s string) string {
 	if len(s) != 0 {
-		return dotPathReplacer.Replace(s[1:])
+		return pointerToGJSONPath.Replace(s[1:])
 	}
 	// @this is a special modifier that can
 	// be used to retrieve the root path.

pointer_test.go

@@ -176,6 +176,26 @@ func Test_toJSONPath(t *testing.T) {
 			"/a/b/c",
 			"a.b.c",
 		},
+		{
+			"/a/b~1c/",
+			"a.b/c.",
+		},
+		{
+			"/a~0/b~0/~1",
+			"a~.b~./",
+		},
+		{
+			"/foo//a~1b",
+			"foo..a/b",
+		},
+		{
+			"/a.b~1/c//d",
+			"a\\.b/.c..d",
+		},
+		{
+			"/a/b?/c*",
+			"a.b\\?.c\\*",
+		},
 		{
 			"",
 			"@this",