From abb8c58b87047c4bd3b7e6b6ad24644b0fed4ef1 Mon Sep 17 00:00:00 2001 From: epenet <6771947+epenet@users.noreply.github.com> Date: Wed, 12 Jun 2024 12:35:01 +0200 Subject: [PATCH] Fix consider-using-tuple pylint warnings in core tests (#119463) --- .../alarm_control_panel/test_device_action.py | 2 +- .../test_device_condition.py | 2 +- .../test_device_trigger.py | 2 +- .../binary_sensor/test_device_condition.py | 2 +- .../binary_sensor/test_device_trigger.py | 2 +- tests/components/button/test_device_action.py | 2 +- .../components/button/test_device_trigger.py | 2 +- tests/components/climate/common.py | 8 +++---- .../components/climate/test_device_action.py | 2 +- .../climate/test_device_condition.py | 2 +- .../components/climate/test_device_trigger.py | 8 +++---- tests/components/cover/test_device_action.py | 2 +- .../components/cover/test_device_condition.py | 2 +- tests/components/cover/test_device_trigger.py | 2 +- .../device_tracker/test_device_condition.py | 4 ++-- .../device_tracker/test_device_trigger.py | 4 ++-- tests/components/fan/common.py | 22 +++++++++---------- tests/components/fan/test_device_action.py | 4 ++-- tests/components/fan/test_device_condition.py | 4 ++-- tests/components/fan/test_device_trigger.py | 4 ++-- tests/components/group/common.py | 4 ++-- tests/components/group/test_init.py | 4 ++-- .../humidifier/test_device_condition.py | 2 +- tests/components/light/common.py | 12 +++++----- tests/components/light/test_device_action.py | 6 ++--- .../components/light/test_device_condition.py | 4 ++-- tests/components/light/test_device_trigger.py | 4 ++-- tests/components/lock/test_device_action.py | 2 +- .../components/lock/test_device_condition.py | 8 +++---- tests/components/lock/test_device_trigger.py | 8 +++---- tests/components/remote/test_device_action.py | 4 ++-- .../remote/test_device_condition.py | 4 ++-- .../components/remote/test_device_trigger.py | 4 ++-- tests/components/select/test_device_action.py | 8 +++---- .../select/test_device_condition.py | 2 +- .../components/select/test_device_trigger.py | 2 +- .../sensor/test_device_condition.py | 2 +- .../components/sensor/test_device_trigger.py | 2 +- tests/components/sensor/test_recorder.py | 18 +++++++-------- tests/components/switch/test_device_action.py | 4 ++-- .../switch/test_device_condition.py | 4 ++-- .../components/switch/test_device_trigger.py | 4 ++-- tests/components/vacuum/test_device_action.py | 4 ++-- .../vacuum/test_device_condition.py | 4 ++-- .../components/vacuum/test_device_trigger.py | 4 ++-- tests/components/water_heater/common.py | 4 ++-- .../water_heater/test_device_action.py | 4 ++-- tests/helpers/test_config_validation.py | 8 +++---- tests/helpers/test_entity_platform.py | 2 +- tests/test_config.py | 16 +++++++------- tests/test_core.py | 16 +++++++------- 51 files changed, 128 insertions(+), 128 deletions(-) diff --git a/tests/components/alarm_control_panel/test_device_action.py b/tests/components/alarm_control_panel/test_device_action.py index 04c0e3b045b..9c5aaffd733 100644 --- a/tests/components/alarm_control_panel/test_device_action.py +++ b/tests/components/alarm_control_panel/test_device_action.py @@ -174,7 +174,7 @@ async def test_get_actions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for action in ["disarm", "arm_away"] + for action in ("disarm", "arm_away") ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id diff --git a/tests/components/alarm_control_panel/test_device_condition.py b/tests/components/alarm_control_panel/test_device_condition.py index 9f8f56ccb6f..da1d77f50a3 100644 --- a/tests/components/alarm_control_panel/test_device_condition.py +++ b/tests/components/alarm_control_panel/test_device_condition.py @@ -167,7 +167,7 @@ async def test_get_conditions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for condition in ["is_disarmed", "is_triggered"] + for condition in ("is_disarmed", "is_triggered") ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id diff --git a/tests/components/alarm_control_panel/test_device_trigger.py b/tests/components/alarm_control_panel/test_device_trigger.py index 6be15cca097..46eba314dc1 100644 --- a/tests/components/alarm_control_panel/test_device_trigger.py +++ b/tests/components/alarm_control_panel/test_device_trigger.py @@ -162,7 +162,7 @@ async def test_get_triggers_hidden_auxiliary( "entity_id": entry.id, "metadata": {"secondary": True}, } - for trigger in ["triggered", "disarmed", "arming"] + for trigger in ("triggered", "disarmed", "arming") ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id diff --git a/tests/components/binary_sensor/test_device_condition.py b/tests/components/binary_sensor/test_device_condition.py index 7d7b4f62c87..c2bd29fad36 100644 --- a/tests/components/binary_sensor/test_device_condition.py +++ b/tests/components/binary_sensor/test_device_condition.py @@ -122,7 +122,7 @@ async def test_get_conditions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for condition in ["is_on", "is_off"] + for condition in ("is_on", "is_off") ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id diff --git a/tests/components/binary_sensor/test_device_trigger.py b/tests/components/binary_sensor/test_device_trigger.py index 2ecd17fd0d1..f91a336061d 100644 --- a/tests/components/binary_sensor/test_device_trigger.py +++ b/tests/components/binary_sensor/test_device_trigger.py @@ -122,7 +122,7 @@ async def test_get_triggers_hidden_auxiliary( "entity_id": entry.id, "metadata": {"secondary": True}, } - for trigger in ["turned_on", "turned_off"] + for trigger in ("turned_on", "turned_off") ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id diff --git a/tests/components/button/test_device_action.py b/tests/components/button/test_device_action.py index c3ba03b60e6..837a433c87c 100644 --- a/tests/components/button/test_device_action.py +++ b/tests/components/button/test_device_action.py @@ -88,7 +88,7 @@ async def test_get_actions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for action in ["press"] + for action in ("press",) ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id diff --git a/tests/components/button/test_device_trigger.py b/tests/components/button/test_device_trigger.py index 1d9a84b0e8f..dee8045a71f 100644 --- a/tests/components/button/test_device_trigger.py +++ b/tests/components/button/test_device_trigger.py @@ -97,7 +97,7 @@ async def test_get_triggers_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for trigger in ["pressed"] + for trigger in ("pressed",) ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id diff --git a/tests/components/climate/common.py b/tests/components/climate/common.py index 20f6bfd880d..c890d3a7bb5 100644 --- a/tests/components/climate/common.py +++ b/tests/components/climate/common.py @@ -86,13 +86,13 @@ async def async_set_temperature( """Set new target temperature.""" kwargs = { key: value - for key, value in [ + for key, value in ( (ATTR_TEMPERATURE, temperature), (ATTR_TARGET_TEMP_HIGH, target_temp_high), (ATTR_TARGET_TEMP_LOW, target_temp_low), (ATTR_ENTITY_ID, entity_id), (ATTR_HVAC_MODE, hvac_mode), - ] + ) if value is not None } _LOGGER.debug("set_temperature start data=%s", kwargs) @@ -113,13 +113,13 @@ def set_temperature( """Set new target temperature.""" kwargs = { key: value - for key, value in [ + for key, value in ( (ATTR_TEMPERATURE, temperature), (ATTR_TARGET_TEMP_HIGH, target_temp_high), (ATTR_TARGET_TEMP_LOW, target_temp_low), (ATTR_ENTITY_ID, entity_id), (ATTR_HVAC_MODE, hvac_mode), - ] + ) if value is not None } _LOGGER.debug("set_temperature start data=%s", kwargs) diff --git a/tests/components/climate/test_device_action.py b/tests/components/climate/test_device_action.py index 850f8b6c843..361aeaec867 100644 --- a/tests/components/climate/test_device_action.py +++ b/tests/components/climate/test_device_action.py @@ -136,7 +136,7 @@ async def test_get_actions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for action in ["set_hvac_mode"] + for action in ("set_hvac_mode",) ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id diff --git a/tests/components/climate/test_device_condition.py b/tests/components/climate/test_device_condition.py index 01513bcc506..0961bd3dc73 100644 --- a/tests/components/climate/test_device_condition.py +++ b/tests/components/climate/test_device_condition.py @@ -139,7 +139,7 @@ async def test_get_conditions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for condition in ["is_hvac_mode"] + for condition in ("is_hvac_mode",) ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id diff --git a/tests/components/climate/test_device_trigger.py b/tests/components/climate/test_device_trigger.py index 094c743f2b3..e8e5b577bf4 100644 --- a/tests/components/climate/test_device_trigger.py +++ b/tests/components/climate/test_device_trigger.py @@ -74,11 +74,11 @@ async def test_get_triggers( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for trigger in [ + for trigger in ( "hvac_mode_changed", "current_temperature_changed", "current_humidity_changed", - ] + ) ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id @@ -135,11 +135,11 @@ async def test_get_triggers_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for trigger in [ + for trigger in ( "hvac_mode_changed", "current_temperature_changed", "current_humidity_changed", - ] + ) ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id diff --git a/tests/components/cover/test_device_action.py b/tests/components/cover/test_device_action.py index d38f02d9c6e..db9e75bcaef 100644 --- a/tests/components/cover/test_device_action.py +++ b/tests/components/cover/test_device_action.py @@ -136,7 +136,7 @@ async def test_get_actions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for action in ["close"] + for action in ("close",) ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id diff --git a/tests/components/cover/test_device_condition.py b/tests/components/cover/test_device_condition.py index 9e5e5db1862..545bdd6587e 100644 --- a/tests/components/cover/test_device_condition.py +++ b/tests/components/cover/test_device_condition.py @@ -165,7 +165,7 @@ async def test_get_conditions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for condition in ["is_open", "is_closed", "is_opening", "is_closing"] + for condition in ("is_open", "is_closed", "is_opening", "is_closing") ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id diff --git a/tests/components/cover/test_device_trigger.py b/tests/components/cover/test_device_trigger.py index 1ad84e52c0c..419eea05f9f 100644 --- a/tests/components/cover/test_device_trigger.py +++ b/tests/components/cover/test_device_trigger.py @@ -166,7 +166,7 @@ async def test_get_triggers_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for trigger in ["opened", "closed", "opening", "closing"] + for trigger in ("opened", "closed", "opening", "closing") ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id diff --git a/tests/components/device_tracker/test_device_condition.py b/tests/components/device_tracker/test_device_condition.py index 3147f7ee2fd..6ea4ed7a372 100644 --- a/tests/components/device_tracker/test_device_condition.py +++ b/tests/components/device_tracker/test_device_condition.py @@ -54,7 +54,7 @@ async def test_get_conditions( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for condition in ["is_not_home", "is_home"] + for condition in ("is_not_home", "is_home") ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id @@ -102,7 +102,7 @@ async def test_get_conditions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for condition in ["is_not_home", "is_home"] + for condition in ("is_not_home", "is_home") ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id diff --git a/tests/components/device_tracker/test_device_trigger.py b/tests/components/device_tracker/test_device_trigger.py index 0a74c009ee3..8932eb15997 100644 --- a/tests/components/device_tracker/test_device_trigger.py +++ b/tests/components/device_tracker/test_device_trigger.py @@ -85,7 +85,7 @@ async def test_get_triggers( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for trigger in ["leaves", "enters"] + for trigger in ("leaves", "enters") ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id @@ -133,7 +133,7 @@ async def test_get_triggers_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for trigger in ["leaves", "enters"] + for trigger in ("leaves", "enters") ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id diff --git a/tests/components/fan/common.py b/tests/components/fan/common.py index fbc7c7bb1bb..74939342fac 100644 --- a/tests/components/fan/common.py +++ b/tests/components/fan/common.py @@ -38,11 +38,11 @@ async def async_turn_on( """Turn all or specified fan on.""" data = { key: value - for key, value in [ + for key, value in ( (ATTR_ENTITY_ID, entity_id), (ATTR_PERCENTAGE, percentage), (ATTR_PRESET_MODE, preset_mode), - ] + ) if value is not None } @@ -64,10 +64,10 @@ async def async_oscillate( """Set oscillation on all or specified fan.""" data = { key: value - for key, value in [ + for key, value in ( (ATTR_ENTITY_ID, entity_id), (ATTR_OSCILLATING, should_oscillate), - ] + ) if value is not None } @@ -81,7 +81,7 @@ async def async_set_preset_mode( """Set preset mode for all or specified fan.""" data = { key: value - for key, value in [(ATTR_ENTITY_ID, entity_id), (ATTR_PRESET_MODE, preset_mode)] + for key, value in ((ATTR_ENTITY_ID, entity_id), (ATTR_PRESET_MODE, preset_mode)) if value is not None } @@ -95,7 +95,7 @@ async def async_set_percentage( """Set percentage for all or specified fan.""" data = { key: value - for key, value in [(ATTR_ENTITY_ID, entity_id), (ATTR_PERCENTAGE, percentage)] + for key, value in ((ATTR_ENTITY_ID, entity_id), (ATTR_PERCENTAGE, percentage)) if value is not None } @@ -109,10 +109,10 @@ async def async_increase_speed( """Increase speed for all or specified fan.""" data = { key: value - for key, value in [ + for key, value in ( (ATTR_ENTITY_ID, entity_id), (ATTR_PERCENTAGE_STEP, percentage_step), - ] + ) if value is not None } @@ -126,10 +126,10 @@ async def async_decrease_speed( """Decrease speed for all or specified fan.""" data = { key: value - for key, value in [ + for key, value in ( (ATTR_ENTITY_ID, entity_id), (ATTR_PERCENTAGE_STEP, percentage_step), - ] + ) if value is not None } @@ -143,7 +143,7 @@ async def async_set_direction( """Set direction for all or specified fan.""" data = { key: value - for key, value in [(ATTR_ENTITY_ID, entity_id), (ATTR_DIRECTION, direction)] + for key, value in ((ATTR_ENTITY_ID, entity_id), (ATTR_DIRECTION, direction)) if value is not None } diff --git a/tests/components/fan/test_device_action.py b/tests/components/fan/test_device_action.py index 96e02ab5592..647e45374ac 100644 --- a/tests/components/fan/test_device_action.py +++ b/tests/components/fan/test_device_action.py @@ -48,7 +48,7 @@ async def test_get_actions( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for action in ["turn_on", "turn_off", "toggle"] + for action in ("turn_on", "turn_off", "toggle") ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id @@ -96,7 +96,7 @@ async def test_get_actions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for action in ["turn_on", "turn_off", "toggle"] + for action in ("turn_on", "turn_off", "toggle") ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id diff --git a/tests/components/fan/test_device_condition.py b/tests/components/fan/test_device_condition.py index d442d91c9dd..9f9bde1a680 100644 --- a/tests/components/fan/test_device_condition.py +++ b/tests/components/fan/test_device_condition.py @@ -54,7 +54,7 @@ async def test_get_conditions( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for condition in ["is_off", "is_on"] + for condition in ("is_off", "is_on") ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id @@ -102,7 +102,7 @@ async def test_get_conditions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for condition in ["is_off", "is_on"] + for condition in ("is_off", "is_on") ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id diff --git a/tests/components/fan/test_device_trigger.py b/tests/components/fan/test_device_trigger.py index 445193b27d4..38f39376592 100644 --- a/tests/components/fan/test_device_trigger.py +++ b/tests/components/fan/test_device_trigger.py @@ -59,7 +59,7 @@ async def test_get_triggers( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for trigger in ["turned_off", "turned_on", "changed_states"] + for trigger in ("turned_off", "turned_on", "changed_states") ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id @@ -107,7 +107,7 @@ async def test_get_triggers_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for trigger in ["turned_off", "turned_on", "changed_states"] + for trigger in ("turned_off", "turned_on", "changed_states") ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id diff --git a/tests/components/group/common.py b/tests/components/group/common.py index 395fc990930..86fe537a776 100644 --- a/tests/components/group/common.py +++ b/tests/components/group/common.py @@ -64,13 +64,13 @@ def async_set_group( """Create/Update a group.""" data = { key: value - for key, value in [ + for key, value in ( (ATTR_OBJECT_ID, object_id), (ATTR_NAME, name), (ATTR_ENTITIES, entity_ids), (ATTR_ICON, icon), (ATTR_ADD_ENTITIES, add), - ] + ) if value is not None } diff --git a/tests/components/group/test_init.py b/tests/components/group/test_init.py index e2e618002ac..7434de74f63 100644 --- a/tests/components/group/test_init.py +++ b/tests/components/group/test_init.py @@ -1236,7 +1236,7 @@ async def test_group_mixed_domains_on(hass: HomeAssistant) -> None: hass.states.async_set("binary_sensor.alexander_garage_side_door_open", "on") hass.states.async_set("cover.small_garage_door", "open") - for domain in ["lock", "binary_sensor", "cover"]: + for domain in ("lock", "binary_sensor", "cover"): assert await async_setup_component(hass, domain, {}) assert await async_setup_component( hass, @@ -1261,7 +1261,7 @@ async def test_group_mixed_domains_off(hass: HomeAssistant) -> None: hass.states.async_set("binary_sensor.alexander_garage_side_door_open", "off") hass.states.async_set("cover.small_garage_door", "closed") - for domain in ["lock", "binary_sensor", "cover"]: + for domain in ("lock", "binary_sensor", "cover"): assert await async_setup_component(hass, domain, {}) assert await async_setup_component( hass, diff --git a/tests/components/humidifier/test_device_condition.py b/tests/components/humidifier/test_device_condition.py index e9b84a1b515..4f4d21adcba 100644 --- a/tests/components/humidifier/test_device_condition.py +++ b/tests/components/humidifier/test_device_condition.py @@ -141,7 +141,7 @@ async def test_get_conditions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for condition in ["is_off", "is_on"] + for condition in ("is_off", "is_on") ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id diff --git a/tests/components/light/common.py b/tests/components/light/common.py index 26c4d18706d..fd9557b05b2 100644 --- a/tests/components/light/common.py +++ b/tests/components/light/common.py @@ -101,7 +101,7 @@ async def async_turn_on( """Turn all or specified light on.""" data = { key: value - for key, value in [ + for key, value in ( (ATTR_ENTITY_ID, entity_id), (ATTR_PROFILE, profile), (ATTR_TRANSITION, transition), @@ -118,7 +118,7 @@ async def async_turn_on( (ATTR_EFFECT, effect), (ATTR_COLOR_NAME, color_name), (ATTR_WHITE, white), - ] + ) if value is not None } @@ -135,11 +135,11 @@ async def async_turn_off(hass, entity_id=ENTITY_MATCH_ALL, transition=None, flas """Turn all or specified light off.""" data = { key: value - for key, value in [ + for key, value in ( (ATTR_ENTITY_ID, entity_id), (ATTR_TRANSITION, transition), (ATTR_FLASH, flash), - ] + ) if value is not None } @@ -202,7 +202,7 @@ async def async_toggle( """Turn all or specified light on.""" data = { key: value - for key, value in [ + for key, value in ( (ATTR_ENTITY_ID, entity_id), (ATTR_PROFILE, profile), (ATTR_TRANSITION, transition), @@ -216,7 +216,7 @@ async def async_toggle( (ATTR_FLASH, flash), (ATTR_EFFECT, effect), (ATTR_COLOR_NAME, color_name), - ] + ) if value is not None } diff --git a/tests/components/light/test_device_action.py b/tests/components/light/test_device_action.py index 1013942f96b..8848ce19621 100644 --- a/tests/components/light/test_device_action.py +++ b/tests/components/light/test_device_action.py @@ -66,14 +66,14 @@ async def test_get_actions( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for action in [ + for action in ( "brightness_decrease", "brightness_increase", "flash", "turn_off", "turn_on", "toggle", - ] + ) ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id @@ -123,7 +123,7 @@ async def test_get_actions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for action in ["turn_on", "turn_off", "toggle"] + for action in ("turn_on", "turn_off", "toggle") ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id diff --git a/tests/components/light/test_device_condition.py b/tests/components/light/test_device_condition.py index cef3ef788cb..11dea49ea60 100644 --- a/tests/components/light/test_device_condition.py +++ b/tests/components/light/test_device_condition.py @@ -62,7 +62,7 @@ async def test_get_conditions( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for condition in ["is_off", "is_on"] + for condition in ("is_off", "is_on") ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id @@ -110,7 +110,7 @@ async def test_get_conditions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for condition in ["is_off", "is_on"] + for condition in ("is_off", "is_on") ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id diff --git a/tests/components/light/test_device_trigger.py b/tests/components/light/test_device_trigger.py index b61b69fef25..ab3babd1b64 100644 --- a/tests/components/light/test_device_trigger.py +++ b/tests/components/light/test_device_trigger.py @@ -67,7 +67,7 @@ async def test_get_triggers( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for trigger in ["changed_states", "turned_off", "turned_on"] + for trigger in ("changed_states", "turned_off", "turned_on") ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id @@ -115,7 +115,7 @@ async def test_get_triggers_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for trigger in ["changed_states", "turned_off", "turned_on"] + for trigger in ("changed_states", "turned_off", "turned_on") ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id diff --git a/tests/components/lock/test_device_action.py b/tests/components/lock/test_device_action.py index 3b46117ccd2..e77e7edd005 100644 --- a/tests/components/lock/test_device_action.py +++ b/tests/components/lock/test_device_action.py @@ -129,7 +129,7 @@ async def test_get_actions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for action in ["lock", "unlock"] + for action in ("lock", "unlock") ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id diff --git a/tests/components/lock/test_device_condition.py b/tests/components/lock/test_device_condition.py index ce7ce773999..97afe9fb759 100644 --- a/tests/components/lock/test_device_condition.py +++ b/tests/components/lock/test_device_condition.py @@ -63,7 +63,7 @@ async def test_get_conditions( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for condition in [ + for condition in ( "is_locked", "is_unlocked", "is_unlocking", @@ -71,7 +71,7 @@ async def test_get_conditions( "is_jammed", "is_open", "is_opening", - ] + ) ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id @@ -119,7 +119,7 @@ async def test_get_conditions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for condition in [ + for condition in ( "is_locked", "is_unlocked", "is_unlocking", @@ -127,7 +127,7 @@ async def test_get_conditions_hidden_auxiliary( "is_jammed", "is_open", "is_opening", - ] + ) ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id diff --git a/tests/components/lock/test_device_trigger.py b/tests/components/lock/test_device_trigger.py index 800b2ea756e..3cbfbb1a04c 100644 --- a/tests/components/lock/test_device_trigger.py +++ b/tests/components/lock/test_device_trigger.py @@ -72,7 +72,7 @@ async def test_get_triggers( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for trigger in [ + for trigger in ( "locked", "unlocked", "unlocking", @@ -80,7 +80,7 @@ async def test_get_triggers( "jammed", "open", "opening", - ] + ) ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id @@ -129,7 +129,7 @@ async def test_get_triggers_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for trigger in [ + for trigger in ( "locked", "unlocked", "unlocking", @@ -137,7 +137,7 @@ async def test_get_triggers_hidden_auxiliary( "jammed", "open", "opening", - ] + ) ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id diff --git a/tests/components/remote/test_device_action.py b/tests/components/remote/test_device_action.py index e228810149c..a6e890937b5 100644 --- a/tests/components/remote/test_device_action.py +++ b/tests/components/remote/test_device_action.py @@ -53,7 +53,7 @@ async def test_get_actions( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for action in ["turn_off", "turn_on", "toggle"] + for action in ("turn_off", "turn_on", "toggle") ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id @@ -101,7 +101,7 @@ async def test_get_actions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for action in ["turn_off", "turn_on", "toggle"] + for action in ("turn_off", "turn_on", "toggle") ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id diff --git a/tests/components/remote/test_device_condition.py b/tests/components/remote/test_device_condition.py index e0c5f6d862b..d13a0480355 100644 --- a/tests/components/remote/test_device_condition.py +++ b/tests/components/remote/test_device_condition.py @@ -59,7 +59,7 @@ async def test_get_conditions( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for condition in ["is_off", "is_on"] + for condition in ("is_off", "is_on") ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id @@ -107,7 +107,7 @@ async def test_get_conditions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for condition in ["is_off", "is_on"] + for condition in ("is_off", "is_on") ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id diff --git a/tests/components/remote/test_device_trigger.py b/tests/components/remote/test_device_trigger.py index 7e8f91a91dc..8a1a0c318d7 100644 --- a/tests/components/remote/test_device_trigger.py +++ b/tests/components/remote/test_device_trigger.py @@ -59,7 +59,7 @@ async def test_get_triggers( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for trigger in ["changed_states", "turned_off", "turned_on"] + for trigger in ("changed_states", "turned_off", "turned_on") ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id @@ -107,7 +107,7 @@ async def test_get_triggers_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for trigger in ["changed_states", "turned_off", "turned_on"] + for trigger in ("changed_states", "turned_off", "turned_on") ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id diff --git a/tests/components/select/test_device_action.py b/tests/components/select/test_device_action.py index c83e2585d5b..0ffb860179d 100644 --- a/tests/components/select/test_device_action.py +++ b/tests/components/select/test_device_action.py @@ -47,13 +47,13 @@ async def test_get_actions( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for action in [ + for action in ( "select_first", "select_last", "select_next", "select_option", "select_previous", - ] + ) ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id @@ -101,13 +101,13 @@ async def test_get_actions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for action in [ + for action in ( "select_first", "select_last", "select_next", "select_option", "select_previous", - ] + ) ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id diff --git a/tests/components/select/test_device_condition.py b/tests/components/select/test_device_condition.py index 526ad678c19..e60df688658 100644 --- a/tests/components/select/test_device_condition.py +++ b/tests/components/select/test_device_condition.py @@ -105,7 +105,7 @@ async def test_get_conditions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for condition in ["selected_option"] + for condition in ("selected_option",) ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id diff --git a/tests/components/select/test_device_trigger.py b/tests/components/select/test_device_trigger.py index 8370a060bcd..c7a55c56202 100644 --- a/tests/components/select/test_device_trigger.py +++ b/tests/components/select/test_device_trigger.py @@ -105,7 +105,7 @@ async def test_get_triggers_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for trigger in ["current_option_changed"] + for trigger in ("current_option_changed",) ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id diff --git a/tests/components/sensor/test_device_condition.py b/tests/components/sensor/test_device_condition.py index dc81ec696f8..5f0646db8db 100644 --- a/tests/components/sensor/test_device_condition.py +++ b/tests/components/sensor/test_device_condition.py @@ -171,7 +171,7 @@ async def test_get_conditions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for condition in ["is_value"] + for condition in ("is_value",) ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id diff --git a/tests/components/sensor/test_device_trigger.py b/tests/components/sensor/test_device_trigger.py index 922a83709f7..71c844e428a 100644 --- a/tests/components/sensor/test_device_trigger.py +++ b/tests/components/sensor/test_device_trigger.py @@ -173,7 +173,7 @@ async def test_get_triggers_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for trigger in ["value"] + for trigger in ("value",) ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id diff --git a/tests/components/sensor/test_recorder.py b/tests/components/sensor/test_recorder.py index 3762b3f083a..0abe5e56e44 100644 --- a/tests/components/sensor/test_recorder.py +++ b/tests/components/sensor/test_recorder.py @@ -2413,7 +2413,7 @@ async def test_list_statistic_ids( "unit_class": unit_class, }, ] - for stat_type in ["mean", "sum", "dogs"]: + for stat_type in ("mean", "sum", "dogs"): statistic_ids = await async_list_statistic_ids(hass, statistic_type=stat_type) if statistic_type == stat_type: assert statistic_ids == [ @@ -3887,12 +3887,12 @@ async def test_compile_statistics_hourly_daily_monthly_summary( start = zero end = zero + timedelta(minutes=5) for i in range(24): - for entity_id in [ + for entity_id in ( "sensor.test1", "sensor.test2", "sensor.test3", "sensor.test4", - ]: + ): expected_average = ( expected_averages[entity_id][i] if entity_id in expected_averages @@ -3936,12 +3936,12 @@ async def test_compile_statistics_hourly_daily_monthly_summary( start = zero end = zero + timedelta(hours=1) for i in range(2): - for entity_id in [ + for entity_id in ( "sensor.test1", "sensor.test2", "sensor.test3", "sensor.test4", - ]: + ): expected_average = ( mean(expected_averages[entity_id][i * 12 : (i + 1) * 12]) if entity_id in expected_averages @@ -3993,12 +3993,12 @@ async def test_compile_statistics_hourly_daily_monthly_summary( start = dt_util.parse_datetime("2021-08-31T06:00:00+00:00") end = start + timedelta(days=1) for i in range(2): - for entity_id in [ + for entity_id in ( "sensor.test1", "sensor.test2", "sensor.test3", "sensor.test4", - ]: + ): expected_average = ( mean(expected_averages[entity_id][i * 12 : (i + 1) * 12]) if entity_id in expected_averages @@ -4050,12 +4050,12 @@ async def test_compile_statistics_hourly_daily_monthly_summary( start = dt_util.parse_datetime("2021-08-01T06:00:00+00:00") end = dt_util.parse_datetime("2021-09-01T06:00:00+00:00") for i in range(2): - for entity_id in [ + for entity_id in ( "sensor.test1", "sensor.test2", "sensor.test3", "sensor.test4", - ]: + ): expected_average = ( mean(expected_averages[entity_id][i * 12 : (i + 1) * 12]) if entity_id in expected_averages diff --git a/tests/components/switch/test_device_action.py b/tests/components/switch/test_device_action.py index ed3ff6f55ac..0b41ce7992d 100644 --- a/tests/components/switch/test_device_action.py +++ b/tests/components/switch/test_device_action.py @@ -54,7 +54,7 @@ async def test_get_actions( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for action in ["turn_off", "turn_on", "toggle"] + for action in ("turn_off", "turn_on", "toggle") ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id @@ -102,7 +102,7 @@ async def test_get_actions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for action in ["turn_off", "turn_on", "toggle"] + for action in ("turn_off", "turn_on", "toggle") ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id diff --git a/tests/components/switch/test_device_condition.py b/tests/components/switch/test_device_condition.py index 43a91b8628a..2ba2c6adb5c 100644 --- a/tests/components/switch/test_device_condition.py +++ b/tests/components/switch/test_device_condition.py @@ -59,7 +59,7 @@ async def test_get_conditions( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for condition in ["is_off", "is_on"] + for condition in ("is_off", "is_on") ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id @@ -107,7 +107,7 @@ async def test_get_conditions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for condition in ["is_off", "is_on"] + for condition in ("is_off", "is_on") ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id diff --git a/tests/components/switch/test_device_trigger.py b/tests/components/switch/test_device_trigger.py index 96479ba1900..092b7a964bb 100644 --- a/tests/components/switch/test_device_trigger.py +++ b/tests/components/switch/test_device_trigger.py @@ -59,7 +59,7 @@ async def test_get_triggers( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for trigger in ["changed_states", "turned_off", "turned_on"] + for trigger in ("changed_states", "turned_off", "turned_on") ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id @@ -107,7 +107,7 @@ async def test_get_triggers_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for trigger in ["changed_states", "turned_off", "turned_on"] + for trigger in ("changed_states", "turned_off", "turned_on") ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id diff --git a/tests/components/vacuum/test_device_action.py b/tests/components/vacuum/test_device_action.py index fec2ca1bf12..08459e05571 100644 --- a/tests/components/vacuum/test_device_action.py +++ b/tests/components/vacuum/test_device_action.py @@ -47,7 +47,7 @@ async def test_get_actions( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for action in ["clean", "dock"] + for action in ("clean", "dock") ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id @@ -95,7 +95,7 @@ async def test_get_actions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for action in ["clean", "dock"] + for action in ("clean", "dock") ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id diff --git a/tests/components/vacuum/test_device_condition.py b/tests/components/vacuum/test_device_condition.py index 1a5a5ed38e0..5cc222a1833 100644 --- a/tests/components/vacuum/test_device_condition.py +++ b/tests/components/vacuum/test_device_condition.py @@ -59,7 +59,7 @@ async def test_get_conditions( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for condition in ["is_cleaning", "is_docked"] + for condition in ("is_cleaning", "is_docked") ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id @@ -107,7 +107,7 @@ async def test_get_conditions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for condition in ["is_cleaning", "is_docked"] + for condition in ("is_cleaning", "is_docked") ] conditions = await async_get_device_automations( hass, DeviceAutomationType.CONDITION, device_entry.id diff --git a/tests/components/vacuum/test_device_trigger.py b/tests/components/vacuum/test_device_trigger.py index 648059e3c8f..56e351a6446 100644 --- a/tests/components/vacuum/test_device_trigger.py +++ b/tests/components/vacuum/test_device_trigger.py @@ -59,7 +59,7 @@ async def test_get_triggers( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for trigger in ["cleaning", "docked"] + for trigger in ("cleaning", "docked") ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id @@ -107,7 +107,7 @@ async def test_get_triggers_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for trigger in ["cleaning", "docked"] + for trigger in ("cleaning", "docked") ] triggers = await async_get_device_automations( hass, DeviceAutomationType.TRIGGER, device_entry.id diff --git a/tests/components/water_heater/common.py b/tests/components/water_heater/common.py index 9e47af4a19f..e0a8075f4cc 100644 --- a/tests/components/water_heater/common.py +++ b/tests/components/water_heater/common.py @@ -35,11 +35,11 @@ async def async_set_temperature( """Set new target temperature.""" kwargs = { key: value - for key, value in [ + for key, value in ( (ATTR_TEMPERATURE, temperature), (ATTR_ENTITY_ID, entity_id), (ATTR_OPERATION_MODE, operation_mode), - ] + ) if value is not None } _LOGGER.debug("set_temperature start data=%s", kwargs) diff --git a/tests/components/water_heater/test_device_action.py b/tests/components/water_heater/test_device_action.py index e08721d3e10..943aa3373a0 100644 --- a/tests/components/water_heater/test_device_action.py +++ b/tests/components/water_heater/test_device_action.py @@ -47,7 +47,7 @@ async def test_get_actions( "entity_id": entity_entry.id, "metadata": {"secondary": False}, } - for action in ["turn_on", "turn_off"] + for action in ("turn_on", "turn_off") ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id @@ -95,7 +95,7 @@ async def test_get_actions_hidden_auxiliary( "entity_id": entity_entry.id, "metadata": {"secondary": True}, } - for action in ["turn_on", "turn_off"] + for action in ("turn_on", "turn_off") ] actions = await async_get_device_automations( hass, DeviceAutomationType.ACTION, device_entry.id diff --git a/tests/helpers/test_config_validation.py b/tests/helpers/test_config_validation.py index f7c6a9bc99a..163a33db988 100644 --- a/tests/helpers/test_config_validation.py +++ b/tests/helpers/test_config_validation.py @@ -768,7 +768,7 @@ def test_date() -> None: """Test date validation.""" schema = vol.Schema(cv.date) - for value in ["Not a date", "23:42", "2016-11-23T18:59:08"]: + for value in ("Not a date", "23:42", "2016-11-23T18:59:08"): with pytest.raises(vol.Invalid): schema(value) @@ -780,7 +780,7 @@ def test_time() -> None: """Test date validation.""" schema = vol.Schema(cv.time) - for value in ["Not a time", "2016-11-23", "2016-11-23T18:59:08"]: + for value in ("Not a time", "2016-11-23", "2016-11-23T18:59:08"): with pytest.raises(vol.Invalid): schema(value) @@ -792,7 +792,7 @@ def test_time() -> None: def test_datetime() -> None: """Test date time validation.""" schema = vol.Schema(cv.datetime) - for value in [date.today(), "Wrong DateTime"]: + for value in (date.today(), "Wrong DateTime"): with pytest.raises(vol.MultipleInvalid): schema(value) @@ -1307,7 +1307,7 @@ def test_uuid4_hex(caplog: pytest.LogCaptureFixture) -> None: """Test uuid validation.""" schema = vol.Schema(cv.uuid4_hex) - for value in ["Not a hex string", "0", 0]: + for value in ("Not a hex string", "0", 0): with pytest.raises(vol.Invalid): schema(value) diff --git a/tests/helpers/test_entity_platform.py b/tests/helpers/test_entity_platform.py index 986c3e5493e..56ddcd9a6c9 100644 --- a/tests/helpers/test_entity_platform.py +++ b/tests/helpers/test_entity_platform.py @@ -615,7 +615,7 @@ async def test_async_remove_with_platform_update_finishes(hass: HomeAssistant) - # Add, remove, and make sure no updates # cause the entity to reappear after removal and # that we can add another entity with the same entity_id - for entity in [entity1, entity2]: + for entity in (entity1, entity2): update_called = asyncio.Event() update_done = asyncio.Event() await component.async_add_entities([entity]) diff --git a/tests/test_config.py b/tests/test_config.py index 27ef8059fd8..9a44333e20c 100644 --- a/tests/test_config.py +++ b/tests/test_config.py @@ -192,13 +192,13 @@ async def mock_non_adr_0007_integration_with_docs(hass: HomeAssistant) -> None: async def mock_adr_0007_integrations(hass: HomeAssistant) -> list[Integration]: """Mock ADR-0007 compliant integrations.""" integrations = [] - for domain in [ + for domain in ( "adr_0007_1", "adr_0007_2", "adr_0007_3", "adr_0007_4", "adr_0007_5", - ]: + ): adr_0007_config_schema = vol.Schema( { domain: vol.Schema( @@ -225,13 +225,13 @@ async def mock_adr_0007_integrations_with_docs( ) -> list[Integration]: """Mock ADR-0007 compliant integrations.""" integrations = [] - for domain in [ + for domain in ( "adr_0007_1", "adr_0007_2", "adr_0007_3", "adr_0007_4", "adr_0007_5", - ]: + ): adr_0007_config_schema = vol.Schema( { domain: vol.Schema( @@ -293,10 +293,10 @@ async def mock_custom_validator_integrations(hass: HomeAssistant) -> list[Integr Mock(async_validate_config=gen_async_validate_config(domain)), ) - for domain, exception in [ + for domain, exception in ( ("custom_validator_bad_1", HomeAssistantError("broken")), ("custom_validator_bad_2", ValueError("broken")), - ]: + ): integrations.append(mock_integration(hass, MockModule(domain))) mock_platform( hass, @@ -352,10 +352,10 @@ async def mock_custom_validator_integrations_with_docs( Mock(async_validate_config=gen_async_validate_config(domain)), ) - for domain, exception in [ + for domain, exception in ( ("custom_validator_bad_1", HomeAssistantError("broken")), ("custom_validator_bad_2", ValueError("broken")), - ]: + ): integrations.append( mock_integration( hass, diff --git a/tests/test_core.py b/tests/test_core.py index 7787a9a3769..541affc729b 100644 --- a/tests/test_core.py +++ b/tests/test_core.py @@ -2230,7 +2230,7 @@ async def test_async_run_job_starts_coro_eagerly(hass: HomeAssistant) -> None: def test_valid_entity_id() -> None: """Test valid entity ID.""" - for invalid in [ + for invalid in ( "_light.kitchen", ".kitchen", ".light.kitchen", @@ -2243,10 +2243,10 @@ def test_valid_entity_id() -> None: "Light.kitchen", "light.Kitchen", "lightkitchen", - ]: + ): assert not ha.valid_entity_id(invalid), invalid - for valid in [ + for valid in ( "1.a", "1light.kitchen", "a.1", @@ -2255,13 +2255,13 @@ def test_valid_entity_id() -> None: "light.1kitchen", "light.kitchen", "light.something_yoo", - ]: + ): assert ha.valid_entity_id(valid), valid def test_valid_domain() -> None: """Test valid domain.""" - for invalid in [ + for invalid in ( "_light", ".kitchen", ".light.kitchen", @@ -2272,16 +2272,16 @@ def test_valid_domain() -> None: "light.kitchen_yo_", "light.kitchen.", "Light", - ]: + ): assert not ha.valid_domain(invalid), invalid - for valid in [ + for valid in ( "1", "1light", "a", "input_boolean", "light", - ]: + ): assert ha.valid_domain(valid), valid