Correct a small issue that made it not really RFC6901 Compliant

Author: TotalTechGeek

general.test.js

@@ -166,7 +166,7 @@ describe('Various Test Cases', () => {
   })
 
   it('is able to access "/" keys from above', async () => {
-    for (const engine of [...normalEngines, ...permissiveEngines]) await testEngine(engine, { map: [[1], { '+': [{ var: '' }, { var: '../../..~1' }] }] }, { '': { '': { '/': 3 } } }, [4])
+    for (const engine of [...normalEngines, ...permissiveEngines]) await testEngine(engine, { map: [[1], { '+': [{ var: '' }, { var: '../..////~1' }] }] }, { '': { '': { '/': 3 } } }, [4])
   })
 
   it('is able to handle path escaping with multiple escapes', async () => {

suites/scopes.json

@@ -58,7 +58,7 @@
         "rule": { 
             "map": [
                 { "var": "arr" }, 
-                { "+": [{ "var": "../../~2~2~1" }, { "var": "../../..~1" }]}
+                { "+": [{ "var": "../../~2~2~1" }, { "var": "../..////~1" }]}
             ] 
         },
         "data": { "arr": [1,2,3], "../": 10, "": { "": { "/": 7 }} },

suites/vars.json

@@ -2,19 +2,19 @@
     "Test Specification for Handling esoteric path traversal",
     {
         "description": "Fetches a value from an empty key",
-        "rule": { "var": "." },
+        "rule": { "var": "/" },
         "data": { "" : 1 },
         "result": 1
     },
     {
         "description": "Fetches a value from a nested empty key",
-        "rule": { "var": ".." },
+        "rule": { "var": "//" },
         "data": { "" : { "": 2 } },
         "result": 2
     },
     {
         "description": "Fetches a value from a doubly nested empty key",
-        "rule": { "var": "..." },
+        "rule": { "var": "///" },
         "data": { "" : { "": { "": 3 } } },
         "result": 3
     },
@@ -38,13 +38,13 @@
     },
     {
         "description": "Fetches a value from a key inside an empty key with a dot in it",
-        "rule": { "var": "/~2key" },
+        "rule": { "var": "//~2key" },
         "data": { "": { ".key" : 6 } },
         "result": 6
     },
     {
         "description": "Going a few levels deep",
-        "rule": { "var": "//~2key." },
+        "rule": { "var": "///~2key/" },
         "data": { "": { "": { ".key": { "": 7 }} }},
         "result": 7
     },
@@ -62,7 +62,7 @@
     },
     {
         "description": "Empty key then dot",
-        "rule": { "var": ".~2" },
+        "rule": { "var": "//~2" },
         "data": { "" : { "." : 11 } },
         "result": 11
     },
@@ -79,20 +79,20 @@
         "result": 13
     },
     {
-        "description": "Fetches a value from an empty key, traditional",
-        "rule": { "var": "/" },
+        "description": "Fetches a value from an empty key, equivalence",
+        "rule": { "var": "." },
         "data": { "" : 1 },
         "result": 1
     },
     {
-        "description": "Fetches a value from a nested empty key, traditional",
-        "rule": { "var": "//" },
+        "description": "Fetches a value from a nested empty key, equivalence",
+        "rule": { "var": ".." },
         "data": { "" : { "": 2 } },
         "result": 2
     },
     {
-        "description": "Fetches a value from a doubly nested empty key, traditional",
-        "rule": { "var": "///" },
+        "description": "Fetches a value from a doubly nested empty key, equivalence",
+        "rule": { "var": "..." },
         "data": { "" : { "": { "": 3 } } },
         "result": 3
     }

utilities/splitPath.js

@@ -46,7 +46,7 @@ export function splitPath (str) {
         // The following else might be something tweaked in a spec.
       } else throw new Error('Invalid escape sequence')
     } else if (char === '.' || char === '/') {
-      parts.push(current)
+      if (i) parts.push(current)
       current = ''
     } else current += char
   }