Merge pull request #13 from jparise/use_cache

Encode 'use_cache' as a JSON boolean value

Author: Donohue

instaparser/client.py

@@ -212,7 +212,7 @@ def article(self, url: str, content: str | None = None, output: str = "html", us
             "output": output,
         }
         if not use_cache:
-            payload["use_cache"] = "false"
+            payload["use_cache"] = False
         if content is not None:
             payload["content"] = content
 
@@ -274,7 +274,7 @@ def summary(
             "stream": stream_callback is not None,
         }
         if not use_cache:
-            payload["use_cache"] = "false"
+            payload["use_cache"] = False
         if content is not None:
             payload["content"] = content
 
@@ -337,7 +337,7 @@ def pdf(
             raise InstaparserValidationError("output must be 'html', 'text', or 'markdown'")
 
         if file is not None:
-            fields = {"output": output}
+            fields: dict[str, Any] = {"output": output}
             if not use_cache:
                 fields["use_cache"] = "false"
             if url:
@@ -353,7 +353,7 @@ def pdf(
             except HTTPError as e:
                 _map_http_error(e)
         elif url:
-            params = {
+            params: dict[str, Any] = {
                 "url": url,
                 "output": output,
             }

tests/test_client.py

@@ -170,7 +170,7 @@ def test_with_content(self, client, mock_request):
     def test_use_cache_false(self, client, mock_request):
         mock_request.return_value = make_response(json_data=ARTICLE_DATA)
         client.article(url="u", use_cache=False)
-        assert mock_request.call_args[1]["json_data"]["use_cache"] == "false"
+        assert mock_request.call_args[1]["json_data"]["use_cache"] is False
 
     def test_invalid_output(self, client):
         with pytest.raises(InstaparserValidationError, match="output must be"):
@@ -200,7 +200,7 @@ def test_with_content(self, client, mock_request):
     def test_use_cache_false(self, client, mock_request):
         mock_request.return_value = make_response(json_data=SUMMARY_DATA)
         client.summary(url="u", use_cache=False)
-        assert mock_request.call_args[1]["json_data"]["use_cache"] == "false"
+        assert mock_request.call_args[1]["json_data"]["use_cache"] is False
 
     def test_empty_response(self, client, mock_request):
         mock_request.return_value = make_response(json_data={})