From 33fb27eb1ade7e8c7886069f88f9b33fea0b1eda Mon Sep 17 00:00:00 2001 From: Jan Bouwhuis Date: Mon, 23 Jan 2023 16:31:02 +0100 Subject: [PATCH] Rename fake kitchen_sink demo locks to more common name (#86452) --- homeassistant/components/kitchen_sink/lock.py | 8 ++-- tests/components/group/test_lock.py | 47 +++++++------------ 2 files changed, 22 insertions(+), 33 deletions(-) diff --git a/homeassistant/components/kitchen_sink/lock.py b/homeassistant/components/kitchen_sink/lock.py index 421b199abfe..343190acb63 100644 --- a/homeassistant/components/kitchen_sink/lock.py +++ b/homeassistant/components/kitchen_sink/lock.py @@ -22,24 +22,24 @@ async def async_setup_platform( [ DemoLock( "kitchen_sink_lock_001", - "Openable kitchen sink lock", + "Openable lock", STATE_LOCKED, LockEntityFeature.OPEN, ), DemoLock( "kitchen_sink_lock_002", - "Another kitchen sink openable lock", + "Another openable lock", STATE_UNLOCKED, LockEntityFeature.OPEN, ), DemoLock( "kitchen_sink_lock_003", - "Basic kitchen sink lock", + "Basic lock", STATE_LOCKED, ), DemoLock( "kitchen_sink_lock_004", - "Another kitchen sink lock", + "Another basic lock", STATE_UNLOCKED, ), ] diff --git a/tests/components/group/test_lock.py b/tests/components/group/test_lock.py index 3c8642ea38b..16dc3797daf 100644 --- a/tests/components/group/test_lock.py +++ b/tests/components/group/test_lock.py @@ -183,8 +183,8 @@ async def test_service_calls_openable(hass: HomeAssistant) -> None: { "platform": DOMAIN, "entities": [ - "lock.openable_kitchen_sink_lock", - "lock.another_kitchen_sink_openable_lock", + "lock.openable_lock", + "lock.another_openable_lock", ], }, ] @@ -194,11 +194,8 @@ async def test_service_calls_openable(hass: HomeAssistant) -> None: group_state = hass.states.get("lock.lock_group") assert group_state.state == STATE_UNLOCKED - assert hass.states.get("lock.openable_kitchen_sink_lock").state == STATE_LOCKED - assert ( - hass.states.get("lock.another_kitchen_sink_openable_lock").state - == STATE_UNLOCKED - ) + assert hass.states.get("lock.openable_lock").state == STATE_LOCKED + assert hass.states.get("lock.another_openable_lock").state == STATE_UNLOCKED await hass.services.async_call( LOCK_DOMAIN, @@ -206,11 +203,8 @@ async def test_service_calls_openable(hass: HomeAssistant) -> None: {ATTR_ENTITY_ID: "lock.lock_group"}, blocking=True, ) - assert hass.states.get("lock.openable_kitchen_sink_lock").state == STATE_UNLOCKED - assert ( - hass.states.get("lock.another_kitchen_sink_openable_lock").state - == STATE_UNLOCKED - ) + assert hass.states.get("lock.openable_lock").state == STATE_UNLOCKED + assert hass.states.get("lock.another_openable_lock").state == STATE_UNLOCKED await hass.services.async_call( LOCK_DOMAIN, @@ -218,10 +212,8 @@ async def test_service_calls_openable(hass: HomeAssistant) -> None: {ATTR_ENTITY_ID: "lock.lock_group"}, blocking=True, ) - assert hass.states.get("lock.openable_kitchen_sink_lock").state == STATE_LOCKED - assert ( - hass.states.get("lock.another_kitchen_sink_openable_lock").state == STATE_LOCKED - ) + assert hass.states.get("lock.openable_lock").state == STATE_LOCKED + assert hass.states.get("lock.another_openable_lock").state == STATE_LOCKED await hass.services.async_call( LOCK_DOMAIN, @@ -229,11 +221,8 @@ async def test_service_calls_openable(hass: HomeAssistant) -> None: {ATTR_ENTITY_ID: "lock.lock_group"}, blocking=True, ) - assert hass.states.get("lock.openable_kitchen_sink_lock").state == STATE_UNLOCKED - assert ( - hass.states.get("lock.another_kitchen_sink_openable_lock").state - == STATE_UNLOCKED - ) + assert hass.states.get("lock.openable_lock").state == STATE_UNLOCKED + assert hass.states.get("lock.another_openable_lock").state == STATE_UNLOCKED async def test_service_calls_basic(hass: HomeAssistant) -> None: @@ -247,8 +236,8 @@ async def test_service_calls_basic(hass: HomeAssistant) -> None: { "platform": DOMAIN, "entities": [ - "lock.basic_kitchen_sink_lock", - "lock.another_kitchen_sink_lock", + "lock.basic_lock", + "lock.another_basic_lock", ], }, ] @@ -258,8 +247,8 @@ async def test_service_calls_basic(hass: HomeAssistant) -> None: group_state = hass.states.get("lock.lock_group") assert group_state.state == STATE_UNLOCKED - assert hass.states.get("lock.basic_kitchen_sink_lock").state == STATE_LOCKED - assert hass.states.get("lock.another_kitchen_sink_lock").state == STATE_UNLOCKED + assert hass.states.get("lock.basic_lock").state == STATE_LOCKED + assert hass.states.get("lock.another_basic_lock").state == STATE_UNLOCKED await hass.services.async_call( LOCK_DOMAIN, @@ -267,8 +256,8 @@ async def test_service_calls_basic(hass: HomeAssistant) -> None: {ATTR_ENTITY_ID: "lock.lock_group"}, blocking=True, ) - assert hass.states.get("lock.basic_kitchen_sink_lock").state == STATE_LOCKED - assert hass.states.get("lock.another_kitchen_sink_lock").state == STATE_LOCKED + assert hass.states.get("lock.basic_lock").state == STATE_LOCKED + assert hass.states.get("lock.another_basic_lock").state == STATE_LOCKED await hass.services.async_call( LOCK_DOMAIN, @@ -276,8 +265,8 @@ async def test_service_calls_basic(hass: HomeAssistant) -> None: {ATTR_ENTITY_ID: "lock.lock_group"}, blocking=True, ) - assert hass.states.get("lock.basic_kitchen_sink_lock").state == STATE_UNLOCKED - assert hass.states.get("lock.another_kitchen_sink_lock").state == STATE_UNLOCKED + assert hass.states.get("lock.basic_lock").state == STATE_UNLOCKED + assert hass.states.get("lock.another_basic_lock").state == STATE_UNLOCKED with pytest.raises(HomeAssistantError): await hass.services.async_call(