Enable Ruff rule PT007 (#113764)
Co-authored-by: Franck Nijhof <frenck@frenck.nl>
This commit is contained in:
parent
089a3ab6d7
commit
00ec7f11f0
204 changed files with 908 additions and 921 deletions
|
@ -933,10 +933,10 @@ async def test_entity_category_property(hass: HomeAssistant) -> None:
|
|||
|
||||
@pytest.mark.parametrize(
|
||||
("value", "expected"),
|
||||
(
|
||||
[
|
||||
("config", entity.EntityCategory.CONFIG),
|
||||
("diagnostic", entity.EntityCategory.DIAGNOSTIC),
|
||||
),
|
||||
],
|
||||
)
|
||||
def test_entity_category_schema(value, expected) -> None:
|
||||
"""Test entity category schema."""
|
||||
|
@ -946,7 +946,7 @@ def test_entity_category_schema(value, expected) -> None:
|
|||
assert isinstance(result, entity.EntityCategory)
|
||||
|
||||
|
||||
@pytest.mark.parametrize("value", (None, "non_existing"))
|
||||
@pytest.mark.parametrize("value", [None, "non_existing"])
|
||||
def test_entity_category_schema_error(value) -> None:
|
||||
"""Test entity category schema."""
|
||||
schema = vol.Schema(entity.ENTITY_CATEGORIES_SCHEMA)
|
||||
|
@ -1007,14 +1007,14 @@ async def _test_friendly_name(
|
|||
"device_name",
|
||||
"expected_friendly_name",
|
||||
),
|
||||
(
|
||||
[
|
||||
(False, "Entity Blu", "Device Bla", "Entity Blu"),
|
||||
(False, None, "Device Bla", None),
|
||||
(True, "Entity Blu", "Device Bla", "Device Bla Entity Blu"),
|
||||
(True, None, "Device Bla", "Device Bla"),
|
||||
(True, "Entity Blu", UNDEFINED, "Entity Blu"),
|
||||
(True, "Entity Blu", None, "Mock Title Entity Blu"),
|
||||
),
|
||||
],
|
||||
)
|
||||
async def test_friendly_name_attr(
|
||||
hass: HomeAssistant,
|
||||
|
@ -1044,14 +1044,14 @@ async def test_friendly_name_attr(
|
|||
|
||||
@pytest.mark.parametrize(
|
||||
("has_entity_name", "entity_name", "expected_friendly_name"),
|
||||
(
|
||||
[
|
||||
(False, "Entity Blu", "Entity Blu"),
|
||||
(False, None, None),
|
||||
(False, UNDEFINED, None),
|
||||
(True, "Entity Blu", "Device Bla Entity Blu"),
|
||||
(True, None, "Device Bla"),
|
||||
(True, UNDEFINED, "Device Bla None"),
|
||||
),
|
||||
],
|
||||
)
|
||||
async def test_friendly_name_description(
|
||||
hass: HomeAssistant,
|
||||
|
@ -1081,14 +1081,14 @@ async def test_friendly_name_description(
|
|||
|
||||
@pytest.mark.parametrize(
|
||||
("has_entity_name", "entity_name", "expected_friendly_name"),
|
||||
(
|
||||
[
|
||||
(False, "Entity Blu", "Entity Blu"),
|
||||
(False, None, None),
|
||||
(False, UNDEFINED, None),
|
||||
(True, "Entity Blu", "Device Bla Entity Blu"),
|
||||
(True, None, "Device Bla"),
|
||||
(True, UNDEFINED, "Device Bla English cls"),
|
||||
),
|
||||
],
|
||||
)
|
||||
async def test_friendly_name_description_device_class_name(
|
||||
hass: HomeAssistant,
|
||||
|
@ -1150,7 +1150,7 @@ async def test_friendly_name_description_device_class_name(
|
|||
"placeholders",
|
||||
"expected_friendly_name",
|
||||
),
|
||||
(
|
||||
[
|
||||
(False, None, None, None, "Entity Blu"),
|
||||
(True, None, None, None, "Device Bla Entity Blu"),
|
||||
(
|
||||
|
@ -1186,7 +1186,7 @@ async def test_friendly_name_description_device_class_name(
|
|||
{"placeholder": "special"},
|
||||
"Device Bla English ent special",
|
||||
),
|
||||
),
|
||||
],
|
||||
)
|
||||
async def test_entity_name_translation_placeholders(
|
||||
hass: HomeAssistant,
|
||||
|
@ -1239,7 +1239,7 @@ async def test_entity_name_translation_placeholders(
|
|||
"release_channel",
|
||||
"expected_error",
|
||||
),
|
||||
(
|
||||
[
|
||||
(
|
||||
"test_entity",
|
||||
{
|
||||
|
@ -1279,7 +1279,7 @@ async def test_entity_name_translation_placeholders(
|
|||
"not match the name '{placeholder} English ent'"
|
||||
),
|
||||
),
|
||||
),
|
||||
],
|
||||
)
|
||||
async def test_entity_name_translation_placeholder_errors(
|
||||
hass: HomeAssistant,
|
||||
|
@ -1341,14 +1341,14 @@ async def test_entity_name_translation_placeholder_errors(
|
|||
|
||||
@pytest.mark.parametrize(
|
||||
("has_entity_name", "entity_name", "expected_friendly_name"),
|
||||
(
|
||||
[
|
||||
(False, "Entity Blu", "Entity Blu"),
|
||||
(False, None, None),
|
||||
(False, UNDEFINED, None),
|
||||
(True, "Entity Blu", "Device Bla Entity Blu"),
|
||||
(True, None, "Device Bla"),
|
||||
(True, UNDEFINED, "Device Bla None"),
|
||||
),
|
||||
],
|
||||
)
|
||||
async def test_friendly_name_property(
|
||||
hass: HomeAssistant,
|
||||
|
@ -1377,7 +1377,7 @@ async def test_friendly_name_property(
|
|||
|
||||
@pytest.mark.parametrize(
|
||||
("has_entity_name", "entity_name", "expected_friendly_name"),
|
||||
(
|
||||
[
|
||||
(False, "Entity Blu", "Entity Blu"),
|
||||
(False, None, None),
|
||||
(False, UNDEFINED, None),
|
||||
|
@ -1385,7 +1385,7 @@ async def test_friendly_name_property(
|
|||
(True, None, "Device Bla"),
|
||||
# Won't use the device class name because the entity overrides the name property
|
||||
(True, UNDEFINED, "Device Bla None"),
|
||||
),
|
||||
],
|
||||
)
|
||||
async def test_friendly_name_property_device_class_name(
|
||||
hass: HomeAssistant,
|
||||
|
@ -1438,10 +1438,10 @@ async def test_friendly_name_property_device_class_name(
|
|||
|
||||
@pytest.mark.parametrize(
|
||||
("has_entity_name", "expected_friendly_name"),
|
||||
(
|
||||
[
|
||||
(False, None),
|
||||
(True, "Device Bla English cls"),
|
||||
),
|
||||
],
|
||||
)
|
||||
async def test_friendly_name_device_class_name(
|
||||
hass: HomeAssistant,
|
||||
|
@ -1497,7 +1497,7 @@ async def test_friendly_name_device_class_name(
|
|||
"expected_friendly_name2",
|
||||
"expected_friendly_name3",
|
||||
),
|
||||
(
|
||||
[
|
||||
(
|
||||
"Entity Blu",
|
||||
"Device Bla Entity Blu",
|
||||
|
@ -1510,7 +1510,7 @@ async def test_friendly_name_device_class_name(
|
|||
"Device Bla2",
|
||||
"New Device",
|
||||
),
|
||||
),
|
||||
],
|
||||
)
|
||||
async def test_friendly_name_updated(
|
||||
hass: HomeAssistant,
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue