Add PlugwiseError exception, adapt to this change

Author: bouwew

plugwise/exceptions.py

@@ -59,6 +59,10 @@ class InvalidSetupError(PlugwiseException):
     """Raised when adding an Anna while an Adam exists."""
 
 
+class PlugwiseError(PlugwiseException):
+    """Raise when a non-specific error happens."""
+
+
 class UnsupportedDeviceError(PlugwiseException):
     """Raised when device is not supported."""
 

plugwise/smile.py

@@ -34,7 +34,7 @@
 from .exceptions import (
     ConnectionFailedError,
     InvalidSetupError,
-    PlugwiseException,
+    PlugwiseError,
     UnsupportedDeviceError,
 )
 from .helper import SmileComm, SmileHelper, update_helper
@@ -467,7 +467,7 @@ async def _set_schedule_state_legacy(self, name: str, status: str) -> bool:
                 schedule_rule_id = rule.attrib["id"]
 
         if schedule_rule_id is None:
-            raise PlugwiseException("No schedule available.")
+            raise PlugwiseError("No schedule available.")
 
         state = "false"
         if status == "on":
@@ -494,7 +494,7 @@ async def set_schedule_state(self, loc_id: str, name: str, state: str) -> bool:
 
         schedule_rule = self._rule_ids_by_name(name, loc_id)
         if not schedule_rule or schedule_rule is None:
-            raise PlugwiseException("No schedule available.")
+            raise PlugwiseError("No schedule with this name available.")
 
         schedule_rule_id: str = next(iter(schedule_rule))
 
@@ -533,7 +533,7 @@ async def _set_preset_legacy(self, preset: str) -> bool:
         """Set the given Preset on the relevant Thermostat - from DOMAIN_OBJECTS."""
         locator = f'rule/directives/when/then[@icon="{preset}"].../.../...'
         if (rule := self._domain_objects.find(locator)) is None:
-            raise PlugwiseException("No presets available.")
+            raise PlugwiseError("Invalid preset.")
 
         uri = RULES
         data = f'<rules><rule id="{rule.attrib["id"]}"><active>true</active></rule></rules>'
@@ -550,7 +550,7 @@ async def set_preset(self, loc_id: str, preset: str) -> bool:
         location_type = current_location.find("type").text
 
         if preset not in self._presets(loc_id):
-            raise PlugwiseException("Preset not available.")
+            raise PlugwiseError("Invalid preset.")
 
         uri = f"{LOCATIONS};id={loc_id}"
         data = (
@@ -637,14 +637,14 @@ async def set_switch_state(
             lock_state: str = self._appliances.find(locator).text
             # Don't bother switching a relay when the corresponding lock-state is true
             if lock_state == "true":
-                raise PlugwiseException("Cannot switch a locked Relay.")
+                raise PlugwiseError("Cannot switch a locked Relay.")
 
         await self._request(uri, method="put", data=data)
 
     async def set_regulation_mode(self, mode: str) -> bool:
         """Set the heating regulation mode."""
         if mode not in self._allowed_modes:
-            raise PlugwiseException("Invalid regulation mode.")
+            raise PlugwiseError("Invalid regulation mode.")
 
         uri = f"{APPLIANCES};type=gateway/regulation_mode_control"
         duration = ""