Do not probe nam
device if the host is already configured (#57843)
This commit is contained in:
parent
378c48da15
commit
0b932b53c9
2 changed files with 20 additions and 0 deletions
|
@ -74,6 +74,9 @@ class NAMFlowHandler(config_entries.ConfigFlow, domain=DOMAIN):
|
||||||
"""Handle zeroconf discovery."""
|
"""Handle zeroconf discovery."""
|
||||||
self.host = discovery_info[CONF_HOST]
|
self.host = discovery_info[CONF_HOST]
|
||||||
|
|
||||||
|
# Do not probe the device if the host is already configured
|
||||||
|
self._async_abort_entries_match({CONF_HOST: self.host})
|
||||||
|
|
||||||
try:
|
try:
|
||||||
mac = await self._async_get_mac(cast(str, self.host))
|
mac = await self._async_get_mac(cast(str, self.host))
|
||||||
except (ApiError, ClientConnectorError, asyncio.TimeoutError):
|
except (ApiError, ClientConnectorError, asyncio.TimeoutError):
|
||||||
|
|
|
@ -150,6 +150,23 @@ async def test_zeroconf(hass):
|
||||||
assert len(mock_setup_entry.mock_calls) == 1
|
assert len(mock_setup_entry.mock_calls) == 1
|
||||||
|
|
||||||
|
|
||||||
|
async def test_zeroconf_host_already_configured(hass):
|
||||||
|
"""Test that errors are shown when host is already configured."""
|
||||||
|
entry = MockConfigEntry(
|
||||||
|
domain=DOMAIN, unique_id="aa:bb:cc:dd:ee:ff", data=VALID_CONFIG
|
||||||
|
)
|
||||||
|
entry.add_to_hass(hass)
|
||||||
|
|
||||||
|
result = await hass.config_entries.flow.async_init(
|
||||||
|
DOMAIN,
|
||||||
|
data=DISCOVERY_INFO,
|
||||||
|
context={"source": SOURCE_ZEROCONF},
|
||||||
|
)
|
||||||
|
|
||||||
|
assert result["type"] == data_entry_flow.RESULT_TYPE_ABORT
|
||||||
|
assert result["reason"] == "already_configured"
|
||||||
|
|
||||||
|
|
||||||
@pytest.mark.parametrize(
|
@pytest.mark.parametrize(
|
||||||
"error",
|
"error",
|
||||||
[
|
[
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue