diff --git a/.coveragerc b/.coveragerc index 07e31e07961..b7d4b0bc68b 100644 --- a/.coveragerc +++ b/.coveragerc @@ -406,8 +406,6 @@ omit = homeassistant/components/fjaraskupan/sensor.py homeassistant/components/fleetgo/device_tracker.py homeassistant/components/flexit/climate.py - homeassistant/components/flexit_bacnet/__init__.py - homeassistant/components/flexit_bacnet/const.py homeassistant/components/flexit_bacnet/climate.py homeassistant/components/flic/binary_sensor.py homeassistant/components/flick_electric/__init__.py diff --git a/homeassistant/components/flexit_bacnet/__init__.py b/homeassistant/components/flexit_bacnet/__init__.py index c9a0b332d93..53ae1bbe775 100644 --- a/homeassistant/components/flexit_bacnet/__init__.py +++ b/homeassistant/components/flexit_bacnet/__init__.py @@ -25,7 +25,7 @@ async def async_setup_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool: await device.update() except (asyncio.exceptions.TimeoutError, ConnectionError, DecodingError) as exc: raise ConfigEntryNotReady( - f"Timeout while connecting to {entry.data['address']}" + f"Timeout while connecting to {entry.data[CONF_IP_ADDRESS]}" ) from exc hass.data.setdefault(DOMAIN, {})[entry.entry_id] = device diff --git a/tests/components/flexit_bacnet/test_init.py b/tests/components/flexit_bacnet/test_init.py index 35f0e04be67..71f79f54302 100644 --- a/tests/components/flexit_bacnet/test_init.py +++ b/tests/components/flexit_bacnet/test_init.py @@ -1,5 +1,7 @@ """Tests for the Flexit Nordic (BACnet) __init__.""" -from homeassistant.components.flexit_bacnet.const import DOMAIN as FLEXIT_BACNET_DOMAIN +from flexit_bacnet import DecodingError + +from homeassistant.components.flexit_bacnet.const import DOMAIN from homeassistant.config_entries import ConfigEntryState from homeassistant.const import Platform from homeassistant.core import HomeAssistant @@ -14,11 +16,20 @@ async def test_loading_and_unloading_config_entry( """Test loading and unloading a config entry.""" await setup_with_selected_platforms(hass, mock_config_entry, [Platform.CLIMATE]) - assert len(hass.config_entries.async_entries(FLEXIT_BACNET_DOMAIN)) == 1 + assert len(hass.config_entries.async_entries(DOMAIN)) == 1 assert mock_config_entry.state == ConfigEntryState.LOADED assert await hass.config_entries.async_unload(mock_config_entry.entry_id) await hass.async_block_till_done() assert mock_config_entry.state is ConfigEntryState.NOT_LOADED - assert not hass.data.get(FLEXIT_BACNET_DOMAIN) + + +async def test_failed_initialization( + hass: HomeAssistant, mock_config_entry: MockConfigEntry, mock_flexit_bacnet +) -> None: + """Test failed initialization.""" + mock_flexit_bacnet.update.side_effect = DecodingError + mock_config_entry.add_to_hass(hass) + assert not await hass.config_entries.async_setup(mock_config_entry.entry_id) + assert mock_config_entry.state == ConfigEntryState.SETUP_RETRY