Raise an error when both can_handle_fun and match_querystring are used.

Author: mindflayer

mocket/mocks/mockhttp.py

@@ -237,7 +237,16 @@ def __init__(
             responses: Response(s) to return
             match_querystring: Whether to match query strings
             can_handle_fun: Custom matching function
+
+        Raises:
+            ValueError: If both can_handle_fun and match_querystring are specified
         """
+        if can_handle_fun and not match_querystring:
+            raise ValueError(
+                "cannot specify both 'can_handle_fun' and 'match_querystring=False': "
+                "when using a custom matching function, 'match_querystring' is ignored"
+            )
+
         self._can_handle_fun = can_handle_fun if can_handle_fun else self._can_handle
 
         uri = urlsplit(uri)

tests/test_http.py

@@ -482,3 +482,39 @@ def test_can_handle(self):
         response = requests.get("http://testme.org/foobar?b=2")
         self.assertEqual(response.status_code, 200)
         self.assertEqual(response.json(), {"message": "Missed!"})
+
+    def test_can_handle_fun_with_match_querystring_false_raises(self):
+        """Test that using both can_handle_fun and match_querystring=False raises ValueError."""
+        with self.assertRaises(ValueError) as context:
+            Entry(
+                "http://testme.org/path",
+                Entry.GET,
+                responses=["test"],
+                can_handle_fun=lambda path, qs: True,
+                match_querystring=False,
+            )
+        self.assertIn(
+            "cannot specify both 'can_handle_fun' and 'match_querystring=False'",
+            str(context.exception),
+        )
+
+    def test_can_handle_fun_with_match_querystring_true_works(self):
+        """Test that using can_handle_fun with match_querystring=True works fine."""
+        entry = Entry(
+            "http://testme.org/path",
+            Entry.GET,
+            responses=["test"],
+            can_handle_fun=lambda path, qs: True,
+            match_querystring=True,
+        )
+        self.assertIsNotNone(entry)
+
+    def test_can_handle_fun_alone_works(self):
+        """Test that using can_handle_fun alone (without specifying match_querystring) works."""
+        entry = Entry(
+            "http://testme.org/path",
+            Entry.GET,
+            responses=["test"],
+            can_handle_fun=lambda path, qs: True,
+        )
+        self.assertIsNotNone(entry)