Allow multivalued query parameters (#2)

Author: m-kus

.github/workflows/ci-python.yml

@@ -1,6 +1,6 @@
 name: CI (Python)
 
-on: [push]
+on: [push, pull_request]
 
 jobs:
   build:

.gitignore

@@ -127,3 +127,6 @@ dmypy.json
 
 # Pyre type checker
 .pyre/
+
+# PyCharm
+.idea/

postgrest_py/request_builder.py

@@ -55,10 +55,14 @@ def not_(self):
 
     def filter(self, column: str, operator: str, criteria: str):
         """Either filter in or filter out based on Self.negate_next."""
-        if self.negate_next == True:
+        if self.negate_next is True:
             self.negate_next = False
             operator = f"not.{operator}"
-        self.session.params[sanitize_param(column)] = f"{operator}.{criteria}"
+        key, val = sanitize_param(column), f"{operator}.{criteria}"
+        if key in self.session.params:
+            self.session.params.update({key: self.session.params.getlist(key) + [val]})
+        else:
+            self.session.params[key] = val
         return self
 
     def eq(self, column: str, value: str):

tests/test_filter_request_builder.py

@@ -28,3 +28,9 @@ def test_filter(filter_request_builder):
     builder = filter_request_builder.filter(":col.name", "eq", "val")
 
     assert builder.session.params['":col.name"'] == "eq.val"
+
+
+def test_multivalued_param(filter_request_builder):
+    builder = filter_request_builder.lte("x", "a").gte("x", "b")
+
+    assert str(builder.session.params) == "x=lte.a&x=gte.b"