Merge pull request #8 from joostlek/playback_nullable

Allow playback to be nullable

Author: joostlek

src/spotifyaio/spotify.py

@@ -93,9 +93,11 @@ async def _put(self, uri: str, data: dict[str, Any]) -> str:
         """Handle a PUT request to Spotify."""
         return await self._request(METH_PUT, uri, data=data)
 
-    async def get_playback(self) -> PlaybackState:
+    async def get_playback(self) -> PlaybackState | None:
         """Get playback state."""
         response = await self._get("v1/me/player")
+        if response == "":
+            return None
         return PlaybackState.from_json(response)
 
     async def transfer_playback(self, device_id: str) -> None:

tests/__snapshots__/test_playback.ambr

@@ -1,5 +1,5 @@
 # serializer version: 1
-# name: test_get_devices
+# name: test_get_playback_state
   dict({
     'context': dict({
       'context_type': <ContextType.COLLECTION: 'collection'>,

tests/test_playback.py

@@ -11,7 +11,7 @@
 from .const import SPOTIFY_URL
 
 
-async def test_get_devices(
+async def test_get_playback_state(
     aresponses: ResponsesMockServer,
     snapshot: SnapshotAssertion,
 ) -> None:
@@ -34,6 +34,27 @@ async def test_get_devices(
         await spotify.close()
 
 
+async def test_get_no_playback_state(
+    aresponses: ResponsesMockServer,
+) -> None:
+    """Test retrieving devices."""
+    aresponses.add(
+        SPOTIFY_URL,
+        "/v1/me/player",
+        METH_GET,
+        aresponses.Response(
+            status=204,
+            headers={"Content-Type": "application/json"},
+        ),
+    )
+    async with aiohttp.ClientSession() as session:
+        spotify = SpotifyClient(session=session)
+        spotify.authenticate("test")
+        response = await spotify.get_playback()
+        assert response is None
+        await spotify.close()
+
+
 async def test_transfer_playback(
     aresponses: ResponsesMockServer,
 ) -> None: