Adjust error handling for Gigya JSONDecodeError (#789)

* Adjust error handling for Gigya JSONDecodeError

* isort

src/renault_api/gigya/__init__.py

@@ -1,5 +1,6 @@
 """Gigya API."""
 import logging
+from json import JSONDecodeError
 from typing import Any
 from typing import cast
 from typing import Dict
@@ -9,6 +10,7 @@
 
 from . import models
 from . import schemas
+from .exceptions import GigyaException
 
 GIGYA_JWT = "gigya_jwt"
 GIGYA_LOGIN_TOKEN = "gigya_login_token"  # noqa: S105
@@ -35,7 +37,10 @@ async def request(
         #    url,
         #    response_text,
         # )
-        gigya_response: models.GigyaResponse = schema.loads(response_text)
+        try:
+            gigya_response: models.GigyaResponse = schema.loads(response_text)
+        except JSONDecodeError as err:
+            raise GigyaException("Gigya responded with invalid JSON") from err
         # Check for Gigya error
         gigya_response.raise_for_error_code()
         # Check for HTTP error

tests/fixtures.py

@@ -111,6 +111,15 @@ def inject_gigya_login(mocked_responses: aioresponses) -> str:
     )
 
 
+def inject_gigya_login_invalid(mocked_responses: aioresponses) -> str:
+    """Inject Gigya login data."""
+    return inject_gigya(
+        mocked_responses,
+        "accounts.login",
+        "login_invalid.txt",
+    )
+
+
 def inject_gigya_account_info(mocked_responses: aioresponses) -> str:
     """Inject Gigya getAccountInfo data."""
     return inject_gigya(

tests/gigya/test_gigya.py

@@ -30,6 +30,23 @@ async def test_login(
     assert response.get_session_cookie() == TEST_LOGIN_TOKEN
 
 
+@pytest.mark.asyncio
+async def test_login_error(
+    websession: aiohttp.ClientSession, mocked_responses: aioresponses
+) -> None:
+    """Test login response."""
+    fixtures.inject_gigya_login_invalid(mocked_responses)
+
+    with pytest.raises(gigya.exceptions.GigyaException):
+        await gigya.login(
+            websession,
+            TEST_GIGYA_URL,
+            TEST_GIGYA_APIKEY,
+            TEST_USERNAME,
+            TEST_PASSWORD,
+        )
+
+
 @pytest.mark.asyncio
 async def test_person_id(
     websession: aiohttp.ClientSession, mocked_responses: aioresponses