Add suggestions

homeassistant/components/plugwise/climate.py

@@ -233,7 +233,10 @@ async def async_set_temperature(self, **kwargs: Any) -> None:
             if temperature is None or not (
                 self._attr_min_temp <= temperature <= self._attr_max_temp
             ):
-                raise ValueError("Invalid temperature change requested")
+                raise ServiceValidationError(
+                    translation_domain=DOMAIN,
+                    translation_key="invalid_temperature_change_requested",
+                )
 
         if mode := kwargs.get(ATTR_HVAC_MODE):
             await self.async_set_hvac_mode(mode)
@@ -244,7 +247,15 @@ async def async_set_temperature(self, **kwargs: Any) -> None:
     async def async_set_hvac_mode(self, hvac_mode: HVACMode) -> None:
         """Set the hvac mode."""
         if hvac_mode not in self.hvac_modes:
-            raise ServiceValidationError("Unsupported hvac_mode")
+            hvac_modes = ", ".join(self.hvac_modes)
+            raise ServiceValidationError(
+                translation_domain=DOMAIN,
+                translation_key="unsupported_hvac_mode_requested",
+                translation_placeholders={
+                    "hvac_mode": hvac_mode,
+                    "hvac_modes": hvac_modes,
+                },
+            )
 
         if hvac_mode == self.hvac_mode:
             return

homeassistant/components/plugwise/quality_scale.yaml

@@ -64,14 +64,12 @@ rules:
   diagnostics: done
   exception-translations:
     status: todo
-    comment: Add coordinator, util and climate exceptions
+    comment: Add coordinator, util exceptions (climate done in core 132175)
   icon-translations: done
   reconfiguration-flow:
     status: todo
     comment: This integration does not have any reconfiguration steps (yet) investigate how/why
-  dynamic-devices:
-    status: todo
-    comment: Investigate missing logic to button for unloading and creation
+  dynamic-devices: done
   discovery-update-info: done
   repair-issues:
     status: exempt

homeassistant/components/plugwise/strings.json

@@ -284,5 +284,13 @@
         "name": "Relay"
       }
     }
+  },
+  "exceptions": {
+    "invalid_temperature_change_requested": {
+      "message": "Invalid temperature change requested."
+    },
+    "unsupported_hvac_mode_requested": {
+      "message": "Unsupported mode {hvac_mode} requested, valid modes are: {hvac_modes}."
+    }
   }
 }

tests/components/plugwise/test_climate.py

@@ -233,7 +233,7 @@ async def test_adam_climate_entity_climate_changes(
         "c50f167537524366a5af7aa3942feb1e", "off"
     )
 
-    with pytest.raises(HomeAssistantError):
+    with pytest.raises(ServiceValidationError):
         await hass.services.async_call(
             CLIMATE_DOMAIN,
             SERVICE_SET_HVAC_MODE,