From 904c3291d98f3368255585d25c98839d7b6d73b7 Mon Sep 17 00:00:00 2001
From: Duco Sebel <74970928+DCSBL@users.noreply.github.com>
Date: Mon, 25 Nov 2024 10:23:07 +0100
Subject: [PATCH] Add exception translation in HomeWizard coordinator (#131404)

---
 homeassistant/components/homewizard/coordinator.py     | 8 ++++++--
 homeassistant/components/homewizard/quality_scale.yaml | 7 +------
 homeassistant/components/homewizard/strings.json       | 4 ++--
 tests/components/homewizard/test_button.py             | 2 +-
 tests/components/homewizard/test_number.py             | 2 +-
 tests/components/homewizard/test_switch.py             | 4 ++--
 6 files changed, 13 insertions(+), 14 deletions(-)

diff --git a/homeassistant/components/homewizard/coordinator.py b/homeassistant/components/homewizard/coordinator.py
index 4a6b8edbca4..8f5045d3b94 100644
--- a/homeassistant/components/homewizard/coordinator.py
+++ b/homeassistant/components/homewizard/coordinator.py
@@ -66,7 +66,9 @@ class HWEnergyDeviceUpdateCoordinator(DataUpdateCoordinator[DeviceResponseEntry]
                     )
 
         except RequestError as ex:
-            raise UpdateFailed(ex) from ex
+            raise UpdateFailed(
+                ex, translation_domain=DOMAIN, translation_key="communication_error"
+            ) from ex
 
         except DisabledError as ex:
             if not self.api_disabled:
@@ -79,7 +81,9 @@ class HWEnergyDeviceUpdateCoordinator(DataUpdateCoordinator[DeviceResponseEntry]
                         self.config_entry.entry_id
                     )
 
-            raise UpdateFailed(ex) from ex
+            raise UpdateFailed(
+                ex, translation_domain=DOMAIN, translation_key="api_disabled"
+            ) from ex
 
         self.api_disabled = False
 
diff --git a/homeassistant/components/homewizard/quality_scale.yaml b/homeassistant/components/homewizard/quality_scale.yaml
index 175a5b6c797..281157465fc 100644
--- a/homeassistant/components/homewizard/quality_scale.yaml
+++ b/homeassistant/components/homewizard/quality_scale.yaml
@@ -67,12 +67,7 @@ rules:
   entity-device-class: done
   entity-disabled-by-default: done
   entity-translations: done
-  exception-translations:
-    status: todo
-    comment: |
-      While the integration provides some of the exception translations, the
-      translation for the error raised in the update error of the coordinator
-      is missing.
+  exception-translations: done
   icon-translations: done
   reconfiguration-flow: todo
   repair-issues:
diff --git a/homeassistant/components/homewizard/strings.json b/homeassistant/components/homewizard/strings.json
index 06959fa47a5..b3fd5a1fef2 100644
--- a/homeassistant/components/homewizard/strings.json
+++ b/homeassistant/components/homewizard/strings.json
@@ -20,7 +20,7 @@
       }
     },
     "error": {
-      "api_not_enabled": "The API is not enabled. Enable API in the HomeWizard Energy App under settings",
+      "api_not_enabled": "The local API is disabled. Go to the HomeWizard Energy app and enable the API in the device settings.",
       "network_error": "Device unreachable, make sure that you have entered the correct IP address and that the device is available in your network"
     },
     "abort": {
@@ -123,7 +123,7 @@
   },
   "exceptions": {
     "api_disabled": {
-      "message": "The local API of the HomeWizard device is disabled"
+      "message": "The local API is disabled."
     },
     "communication_error": {
       "message": "An error occurred while communicating with HomeWizard device"
diff --git a/tests/components/homewizard/test_button.py b/tests/components/homewizard/test_button.py
index 928e6f21901..d0a6d92b36f 100644
--- a/tests/components/homewizard/test_button.py
+++ b/tests/components/homewizard/test_button.py
@@ -79,7 +79,7 @@ async def test_identify_button(
 
     with pytest.raises(
         HomeAssistantError,
-        match=r"^The local API of the HomeWizard device is disabled$",
+        match=r"^The local API is disabled$",
     ):
         await hass.services.async_call(
             button.DOMAIN,
diff --git a/tests/components/homewizard/test_number.py b/tests/components/homewizard/test_number.py
index ff27fb1b257..ddadf09bb6e 100644
--- a/tests/components/homewizard/test_number.py
+++ b/tests/components/homewizard/test_number.py
@@ -85,7 +85,7 @@ async def test_number_entities(
     mock_homewizardenergy.state_set.side_effect = DisabledError
     with pytest.raises(
         HomeAssistantError,
-        match=r"^The local API of the HomeWizard device is disabled$",
+        match=r"^The local API is disabled$",
     ):
         await hass.services.async_call(
             number.DOMAIN,
diff --git a/tests/components/homewizard/test_switch.py b/tests/components/homewizard/test_switch.py
index b9e812620e8..d9f1ac26b4f 100644
--- a/tests/components/homewizard/test_switch.py
+++ b/tests/components/homewizard/test_switch.py
@@ -174,7 +174,7 @@ async def test_switch_entities(
 
     with pytest.raises(
         HomeAssistantError,
-        match=r"^The local API of the HomeWizard device is disabled$",
+        match=r"^The local API is disabled$",
     ):
         await hass.services.async_call(
             switch.DOMAIN,
@@ -185,7 +185,7 @@ async def test_switch_entities(
 
     with pytest.raises(
         HomeAssistantError,
-        match=r"^The local API of the HomeWizard device is disabled$",
+        match=r"^The local API is disabled$",
     ):
         await hass.services.async_call(
             switch.DOMAIN,