Revert "Use water_sense_mode device API for smart program on/off"

This reverts commit 54a14d6.

Author: sebr

custom_components/bhyve/pybhyve/client.py

@@ -249,13 +249,6 @@ async def update_program(self, program_id: str, program: BHyveTimerProgram) -> N
         json = {"sprinkler_timer_program": program}
         await self._request("put", path, json=json)
 
-    async def update_device(self, device: dict) -> None:
-        """Update device settings."""
-        device_id = device.get("id")
-        path = f"{DEVICES_PATH}/{device_id}"
-        json = {"device": device}
-        await self._request("put", path, json=json)
-
     async def send_message(self, payload: Any) -> None:
         """Send a message via the websocket."""
         if self._websocket is not None:

custom_components/bhyve/switch.py

@@ -217,35 +217,17 @@ def is_on(self) -> bool:
 
     async def async_turn_on(self, **_kwargs: Any) -> None:
         """Turn the switch on."""
-        if self.program_data.get("is_smart_program"):
-            await self.coordinator.client.update_device(
-                {
-                    "id": self._device_id,
-                    "type": self._device_type,
-                    "mac_address": self._mac_address,
-                    "water_sense_mode": "auto",
-                }
-            )
-        else:
-            program = BHyveTimerProgram(self.program_data)
-            program["enabled"] = True
-            await self.coordinator.client.update_program(self._program_id, program)
+        program = BHyveTimerProgram(self.program_data)
+        program["enabled"] = True
+        await self.coordinator.client.update_program(self._program_id, program)
+        # Coordinator updates via WebSocket event
 
     async def async_turn_off(self, **_kwargs: Any) -> None:
         """Turn the switch off."""
-        if self.program_data.get("is_smart_program"):
-            await self.coordinator.client.update_device(
-                {
-                    "id": self._device_id,
-                    "type": self._device_type,
-                    "mac_address": self._mac_address,
-                    "water_sense_mode": "off",
-                }
-            )
-        else:
-            program = BHyveTimerProgram(self.program_data)
-            program["enabled"] = False
-            await self.coordinator.client.update_program(self._program_id, program)
+        program = BHyveTimerProgram(self.program_data)
+        program["enabled"] = False
+        await self.coordinator.client.update_program(self._program_id, program)
+        # Coordinator updates via WebSocket event
 
     async def start_program(self) -> None:
         """Begins running a program."""

tests/test_switch.py

@@ -37,7 +37,6 @@ def create_mock_coordinator(devices: dict, programs: dict) -> MagicMock:
     coordinator.client = MagicMock()
     coordinator.client.send_message = AsyncMock()
     coordinator.client.update_program = AsyncMock()
-    coordinator.client.update_device = AsyncMock()
     return coordinator
 
 
@@ -413,14 +412,14 @@ async def test_program_switch_turn_on(
             description=description,
         )
 
-        # Turn on the switch (smart program uses update_device)
+        # Turn on the switch
         await switch.async_turn_on()
 
-        # Verify update_device was called with water_sense_mode=auto
-        coordinator.client.update_device.assert_called_once()
-        call_args = coordinator.client.update_device.call_args[0][0]
-        assert call_args["id"] == TEST_DEVICE_ID
-        assert call_args["water_sense_mode"] == "auto"
+        # Verify update_program was called with enabled=True
+        coordinator.client.update_program.assert_called_once()
+        call_args = coordinator.client.update_program.call_args[0]
+        assert call_args[0] == TEST_PROGRAM_ID
+        assert call_args[1]["enabled"] is True
 
     async def test_program_switch_turn_off(
         self,