Fix Lutron Caseta battery sensor crash on unsupported devices (#171829)

homeassistant/components/lutron_caseta/binary_sensor.py

@@ -5,7 +5,7 @@ from __future__ import annotations
 from datetime import timedelta
 from typing import Any
 
-from pylutron_caseta import OCCUPANCY_GROUP_OCCUPIED
+from pylutron_caseta import OCCUPANCY_GROUP_OCCUPIED, BridgeResponseError
 
 from homeassistant.components.binary_sensor import (
     BinarySensorDeviceClass,
@@ -133,7 +133,11 @@ class LutronCasetaBatterySensor(LutronCasetaEntity, BinarySensorEntity):
 
     async def async_update(self) -> None:
         """Fetch the latest battery status from the bridge."""
-        status = await self._smartbridge.get_battery_status(self.device_id)
+        try:
+            status = await self._smartbridge.get_battery_status(self.device_id)
+        except BridgeResponseError:
+            self._attr_is_on = None
+            return
         normalized_status = status.strip().casefold() if status else None
         if normalized_status == BATTERY_STATUS_LOW:
             self._attr_is_on = True

tests/components/lutron_caseta/test_binary_sensor.py

@@ -1,9 +1,10 @@
 """Tests for the Lutron Caseta binary sensors."""
 
 from typing import Any
-from unittest.mock import AsyncMock
+from unittest.mock import AsyncMock, MagicMock
 
 from freezegun.api import FrozenDateTimeFactory
+from pylutron_caseta import BridgeResponseError
 
 from homeassistant.components.binary_sensor import BinarySensorDeviceClass
 from homeassistant.components.lutron_caseta.binary_sensor import SCAN_INTERVAL
@@ -114,3 +115,32 @@ async def test_battery_sensor_updates_on_schedule(
     assert unknown_state is not None
     assert unknown_state.state == STATE_UNKNOWN
     assert instance.get_battery_status.await_count == 3
+
+
+async def test_battery_sensor_handles_bridge_response_error(
+    hass: HomeAssistant,
+) -> None:
+    """Test battery sensor handles BridgeResponseError gracefully.
+
+    Regression test for https://github.com/home-assistant/core/issues/169965
+    """
+    instance = MockBridge()
+
+    def factory(*args: Any, **kwargs: Any) -> MockBridge:
+        """Return the mock bridge instance."""
+        return instance
+
+    mock_response = MagicMock()
+    mock_response.Header.StatusCode = "404 NotFound"
+    instance.get_battery_status = AsyncMock(
+        side_effect=BridgeResponseError(mock_response)
+    )
+
+    await async_setup_integration(hass, factory)
+    await hass.async_block_till_done()
+
+    state = hass.states.get(
+        "binary_sensor.basement_bedroom_basement_bedroom_left_shade_battery"
+    )
+    assert state is not None
+    assert state.state == STATE_UNKNOWN