jeans/core
jeans/core
1
0
Fork 0
mirror of https://github.com/home-assistant/core.git synced 2025-07-19 19:27:45 +00:00
Code Issues Packages Projects Releases Wiki Activity

Mask spotify content in owntone library (#79247)

Browse Source
This commit is contained in:
uvjustin 2022-09-29 18:07:26 -07:00 committed by GitHub
parent 21b078eeb7
commit 2688e5b2d4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 31 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
homeassistant/components/forked_daapd/browse_media.py
View File

@ -229,6 +229,10 @@ def create_browse_media_response(
if not children: # Directory searches will pass in subdirectories as children if not children: # Directory searches will pass in subdirectories as children
children = [] children = []
for item in result: for item in result:
if item.get("data_kind") == "spotify" or (
"path" in item and cast(str, item["path"]).startswith("spotify")
): # Exclude spotify data from Owntone library
continue
assert isinstance(item["uri"], str) assert isinstance(item["uri"], str)
media_type = OWNTONE_TYPE_TO_MEDIA_TYPE[item["uri"].split(":")[1]] media_type = OWNTONE_TYPE_TO_MEDIA_TYPE[item["uri"].split(":")[1]]
title = item.get("name") or item.get("title") # only tracks use title title = item.get("name") or item.get("title") # only tracks use title

28
tests/components/forked_daapd/test_browse_media.py
View File

@ -4,7 +4,11 @@ from http import HTTPStatus
from unittest.mock import patch from unittest.mock import patch
from homeassistant.components import media_source, spotify from homeassistant.components import media_source, spotify
from homeassistant.components.forked_daapd.browse_media import create_media_content_id from homeassistant.components.forked_daapd.browse_media import (
MediaContent,
create_media_content_id,
is_owntone_media_content_id,
)
from homeassistant.components.media_player import BrowseMedia, MediaClass, MediaType from homeassistant.components.media_player import BrowseMedia, MediaClass, MediaType
from homeassistant.components.spotify.const import ( from homeassistant.components.spotify.const import (
MEDIA_PLAYER_PREFIX as SPOTIFY_MEDIA_PLAYER_PREFIX, MEDIA_PLAYER_PREFIX as SPOTIFY_MEDIA_PLAYER_PREFIX,
@ -111,6 +115,16 @@ async def test_async_browse_media(hass, hass_ws_client, config_entry):
"length_ms": 2951554, "length_ms": 2951554,
"uri": "library:artist:3815427709949443149", "uri": "library:artist:3815427709949443149",
}, },
{
"id": "456",
"name": "Spotify Artist",
"name_sort": "Spotify Artist",
"album_count": 1,
"track_count": 10,
"length_ms": 2254,
"uri": "spotify:artist:abc123",
"data_kind": "spotify",
},
] ]
mock_api.return_value.get_genres.return_value = [ mock_api.return_value.get_genres.return_value = [
{"name": "Classical"}, {"name": "Classical"},
@ -127,6 +141,13 @@ async def test_async_browse_media(hass, hass_ws_client, config_entry):
"smart_playlist": False, "smart_playlist": False,
"uri": "library:playlist:1", "uri": "library:playlist:1",
}, },
{
"id": 2,
"name": "Spotify Playlist",
"path": "spotify:playlist:abc123",
"smart_playlist": False,
"uri": "library:playlist:2",
},
] ]
# Request browse root through WebSocket # Request browse root through WebSocket
@ -150,6 +171,11 @@ async def test_async_browse_media(hass, hass_ws_client, config_entry):
"""Browse the children of this BrowseMedia.""" """Browse the children of this BrowseMedia."""
nonlocal msg_id nonlocal msg_id
for child in children: for child in children:
# Assert Spotify content is not passed through as Owntone media
assert not (
is_owntone_media_content_id(child["media_content_id"])
and "Spotify" in MediaContent(child["media_content_id"]).title
)
if child["can_expand"]: if child["can_expand"]:
await client.send_json( await client.send_json(
{ {