deezer-python-gql 0.22.1__tar.gz → 0.22.2__tar.gz

{deezer_python_gql-0.22.1/deezer_python_gql.egg-info → deezer_python_gql-0.22.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: deezer-python-gql
-Version: 0.22.1
+Version: 0.22.2
 Summary: Async typed Python client for Deezer's Pipe GraphQL API.
 Author-email: Julian Daberkow <jdaberkow@users.noreply.github.com>
 License: Apache-2.0

{deezer_python_gql-0.22.1 → deezer_python_gql-0.22.2}/deezer_python_gql/base_client.py

@@ -288,11 +288,11 @@ class DeezerBaseClient:
             variables = response.variables
             if data:
                 # Partial success — some items failed (e.g. deleted albums in favorites).
-                # Log the errors but return the valid data.
+                # That is expected, so log at debug level and return the valid data.
                 error_details = [
                     f"{e.get('message', 'Unknown error')} (path={e.get('path')})" for e in errors
                 ]
-                logger.warning(
+                logger.debug(
                     "GraphQL response for %s (variables=%s) contained %d error(s): %s",
                     op,
                     variables,
@@ -377,7 +377,8 @@ class DeezerBaseClient:
                 text = await resp.text()
 
             if not is_success:
-                if 400 <= status < 500:
+                # 429 is rate limiting, not a rejected ARL
+                if 400 <= status < 500 and status != 429:
                     msg = f"ARL authentication rejected by auth.deezer.com (HTTP {status})"
                     raise GraphQLClientAuthError(msg)
                 raise GraphQLClientHttpError(

{deezer_python_gql-0.22.1 → deezer_python_gql-0.22.2}/deezer_python_gql/generated/base_client.py

@@ -290,11 +290,11 @@ class DeezerBaseClient:
             variables = response.variables
             if data:
                 # Partial success — some items failed (e.g. deleted albums in favorites).
-                # Log the errors but return the valid data.
+                # That is expected, so log at debug level and return the valid data.
                 error_details = [
                     f"{e.get('message', 'Unknown error')} (path={e.get('path')})" for e in errors
                 ]
-                logger.warning(
+                logger.debug(
                     "GraphQL response for %s (variables=%s) contained %d error(s): %s",
                     op,
                     variables,
@@ -379,7 +379,8 @@ class DeezerBaseClient:
                 text = await resp.text()
 
             if not is_success:
-                if 400 <= status < 500:
+                # 429 is rate limiting, not a rejected ARL
+                if 400 <= status < 500 and status != 429:
                     msg = f"ARL authentication rejected by auth.deezer.com (HTTP {status})"
                     raise GraphQLClientAuthError(msg)
                 raise GraphQLClientHttpError(

{deezer_python_gql-0.22.1 → deezer_python_gql-0.22.2/deezer_python_gql.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: deezer-python-gql
-Version: 0.22.1
+Version: 0.22.2
 Summary: Async typed Python client for Deezer's Pipe GraphQL API.
 Author-email: Julian Daberkow <jdaberkow@users.noreply.github.com>
 License: Apache-2.0

{deezer_python_gql-0.22.1 → deezer_python_gql-0.22.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "deezer-python-gql"
-version = "0.22.1"
+version = "0.22.2"
 description = "Async typed Python client for Deezer's Pipe GraphQL API."
 readme = "README.md"
 requires-python = ">=3.12"

{deezer_python_gql-0.22.1 → deezer_python_gql-0.22.2}/tests/test_client.py

@@ -12,6 +12,7 @@ from __future__ import annotations
 import asyncio
 import base64
 import json
+import logging
 import time
 from http.cookies import SimpleCookie
 from pathlib import Path
@@ -648,10 +649,11 @@ async def test_auth_rejected_arl_raises_auth_error() -> None:
 
 
 @pytest.mark.asyncio
-async def test_auth_server_error_raises_http_error() -> None:
-    """Verify a 5xx from auth.deezer.com raises GraphQLClientHttpError, not AuthError."""
+@pytest.mark.parametrize("status", [429, 503])
+async def test_auth_server_error_raises_http_error(status: int) -> None:
+    """Verify a 5xx or 429 from auth.deezer.com raises GraphQLClientHttpError, not AuthError."""
     client = DeezerBaseClient(arl="test_arl")
-    auth_cm = _mock_post_context_manager(b"Service Unavailable", status=503)
+    auth_cm = _mock_post_context_manager(b"Service Unavailable", status=status)
 
     with patch("deezer_python_gql.base_client.ClientSession") as mock_cls:
         mock_instance = MagicMock()
@@ -661,7 +663,7 @@ async def test_auth_server_error_raises_http_error() -> None:
 
         with pytest.raises(GraphQLClientHttpError) as exc_info:
             await client._ensure_jwt()  # noqa: SLF001
-    assert exc_info.value.status_code == 503
+    assert exc_info.value.status_code == status
 
 
 @pytest.mark.asyncio
@@ -784,6 +786,25 @@ def test_get_data_raises_on_missing_data_key() -> None:
         client.get_data(response)
 
 
+def test_get_data_logs_partial_errors_at_debug(caplog: pytest.LogCaptureFixture) -> None:
+    """Verify items Deezer no longer has are logged at debug level, not as a warning."""
+    client = DeezerBaseClient(arl="test")
+    response = GQLResponse(
+        status=200,
+        data=json.dumps(
+            {
+                "data": {"me": {"id": "1"}},
+                "errors": [{"message": "PodcastEpisode does not exist", "path": ["me"]}],
+            }
+        ).encode(),
+        is_success=True,
+    )
+
+    with caplog.at_level(logging.DEBUG, logger="deezer_python_gql"):
+        assert client.get_data(response) == {"me": {"id": "1"}}
+    assert [record.levelno for record in caplog.records] == [logging.DEBUG]
+
+
 def test_get_data_raises_on_graphql_errors() -> None:
     """Verify get_data raises GraphQLClientGraphQLMultiError for GraphQL-level errors."""
     client = DeezerBaseClient(arl="test")