Merge pull request #71 from jg-rp/fix-re-repr

Fix the string representation of regex literals. Fixes #70.

Author: jg-rp

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Python JSONPath Change Log
 
+## Version 1.2.1 (unreleased)
+
+**Fixes**
+
+- Fixed the string representation regex literals in filter expressions. See [issue #70](https://github.com/jg-rp/python-jsonpath/issues/70).
+
 ## Version 1.2.0
 
 **Fixes**

jsonpath/__about__.py

@@ -1,4 +1,4 @@
 # SPDX-FileCopyrightText: 2023-present James Prior <[email protected]>
 #
 # SPDX-License-Identifier: MIT
-__version__ = "1.2.0"
+__version__ = "1.2.1"

jsonpath/filter.py

@@ -243,8 +243,7 @@ def __str__(self) -> str:
             if self.value.flags & flag:
                 flags.append(ch)
 
-        pattern = re.sub(r"\\(.)", r"\1", self.value.pattern)
-        return f"/{pattern}/{''.join(flags)}"
+        return f"/{self.value.pattern}/{''.join(flags)}"
 
 
 class ListLiteral(FilterExpression):

tests/test_fluent_api.py

@@ -4,7 +4,7 @@
 
 from jsonpath import JSONPathMatch
 from jsonpath import JSONPointer
-from jsonpath import compile
+from jsonpath import compile  # noqa: A004
 from jsonpath import query
 
 

tests/test_parse.py

@@ -200,6 +200,21 @@ class Case:
         path="$[?!(@.a && [email protected])]",
         want="$[?!(@['a'] && !@['b'])]",
     ),
+    Case(
+        description="issue 70",
+        path=r"$[?@ =~ /\d/]",
+        want="$[?@ =~ /\\d/]",
+    ),
+    Case(
+        description="escaped slash in regex literal",
+        path=r"$[?@ =~ /\\d/]",
+        want="$[?@ =~ /\\\\d/]",
+    ),
+    Case(
+        description="match function",
+        path=r"$[?match(@, '\\d')]",
+        want='$[?match(@, "\\\\d")]',
+    ),
 ]
 
 

tests/test_re.py

@@ -39,6 +39,12 @@ class Case:
         data={"some": [{"thing": "foO"}]},
         want=[{"thing": "foO"}],
     ),
+    Case(
+        description="escaped slash",
+        path="$.some[?(@.thing =~ /fo\\\\[a-z]/)]",
+        data={"some": [{"thing": "fo\\b"}]},
+        want=[{"thing": "fo\\b"}],
+    ),
 ]