fix(client): fix handling of all_secrets for content_scan

Author: gg-mmill

pygitguardian/client.py

@@ -389,6 +389,7 @@ def content_scan(
             endpoint="scan",
             data=request_obj,
             extra_headers=extra_headers,
+            params=params,
         )
 
         obj: Union[Detail, ScanResult]

tests/test_client.py

@@ -574,6 +574,34 @@ def test_extra_headers(
     assert mock_response.call_count == 1
 
 
+@responses.activate
+@pytest.mark.parametrize("all_secrets", (None, True, False))
+def test_scan_parameters(client: GGClient, all_secrets):
+    """
+    GIVEN a ggclient
+    WHEN calling content_scan with parameters
+    THEN the parameters are passed in the request
+    """
+
+    to_match = {}
+    if all_secrets is not None:
+        to_match["all_secrets"] = all_secrets
+
+    mock_response = responses.post(
+        url=client._url_from_endpoint("scan", "v1"),
+        status=200,
+        match=[matchers.query_param_matcher(to_match)],
+    )
+
+    client.content_scan(
+        DOCUMENT,
+        FILENAME,
+        all_secrets=all_secrets,
+    )
+
+    assert mock_response.call_count == 1
+
+
 @responses.activate
 @pytest.mark.parametrize("ignore_known_secrets", (None, True, False))
 @pytest.mark.parametrize("all_secrets", (None, True, False))