evlli/hass-core
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Get rid of name collision in iaqualink tests (#63642)

Browse source
This commit is contained in:
Florent Thoumie 2022-01-19 04:25:12 -08:00 • committed by GitHub
parent 8d7cca9774
commit 7520a3fd01
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 27 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
tests/components/iaqualink/conftest.py
View file

@ -55,9 +55,6 @@ def get_aqualink_device(system, cls=None, data=None):
if data is None: if data is None:
data = {} data = {}
num = random.randint(0, 999)
data["name"] = f"name_{num:03}"
return cls(system=system, data=data) return cls(system=system, data=data)

41
tests/components/iaqualink/test_init.py
View file

@ -67,7 +67,8 @@ async def test_setup_systems_exception(hass, config_entry):
config_entry.add_to_hass(hass) config_entry.add_to_hass(hass)
with patch( with patch(
"homeassistant.components.iaqualink.AqualinkClient.login", return_value=None "homeassistant.components.iaqualink.AqualinkClient.login",
return_value=None,
), patch( ), patch(
"homeassistant.components.iaqualink.AqualinkClient.get_systems", "homeassistant.components.iaqualink.AqualinkClient.get_systems",
side_effect=AqualinkServiceException, side_effect=AqualinkServiceException,
@ -83,7 +84,8 @@ async def test_setup_no_systems_recognized(hass, config_entry):
config_entry.add_to_hass(hass) config_entry.add_to_hass(hass)
with patch( with patch(
"homeassistant.components.iaqualink.AqualinkClient.login", return_value=None "homeassistant.components.iaqualink.AqualinkClient.login",
return_value=None,
), patch( ), patch(
"homeassistant.components.iaqualink.AqualinkClient.get_systems", "homeassistant.components.iaqualink.AqualinkClient.get_systems",
return_value={}, return_value={},
@ -102,7 +104,8 @@ async def test_setup_devices_exception(hass, config_entry, client):
systems = {system.serial: system} systems = {system.serial: system}
with patch( with patch(
"homeassistant.components.iaqualink.AqualinkClient.login", return_value=None "homeassistant.components.iaqualink.AqualinkClient.login",
return_value=None,
), patch( ), patch(
"homeassistant.components.iaqualink.AqualinkClient.get_systems", "homeassistant.components.iaqualink.AqualinkClient.get_systems",
return_value=systems, return_value=systems,
@ -123,11 +126,12 @@ async def test_setup_all_good_no_recognized_devices(hass, config_entry, client):
system = get_aqualink_system(client) system = get_aqualink_system(client)
systems = {system.serial: system} systems = {system.serial: system}
device = get_aqualink_device(system, AqualinkDevice) device = get_aqualink_device(system, AqualinkDevice, data={"name": "dev_1"})
devices = {device.name: device} devices = {device.name: device}
with patch( with patch(
"homeassistant.components.iaqualink.AqualinkClient.login", return_value=None "homeassistant.components.iaqualink.AqualinkClient.login",
return_value=None,
), patch( ), patch(
"homeassistant.components.iaqualink.AqualinkClient.get_systems", "homeassistant.components.iaqualink.AqualinkClient.get_systems",
return_value=systems, return_value=systems,
@ -160,18 +164,23 @@ async def test_setup_all_good_all_device_types(hass, config_entry, client):
systems = {system.serial: system} systems = {system.serial: system}
devices = [ devices = [
get_aqualink_device(system, AqualinkAuxToggle), get_aqualink_device(system, AqualinkAuxToggle, data={"name": "aux_1"}),
get_aqualink_device(system, AqualinkBinarySensor), get_aqualink_device(
get_aqualink_device(system, AqualinkLightToggle), system, AqualinkBinarySensor, data={"name": "freeze_protection"}
get_aqualink_device(system, AqualinkSensor), ),
get_aqualink_device(system, AqualinkThermostat), get_aqualink_device(system, AqualinkLightToggle, data={"name": "aux_2"}),
get_aqualink_device(system, AqualinkSensor, data={"name": "ph"}),
get_aqualink_device(
system, AqualinkThermostat, data={"name": "pool_set_point"}
),
] ]
devices = {d.name: d for d in devices} devices = {d.name: d for d in devices}
system.get_devices = AsyncMock(return_value=devices) system.get_devices = AsyncMock(return_value=devices)
with patch( with patch(
"homeassistant.components.iaqualink.AqualinkClient.login", return_value=None "homeassistant.components.iaqualink.AqualinkClient.login",
return_value=None,
), patch( ), patch(
"homeassistant.components.iaqualink.AqualinkClient.get_systems", "homeassistant.components.iaqualink.AqualinkClient.get_systems",
return_value=systems, return_value=systems,
@ -205,7 +214,8 @@ async def test_multiple_updates(hass, config_entry, caplog, client):
caplog.set_level(logging.WARNING) caplog.set_level(logging.WARNING)
with patch( with patch(
"homeassistant.components.iaqualink.AqualinkClient.login", return_value=None "homeassistant.components.iaqualink.AqualinkClient.login",
return_value=None,
), patch( ), patch(
"homeassistant.components.iaqualink.AqualinkClient.get_systems", "homeassistant.components.iaqualink.AqualinkClient.get_systems",
return_value=systems, return_value=systems,
@ -303,13 +313,16 @@ async def test_entity_assumed_and_available(hass, config_entry, client):
system = get_aqualink_system(client) system = get_aqualink_system(client)
systems = {system.serial: system} systems = {system.serial: system}
light = get_aqualink_device(system, AqualinkLightToggle, data={"state": "1"}) light = get_aqualink_device(
system, AqualinkLightToggle, data={"name": "aux_1", "state": "1"}
)
devices = {d.name: d for d in [light]} devices = {d.name: d for d in [light]}
system.get_devices = AsyncMock(return_value=devices) system.get_devices = AsyncMock(return_value=devices)
system.update = AsyncMock() system.update = AsyncMock()
with patch( with patch(
"homeassistant.components.iaqualink.AqualinkClient.login", return_value=None "homeassistant.components.iaqualink.AqualinkClient.login",
return_value=None,
), patch( ), patch(
"homeassistant.components.iaqualink.AqualinkClient.get_systems", "homeassistant.components.iaqualink.AqualinkClient.get_systems",
return_value=systems, return_value=systems,