BREAK(jsonp) empty path => empty-parts ...

bc standard is not involved with *parts*.

Author: ankostis

graphtik/jsonpointer.py

@@ -149,8 +149,8 @@ def jsonp_path(jsonpointer: str) -> List[str]:
 
         >>> jsonp_path('/')
         ['']
-        >>> jsonp_path('')  # this becomes also root!
-        ['']
+        >>> jsonp_path('')
+        []
 
         >>> jsonp_path('a/b//c')
         ['', 'c']
@@ -162,6 +162,12 @@ def jsonp_path(jsonpointer: str) -> List[str]:
     if parts is False:
         parts = [jsonpointer]
     elif parts is None:
+        ## For empty-paths, the jsonpointer standard specifies
+        #  they must return the whole document (satisfied by `resolve_path()`)
+        #  but not what their *parts* should be.
+        if jsonpointer == "":
+            return []
+
         parts = [
             unescape_jsonpointer_part(part)
             for part in re.sub(".+(?:/$|/{2})", "/", jsonpointer).split("/")

test/test_jsonpointer.py

@@ -38,7 +38,7 @@ def un_esc(part):
 
 
 def test_jsonp_path_empty():
-    assert jsonp_path("") == [""]
+    assert jsonp_path("") == []
 
 
 def test_jsonp_path_root():
@@ -92,7 +92,6 @@ class C(str):
         ("/a//b", ["a", "", "b"]),
         ("/a/../b", ["a", "..", "b"]),
         ("/", [""]),
-        ("", []),
         ("/ some ", [" some "]),
         ("/ some /", [" some ", ""]),
         ("/ some /  ", [" some ", "  "]),
@@ -118,7 +117,6 @@ def test_jsonp_path_massive(inp, exp):
         ("/a/b/", [""]),
         ("/a//b", ["", "b"]),
         ("/", [""]),
-        ("", [""]),
         ("/ some ", ["", " some "]),
         ("/ some /", [""]),
         ("/ some /  ", ["", " some ", "  "]),