From 05c32c51fd4aee21a8a06d693e96389ae5ea03f8 Mon Sep 17 00:00:00 2001 From: Franck Nijhof Date: Tue, 10 Jan 2023 09:57:13 +0100 Subject: [PATCH] Code styling tweaks to the Cast integration (#85560) --- homeassistant/components/cast/__init__.py | 3 ++- homeassistant/components/cast/helpers.py | 3 ++- .../components/cast/home_assistant_cast.py | 3 ++- tests/components/cast/test_media_player.py | 23 +++++++++++++++---- 4 files changed, 24 insertions(+), 8 deletions(-) diff --git a/homeassistant/components/cast/__init__.py b/homeassistant/components/cast/__init__.py index 467678ba82b..1ef18081f93 100644 --- a/homeassistant/components/cast/__init__.py +++ b/homeassistant/components/cast/__init__.py @@ -80,7 +80,8 @@ class CastProtocol(Protocol): ) -> BrowseMedia | None: """Browse media. - Return a BrowseMedia object or None if the media does not belong to this platform. + Return a BrowseMedia object or None if the media does not belong to + this platform. """ async def async_play_media( diff --git a/homeassistant/components/cast/helpers.py b/homeassistant/components/cast/helpers.py index 26759ca9606..bd921bfa686 100644 --- a/homeassistant/components/cast/helpers.py +++ b/homeassistant/components/cast/helpers.py @@ -59,7 +59,8 @@ class ChromecastInfo: if self.cast_info.cast_type is None or self.cast_info.manufacturer is None: unknown_models = hass.data[DOMAIN]["unknown_models"] if self.cast_info.model_name not in unknown_models: - # Manufacturer and cast type is not available in mDNS data, get it over http + # Manufacturer and cast type is not available in mDNS data, + # get it over HTTP cast_info = dial.get_cast_type( cast_info, zconf=ChromeCastZeroconf.get_zeroconf(), diff --git a/homeassistant/components/cast/home_assistant_cast.py b/homeassistant/components/cast/home_assistant_cast.py index 14aa454c2e0..5460b831bcf 100644 --- a/homeassistant/components/cast/home_assistant_cast.py +++ b/homeassistant/components/cast/home_assistant_cast.py @@ -55,7 +55,8 @@ async def async_setup_ha_cast( hass_uuid = await instance_id.async_get(hass) controller = HomeAssistantController( - # If you are developing Home Assistant Cast, uncomment and set to your dev app id. + # If you are developing Home Assistant Cast, uncomment and set to + # your dev app id. # app_id="5FE44367", hass_url=hass_url, hass_uuid=hass_uuid, diff --git a/tests/components/cast/test_media_player.py b/tests/components/cast/test_media_player.py index 4df11e49ad5..0bec5a388a4 100644 --- a/tests/components/cast/test_media_player.py +++ b/tests/components/cast/test_media_player.py @@ -992,7 +992,9 @@ async def test_entity_browse_media(hass: HomeAssistant, hass_ws_client): "title": "Epic Sax Guy 10 Hours.mp4", "media_class": "video", "media_content_type": "video/mp4", - "media_content_id": "media-source://media_source/local/Epic Sax Guy 10 Hours.mp4", + "media_content_id": ( + "media-source://media_source/local/Epic Sax Guy 10 Hours.mp4" + ), "can_play": True, "can_expand": False, "thumbnail": None, @@ -1048,7 +1050,9 @@ async def test_entity_browse_media_audio_only( "title": "Epic Sax Guy 10 Hours.mp4", "media_class": "video", "media_content_type": "video/mp4", - "media_content_id": "media-source://media_source/local/Epic Sax Guy 10 Hours.mp4", + "media_content_id": ( + "media-source://media_source/local/Epic Sax Guy 10 Hours.mp4" + ), "can_play": True, "can_expand": False, "thumbnail": None, @@ -1254,10 +1258,17 @@ async def test_entity_play_media_sign_URL(hass: HomeAssistant, quick_play_mock): ), # Test HLS playlist is forwarded to the device ( - "http://a.files.bbci.co.uk/media/live/manifesto/audio/simulcast/hls/nonuk/sbr_low/ak/bbc_radio_fourfm.m3u8", + ( + "http://a.files.bbci.co.uk/media/live/manifesto" + "/audio/simulcast/hls/nonuk/sbr_low/ak/bbc_radio_fourfm.m3u8" + ), "bbc_radio_fourfm.m3u8", { - "media_id": "http://a.files.bbci.co.uk/media/live/manifesto/audio/simulcast/hls/nonuk/sbr_low/ak/bbc_radio_fourfm.m3u8", + "media_id": ( + "http://a.files.bbci.co.uk/media/live/manifesto" + "/audio/simulcast/hls/nonuk/sbr_low/ak" + "/bbc_radio_fourfm.m3u8" + ), "media_type": "audio", }, ), @@ -2237,7 +2248,9 @@ async def test_cast_platform_play_media_local_media( { ATTR_ENTITY_ID: entity_id, media_player.ATTR_MEDIA_CONTENT_TYPE: "application/vnd.apple.mpegurl", - media_player.ATTR_MEDIA_CONTENT_ID: f"{network.get_url(hass)}/api/hls/bla/master_playlist.m3u8?token=bla", + media_player.ATTR_MEDIA_CONTENT_ID: ( + f"{network.get_url(hass)}" "/api/hls/bla/master_playlist.m3u8?token=bla" + ), }, blocking=True, )