From 6464ab8356b4beb80a1d7341a9ef38422728f4ce Mon Sep 17 00:00:00 2001 From: G Johansson Date: Sat, 19 Feb 2022 19:00:49 +0100 Subject: [PATCH] Bump pysensibo to 1.0.4 (#66886) --- homeassistant/components/sensibo/climate.py | 3 ++- homeassistant/components/sensibo/config_flow.py | 4 +++- homeassistant/components/sensibo/coordinator.py | 7 ++++--- homeassistant/components/sensibo/manifest.json | 4 ++-- requirements_all.txt | 2 +- requirements_test_all.txt | 2 +- tests/components/sensibo/test_config_flow.py | 3 ++- 7 files changed, 15 insertions(+), 10 deletions(-) diff --git a/homeassistant/components/sensibo/climate.py b/homeassistant/components/sensibo/climate.py index 0963a4f927e..a9629ba42f4 100644 --- a/homeassistant/components/sensibo/climate.py +++ b/homeassistant/components/sensibo/climate.py @@ -5,7 +5,7 @@ import asyncio from aiohttp.client_exceptions import ClientConnectionError import async_timeout -from pysensibo import SensiboError +from pysensibo.exceptions import AuthenticationError, SensiboError import voluptuous as vol from homeassistant.components.climate import ( @@ -318,6 +318,7 @@ class SensiboClimate(CoordinatorEntity, ClimateEntity): except ( ClientConnectionError, asyncio.TimeoutError, + AuthenticationError, SensiboError, ) as err: raise HomeAssistantError( diff --git a/homeassistant/components/sensibo/config_flow.py b/homeassistant/components/sensibo/config_flow.py index 8544972baee..f970581e2a8 100644 --- a/homeassistant/components/sensibo/config_flow.py +++ b/homeassistant/components/sensibo/config_flow.py @@ -6,7 +6,8 @@ import logging import aiohttp import async_timeout -from pysensibo import SensiboClient, SensiboError +from pysensibo import SensiboClient +from pysensibo.exceptions import AuthenticationError, SensiboError import voluptuous as vol from homeassistant import config_entries @@ -42,6 +43,7 @@ async def async_validate_api(hass: HomeAssistant, api_key: str) -> bool: except ( aiohttp.ClientConnectionError, asyncio.TimeoutError, + AuthenticationError, SensiboError, ) as err: _LOGGER.error("Failed to get devices from Sensibo servers %s", err) diff --git a/homeassistant/components/sensibo/coordinator.py b/homeassistant/components/sensibo/coordinator.py index 41c44e741e9..c79fb5b7bb5 100644 --- a/homeassistant/components/sensibo/coordinator.py +++ b/homeassistant/components/sensibo/coordinator.py @@ -4,7 +4,8 @@ from __future__ import annotations from datetime import timedelta from typing import Any -import pysensibo +from pysensibo import SensiboClient +from pysensibo.exceptions import AuthenticationError, SensiboError from homeassistant.config_entries import ConfigEntry from homeassistant.const import CONF_API_KEY @@ -20,7 +21,7 @@ class SensiboDataUpdateCoordinator(DataUpdateCoordinator): def __init__(self, hass: HomeAssistant, entry: ConfigEntry) -> None: """Initialize the Sensibo coordinator.""" - self.client = pysensibo.SensiboClient( + self.client = SensiboClient( entry.data[CONF_API_KEY], session=async_get_clientsession(hass), timeout=TIMEOUT, @@ -39,7 +40,7 @@ class SensiboDataUpdateCoordinator(DataUpdateCoordinator): try: for dev in await self.client.async_get_devices(): devices.append(dev) - except (pysensibo.SensiboError) as error: + except (AuthenticationError, SensiboError) as error: raise UpdateFailed from error device_data: dict[str, dict[str, Any]] = {} diff --git a/homeassistant/components/sensibo/manifest.json b/homeassistant/components/sensibo/manifest.json index 71fd3e1c241..e17c5339d67 100644 --- a/homeassistant/components/sensibo/manifest.json +++ b/homeassistant/components/sensibo/manifest.json @@ -2,7 +2,7 @@ "domain": "sensibo", "name": "Sensibo", "documentation": "https://www.home-assistant.io/integrations/sensibo", - "requirements": ["pysensibo==1.0.3"], + "requirements": ["pysensibo==1.0.4"], "config_flow": true, "codeowners": ["@andrey-git", "@gjohansson-ST"], "iot_class": "cloud_polling", @@ -11,6 +11,6 @@ }, "dhcp": [ {"hostname":"sensibo*"} - ], + ], "loggers": ["pysensibo"] } diff --git a/requirements_all.txt b/requirements_all.txt index 9adb28cc685..bea0716741f 100644 --- a/requirements_all.txt +++ b/requirements_all.txt @@ -1818,7 +1818,7 @@ pysaj==0.0.16 pysdcp==1 # homeassistant.components.sensibo -pysensibo==1.0.3 +pysensibo==1.0.4 # homeassistant.components.serial # homeassistant.components.zha diff --git a/requirements_test_all.txt b/requirements_test_all.txt index 1afbb1adff8..7acb27ca344 100644 --- a/requirements_test_all.txt +++ b/requirements_test_all.txt @@ -1160,7 +1160,7 @@ pyrituals==0.0.6 pyruckus==0.12 # homeassistant.components.sensibo -pysensibo==1.0.3 +pysensibo==1.0.4 # homeassistant.components.serial # homeassistant.components.zha diff --git a/tests/components/sensibo/test_config_flow.py b/tests/components/sensibo/test_config_flow.py index cf3716f09e4..9c59fc70763 100644 --- a/tests/components/sensibo/test_config_flow.py +++ b/tests/components/sensibo/test_config_flow.py @@ -5,7 +5,7 @@ import asyncio from unittest.mock import patch import aiohttp -from pysensibo import SensiboError +from pysensibo import AuthenticationError, SensiboError import pytest from homeassistant import config_entries @@ -123,6 +123,7 @@ async def test_import_flow_already_exist(hass: HomeAssistant) -> None: [ (aiohttp.ClientConnectionError), (asyncio.TimeoutError), + (AuthenticationError), (SensiboError), ], )