Don't prefilter on regexes where the cleaning step didn't work

Author: kannibalox

src/pyrosimple/util/matching.py

@@ -348,6 +348,11 @@ def pre_filter_eq(self) -> str:
         else:
             needle = self.CLEAN_PRE_VAL_RE.sub(" ", self._value)
             split_needle = self.SPLIT_PRE_GLOB_RE.split(needle)
+        # If the cleaning of the needle did not succeed and we still
+        # have regex-y values, don't prefilter
+        print(needle)
+        if any(metachar in needle for metachar in r'{}[]\\|()'):
+            return ""
         # Grab the longest needle available from the array
         needle = list(sorted(split_needle, key=len))[-1]
 

src/tests/test_matching.py

@@ -203,6 +203,7 @@ def test_matcher_fail(matcher, item):
         ('name="arch linux"', 'string.contains_i=$d.name=,"arch linux"'),
         ("name=/arch/", 'string.contains_i=$d.name=,"arch"'),
         ("name=/(arch|foo)k+/", 'string.contains_i=$d.name=,"k"'),
+        ("name=/((arch|ubuntu)|foo)k+/", ''),
         ("name=ARCH", 'string.contains_i=$d.name=,"ARCH"'),
         ("size<1G", "less=d.size_bytes=,value=1073741824"),
         ("is_complete=no", "equal=d.complete=,value=0"),