From 214731e9645a3f2191dce0728487f4d802a223cb Mon Sep 17 00:00:00 2001 From: Magnus Date: Thu, 13 Nov 2025 20:24:24 +0100 Subject: [PATCH] Component asuswrt: Type check is redundant for this value (#154535) --- homeassistant/components/asuswrt/bridge.py | 2 -- tests/components/asuswrt/test_sensor.py | 16 ++++++---------- 2 files changed, 6 insertions(+), 12 deletions(-) diff --git a/homeassistant/components/asuswrt/bridge.py b/homeassistant/components/asuswrt/bridge.py index 81ace7d2873..a1bfad989d4 100644 --- a/homeassistant/components/asuswrt/bridge.py +++ b/homeassistant/components/asuswrt/bridge.py @@ -111,8 +111,6 @@ def handle_errors_and_zip[_AsusWrtBridgeT: AsusWrtBridge]( if isinstance(data, dict): return dict(zip(keys, list(data.values()), strict=False)) - if not isinstance(data, (list, tuple)): - raise UpdateFailed("Received invalid data type") return dict(zip(keys, data, strict=False)) return _wrapper diff --git a/tests/components/asuswrt/test_sensor.py b/tests/components/asuswrt/test_sensor.py index f220b233b1d..6fa503c8e97 100644 --- a/tests/components/asuswrt/test_sensor.py +++ b/tests/components/asuswrt/test_sensor.py @@ -616,12 +616,10 @@ async def test_decorator_errors( config_entry.add_to_hass(hass) mock_available_temps[1] = True - connect_legacy.return_value.async_get_bytes_total.return_value = "bad_response" - connect_legacy.return_value.async_get_current_transfer_rates.return_value = ( - "bad_response" - ) - connect_legacy.return_value.async_get_temperature.return_value = "bad_response" - connect_legacy.return_value.async_get_loadavg.return_value = "bad_response" + connect_legacy.return_value.async_get_bytes_total.return_value = None + connect_legacy.return_value.async_get_current_transfer_rates.return_value = None + connect_legacy.return_value.async_get_temperature.return_value = None + connect_legacy.return_value.async_get_loadavg.return_value = None # initial devices setup assert await hass.config_entries.async_setup(config_entry.entry_id) @@ -631,7 +629,5 @@ async def test_decorator_errors( await hass.async_block_till_done() for sensor_name in sensors: - assert ( - hass.states.get(f"{sensor_prefix}_{slugify(sensor_name)}").state - == STATE_UNAVAILABLE - ) + sensor = hass.states.get(f"{sensor_prefix}_{slugify(sensor_name)}") + assert sensor and sensor.state == STATE_UNAVAILABLE