From 63c878db2958f43c03c115e746fa34594281574e Mon Sep 17 00:00:00 2001 From: Jon Parise Date: Tue, 31 Mar 2026 20:21:32 -0400 Subject: [PATCH] Encode 'use_cache' as a JSON boolean value --- instaparser/client.py | 8 ++++---- tests/test_client.py | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/instaparser/client.py b/instaparser/client.py index 2d906be..5c9e31a 100644 --- a/instaparser/client.py +++ b/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, } diff --git a/tests/test_client.py b/tests/test_client.py index f672473..677d7ee 100644 --- a/tests/test_client.py +++ b/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={})