Address late review of tolo integration (#60453)

* improvements requested by @MartinHjelmare

* addressed requested changes

* more improvements
This commit is contained in:
Matthias Lohr 2021-11-28 16:50:57 +01:00 committed by GitHub
parent bae01ca7d8
commit 2d1d9e9e20
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
5 changed files with 5 additions and 9 deletions

View File

@ -64,7 +64,7 @@ class ToloSaunaUpdateCoordinator(DataUpdateCoordinator[ToloSaunaData]):
hass=hass,
logger=_LOGGER,
name=f"{entry.title} ({entry.data[CONF_HOST]}) Data Update Coordinator",
update_interval=timedelta(seconds=3),
update_interval=timedelta(seconds=5),
)
async def _async_update_data(self) -> ToloSaunaData:
@ -78,9 +78,9 @@ class ToloSaunaUpdateCoordinator(DataUpdateCoordinator[ToloSaunaData]):
settings = self.client.get_settings_info(
resend_timeout=DEFAULT_RETRY_TIMEOUT, retries=DEFAULT_RETRY_COUNT
)
return ToloSaunaData(status, settings)
except ResponseTimedOutError as error:
raise UpdateFailed("communication timeout") from error
return ToloSaunaData(status, settings)
class ToloSaunaCoordinatorEntity(CoordinatorEntity):

View File

@ -35,9 +35,9 @@ class ToloSaunaConfigFlow(ConfigFlow, domain=DOMAIN):
result = client.get_status_info(
resend_timeout=DEFAULT_RETRY_TIMEOUT, retries=DEFAULT_RETRY_COUNT
)
return result is not None
except ResponseTimedOutError:
return False
return result is not None
async def async_step_user(
self, user_input: dict[str, Any] | None = None

View File

@ -16,7 +16,6 @@
"cannot_connect": "[%key:common::config_flow::error::cannot_connect%]"
},
"abort": {
"no_devices_found": "[%key:common::config_flow::abort::no_devices_found%]",
"already_configured": "[%key:common::config_flow::abort::already_configured_device%]"
}
}

View File

@ -1,8 +1,7 @@
{
"config": {
"abort": {
"already_configured": "Device is already configured",
"no_devices_found": "No devices found on the network"
"already_configured": "Device is already configured"
},
"error": {
"cannot_connect": "Failed to connect"

View File

@ -27,9 +27,7 @@ def toloclient_fixture() -> Mock:
async def test_user_with_timed_out_host(hass: HomeAssistant, toloclient: Mock):
"""Test a user initiated config flow with provided host which times out."""
toloclient().get_status_info.side_effect = lambda *args, **kwargs: (
_ for _ in ()
).throw(ResponseTimedOutError())
toloclient().get_status_info.side_effect = ResponseTimedOutError()
result = await hass.config_entries.flow.async_init(
DOMAIN,