Merge pull request #90 from Marcono1234/invalid-selectors

Add more tests for selectors
jsonpath-standard · Aug 3, 2024 · c47c0bb · c47c0bb
2 parents 4c7cee1 + 4e9f117
commit c47c0bb
Show file tree

Hide file tree

Showing 3 changed files with 214 additions and 0 deletions.
diff --git a/tests/basic.json b/tests/basic.json
@@ -170,6 +170,16 @@
       "selector": "$[0 2]",
       "invalid_selector": true
     },
+    {
+      "name": "selector, leading comma",
+      "selector": "$[,0]",
+      "invalid_selector": true
+    },
+    {
+      "name": "selector, trailing comma",
+      "selector": "$[0,]",
+      "invalid_selector": true
+    },
     {
       "name": "multiple selectors, name and index, array data",
       "selector": "$['a',1]",

diff --git a/tests/index_selector.json b/tests/index_selector.json
@@ -31,6 +31,34 @@
       ],
       "result": []
     },
+    {
+      "name": "min exact index",
+      "selector": "$[-9007199254740991]",
+      "document": [
+        "first",
+        "second"
+      ],
+      "result": []
+    },
+    {
+      "name": "max exact index",
+      "selector": "$[9007199254740991]",
+      "document": [
+        "first",
+        "second"
+      ],
+      "result": []
+    },
+    {
+      "name": "min exact index - 1",
+      "selector": "$[-9007199254740992]",
+      "invalid_selector": true
+    },
+    {
+      "name": "max exact index + 1",
+      "selector": "$[9007199254740992]",
+      "invalid_selector": true
+    },
     {
       "name": "overflowing index",
       "selector": "$[231584178474632390847141970017375815706539969331281128078915168015826259279872]",
@@ -85,6 +113,26 @@
       "selector": "$[01]",
       "invalid_selector": true
     },
+    {
+      "name": "decimal",
+      "selector": "$[1.0]",
+      "invalid_selector": true
+    },
+    {
+      "name": "plus",
+      "selector": "$[+1]",
+      "invalid_selector": true
+    },
+    {
+      "name": "minus space",
+      "selector": "$[- 1]",
+      "invalid_selector": true
+    },
+    {
+      "name": "-0",
+      "selector": "$[-0]",
+      "invalid_selector": true
+    },
     {
       "name": "leading -0",
       "selector": "$[-01]",

diff --git a/tests/slice_selector.json b/tests/slice_selector.json
@@ -610,6 +610,72 @@
         9
       ]
     },
+    {
+      "name": "start, min exact",
+      "selector": "$[-9007199254740991::]",
+      "document": [],
+      "result": []
+    },
+    {
+      "name": "start, max exact",
+      "selector": "$[9007199254740991::]",
+      "document": [],
+      "result": []
+    },
+    {
+      "name": "start, min exact - 1",
+      "selector": "$[-9007199254740992::]",
+      "invalid_selector": true
+    },
+    {
+      "name": "start, max exact + 1",
+      "selector": "$[9007199254740992::]",
+      "invalid_selector": true
+    },
+    {
+      "name": "end, min exact",
+      "selector": "$[:-9007199254740991:]",
+      "document": [],
+      "result": []
+    },
+    {
+      "name": "end, max exact",
+      "selector": "$[:9007199254740991:]",
+      "document": [],
+      "result": []
+    },
+    {
+      "name": "end, min exact - 1",
+      "selector": "$[:-9007199254740992:]",
+      "invalid_selector": true
+    },
+    {
+      "name": "end, max exact + 1",
+      "selector": "$[:9007199254740992:]",
+      "invalid_selector": true
+    },
+    {
+      "name": "step, min exact",
+      "selector": "$[::-9007199254740991]",
+      "document": [],
+      "result": []
+    },
+    {
+      "name": "step, max exact",
+      "selector": "$[::9007199254740991]",
+      "document": [],
+      "result": []
+    },
+    {
+      "name": "step, min exact - 1",
+      "selector": "$[::-9007199254740992]",
+      "invalid_selector": true
+    },
+    {
+      "name": "step, max exact + 1",
+      "selector": "$[::9007199254740992]",
+      "invalid_selector": true
+    },
     {
       "name": "overflowing to value",
       "selector": "$[2:231584178474632390847141970017375815706539969331281128078915168015826259279872]",
@@ -639,6 +705,96 @@
       "name": "underflowing step",
       "selector": "$[-1:-10:-231584178474632390847141970017375815706539969331281128078915168015826259279872]",
       "invalid_selector": true
+    },
+    {
+      "name": "start, leading 0",
+      "selector": "$[01::]",
+      "invalid_selector": true
+    },
+    {
+      "name": "start, decimal",
+      "selector": "$[1.0::]",
+      "invalid_selector": true
+    },
+    {
+      "name": "start, plus",
+      "selector": "$[+1::]",
+      "invalid_selector": true
+    },
+    {
+      "name": "start, minus space",
+      "selector": "$[- 1::]",
+      "invalid_selector": true
+    },
+    {
+      "name": "start, -0",
+      "selector": "$[-0::]",
+      "invalid_selector": true
+    },
+    {
+      "name": "start, leading -0",
+      "selector": "$[-01::]",
+      "invalid_selector": true
+    },
+    {
+      "name": "end, leading 0",
+      "selector": "$[:01:]",
+      "invalid_selector": true
+    },
+    {
+      "name": "end, decimal",
+      "selector": "$[:1.0:]",
+      "invalid_selector": true
+    },
+    {
+      "name": "end, plus",
+      "selector": "$[:+1:]",
+      "invalid_selector": true
+    },
+    {
+      "name": "end, minus space",
+      "selector": "$[:- 1:]",
+      "invalid_selector": true
+    },
+    {
+      "name": "end, -0",
+      "selector": "$[:-0:]",
+      "invalid_selector": true
+    },
+    {
+      "name": "end, leading -0",
+      "selector": "$[:-01:]",
+      "invalid_selector": true
+    },
+    {
+      "name": "step, leading 0",
+      "selector": "$[::01]",
+      "invalid_selector": true
+    },
+    {
+      "name": "step, decimal",
+      "selector": "$[::1.0]",
+      "invalid_selector": true
+    },
+    {
+      "name": "step, plus",
+      "selector": "$[::+1]",
+      "invalid_selector": true
+    },
+    {
+      "name": "step, minus space",
+      "selector": "$[::- 1]",
+      "invalid_selector": true
+    },
+    {
+      "name": "step, -0",
+      "selector": "$[::-0]",
+      "invalid_selector": true
+    },
+    {
+      "name": "step, leading -0",
+      "selector": "$[::-01]",
+      "invalid_selector": true
     }
   ]
 }