fix RedisJSON#947 step value can only positive none zero (RedisJSON#952) (RedisJSON#966)

gkorland · web-flow · commit 110fc056c4c7 · 2023-04-17T18:44:14.000+03:00
(cherry picked from commit a2fdfb4)
diff --git a/src/jsonpath/grammer.pest b/src/jsonpath/grammer.pest
@@ -8,7 +8,7 @@ string = _{("'" ~ (string_value) ~ "'") | ("\"" ~ (string_value) ~ "\"") | ("\""
 
 string_list = {string ~ ("," ~ string)*}
 
-pos_number = @{ASCII_NONZERO_DIGIT ~ ASCII_DIGIT* | ASCII_DIGIT}
+pos_number = @{ASCII_NONZERO_DIGIT ~ ASCII_DIGIT*}
 number = @{"-"? ~ ASCII_NONZERO_DIGIT ~ ASCII_DIGIT* | "-"? ~ ASCII_DIGIT}
 
 decimal = @{number ~ ("." ~ ASCII_DIGIT+)?}
diff --git a/src/jsonpath/mod.rs b/src/jsonpath/mod.rs
@@ -206,6 +206,14 @@ mod json_path_tests {
         verify_json!(path:"$.foo.[\"boo\"][-3,-1]", json:{"foo":{"boo":[1,2,3]}}, results:[1,3]);
     }
 
+    #[test]
+    fn basic_full_range() {
+        setup();
+        verify_json!(path:"$.foo.[\"boo\"][0:2:1]", json:{"foo":{"boo":[1,2,3]}}, results:[1,2]);
+        verify_json!(path:"$.foo.[\"boo\"][0:3:2]", json:{"foo":{"boo":[1,2,3]}}, results:[1,3]);
+        assert!(crate::jsonpath::compile("$.foo.[\"boo\"][0:3:0]").is_err());
+    }
+
     #[test]
     fn basic_bracket_notation_with_range() {
         setup();
