Improve error HTTP handlin

If HTTP error occurred thorough a response (e.g., non-2xx status code) then save the response test in the exception.
Should allow for easier debugging, as the API sends errors as a JSON.

Author: Electronic-Mango

docs/usage.md

@@ -290,9 +290,14 @@ from simplejustwatchapi import search, JustWatchHttpError
 try:
     results = search("The Matrix", count="five")
 except JustWatchHttpError as e:
-    print(str(e))
+    print(str(e), e.response)
 ```
 
+Aside from exception message, the error can also contains optional field `response` with
+full response text as string. This will only contain data if error is related to the
+**response** message, otherwise it's `None`. Usually it is a JSON (in string form) with
+API errors, if these specific errors are also causing non-`2xx` status codes.
+
 !!! note "Numeric strings instead of `int`"
     Since requests are send as a JSON you can use strings for `int` arguments, as long
     as they are numeric strings, like `5`, instead of `five`:

src/simplejustwatchapi/exceptions.py

@@ -40,4 +40,15 @@ class JustWatchHttpError(JustWatchError):
 
     This is a general exception for any HTTP-related errors, such as non-`2xx` status
     codes, network errors, timeouts, etc.
+
+    Attributes:
+        msg (str): Error message describing the HTTP error.
+        response (str | None): Optional text of the HTTP response, if available.
+            Usucally contains JSON with error responses from the API.
+
     """
+
+    def __init__(self, msg: str, response: str | None = None) -> None:
+        """Init JustWatchHttpError with error message and optional response text."""
+        super().__init__(msg)
+        self.response = response

src/simplejustwatchapi/justwatch.py

@@ -47,7 +47,7 @@
     country code. |
 """
 
-from httpx import HTTPError, post
+from httpx import HTTPError, HTTPStatusError, post
 
 from simplejustwatchapi.exceptions import JustWatchHttpError
 from simplejustwatchapi.query import (
@@ -538,6 +538,8 @@ def _post_to_jw_graphql_api(request_json: dict) -> dict:
     try:
         response = post(_GRAPHQL_API_URL, json=request_json)
         response.raise_for_status()
+        return response.json()
+    except HTTPStatusError as e:
+        raise JustWatchHttpError(str(e), e.response.text) from e
     except HTTPError as e:
         raise JustWatchHttpError(str(e)) from e
-    return response.json()

test/simplejustwatchapi/test_justwatch.py

@@ -27,6 +27,10 @@
 DUMMY_RESPONSE = {"dummy": "response"}
 ENTRIES = [MagicMock(), MagicMock(), None]
 
+REQUEST_ERROR_MESSAGE = "HTTP request error"
+RESPONSE_ERROR_STATUS_CODE = 420
+RESPONSE_ERROR_MESSAGE = "HTTP response error"
+
 
 @fixture
 def post_mock_success(mocker):
@@ -39,7 +43,7 @@ def post_mock_success(mocker):
 @fixture
 def post_mock_request_error(mocker):
     post_mock = mocker.patch("simplejustwatchapi.justwatch.post")
-    post_mock.side_effect = RequestError("HTTP request error")
+    post_mock.side_effect = RequestError(REQUEST_ERROR_MESSAGE)
     mock_request = Request(method="POST", url=JUSTWATCH_GRAPHQL_URL)
     post_mock.return_value = Response(status_code=200, request=mock_request)
     # Technically setting the return value is not necessary, since the side effect will
@@ -52,7 +56,11 @@ def post_mock_request_error(mocker):
 def post_mock_status_error(mocker):
     post_mock = mocker.patch("simplejustwatchapi.justwatch.post")
     mock_request = Request(method="POST", url=JUSTWATCH_GRAPHQL_URL)
-    post_mock.return_value = Response(status_code=420, request=mock_request)
+    post_mock.return_value = Response(
+        status_code=RESPONSE_ERROR_STATUS_CODE,
+        request=mock_request,
+        text=RESPONSE_ERROR_MESSAGE,
+    )
     return post_mock
 
 
@@ -158,8 +166,10 @@ def test_providers(requests_mock, parser_mock, post_mock_success):
 )
 def test_http_request_error(prepare_name, function, inputs, post_mock_request_error):
     full_mock_name = f"simplejustwatchapi.justwatch.{prepare_name}"
-    with patch(full_mock_name), raises(JustWatchHttpError):
+    with patch(full_mock_name), raises(JustWatchHttpError) as e:
         function(*inputs)
+    assert str(e.value) == REQUEST_ERROR_MESSAGE
+    assert e.value.response is None
 
 
 @mark.parametrize(
@@ -176,5 +186,7 @@ def test_http_request_error(prepare_name, function, inputs, post_mock_request_er
 )
 def test_http_status_error(prepare_name, function, inputs, post_mock_status_error):
     full_mock_name = f"simplejustwatchapi.justwatch.{prepare_name}"
-    with patch(full_mock_name), raises(JustWatchHttpError):
+    with patch(full_mock_name), raises(JustWatchHttpError) as e:
         function(*inputs)
+    assert str(RESPONSE_ERROR_STATUS_CODE) in str(e.value)
+    assert e.value.response == RESPONSE_ERROR_MESSAGE