Add KEY_HOSTNAME for Daikin zeroconf (home-assistant#36253)

* Add KEY_HOSTNAME for Daikin zeroconf

* Update tests and use CONF_HOST as hostname

Author: fredrike

homeassistant/components/daikin/config_flow.py

@@ -11,7 +11,7 @@
 from homeassistant import config_entries
 from homeassistant.const import CONF_HOST, CONF_PASSWORD
 
-from .const import CONF_KEY, CONF_UUID, KEY_IP, KEY_MAC, TIMEOUT
+from .const import CONF_KEY, CONF_UUID, KEY_HOSTNAME, KEY_IP, KEY_MAC, TIMEOUT
 
 _LOGGER = logging.getLogger(__name__)
 
@@ -128,7 +128,7 @@ async def async_step_discovery(self, discovery_info):
     async def async_step_zeroconf(self, discovery_info):
         """Prepare configuration for a discovered Daikin device."""
         _LOGGER.debug("Zeroconf discovery_info: %s", discovery_info)
-        await self.async_set_unique_id(discovery_info.get(CONF_HOST))
+        await self.async_set_unique_id(discovery_info[KEY_HOSTNAME])
         self._abort_if_unique_id_configured()
-        self.host = discovery_info.get(CONF_HOST)
+        self.host = discovery_info[CONF_HOST]
         return await self.async_step_user()

homeassistant/components/daikin/const.py

@@ -64,5 +64,6 @@
 
 KEY_MAC = "mac"
 KEY_IP = "ip"
+KEY_HOSTNAME = "hostname"
 
 TIMEOUT = 60

tests/components/daikin/test_config_flow.py

@@ -6,7 +6,7 @@
 from aiohttp.web_exceptions import HTTPForbidden
 import pytest
 
-from homeassistant.components.daikin.const import KEY_IP, KEY_MAC
+from homeassistant.components.daikin.const import KEY_HOSTNAME, KEY_IP, KEY_MAC
 from homeassistant.config_entries import (
     SOURCE_DISCOVERY,
     SOURCE_IMPORT,
@@ -112,7 +112,11 @@ async def test_device_abort(hass, mock_daikin, s_effect, reason):
     "source, data, unique_id",
     [
         (SOURCE_DISCOVERY, {KEY_IP: HOST, KEY_MAC: MAC}, MAC),
-        (SOURCE_ZEROCONF, {CONF_HOST: HOST}, HOST),
+        (
+            SOURCE_ZEROCONF,
+            {CONF_HOST: HOST, KEY_HOSTNAME: "DaikinUNIQE.local"},
+            "DaikinUNIQE.local",
+        ),
     ],
 )
 async def test_discovery_zeroconf(hass, mock_daikin, source, data, unique_id):