From 764a0f29cc3f557d1365da08f6e6c24ffbd65b91 Mon Sep 17 00:00:00 2001
From: Erik Montnemery <erik@montnemery.com>
Date: Wed, 17 Apr 2024 15:04:37 +0200
Subject: [PATCH] Allow [##:##:##] type keypad address in homeworks (#115762)

Allow [##:##:##] type keypad address
---
 homeassistant/components/homeworks/config_flow.py |  2 +-
 tests/components/homeworks/test_config_flow.py    | 14 +++++++++-----
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/homeassistant/components/homeworks/config_flow.py b/homeassistant/components/homeworks/config_flow.py
index e54bbc61141..b9515c306d6 100644
--- a/homeassistant/components/homeworks/config_flow.py
+++ b/homeassistant/components/homeworks/config_flow.py
@@ -93,7 +93,7 @@ BUTTON_EDIT = {
 }
 
 
-validate_addr = cv.matches_regex(r"\[\d\d:\d\d:\d\d:\d\d\]")
+validate_addr = cv.matches_regex(r"\[(?:\d\d:)?\d\d:\d\d:\d\d\]")
 
 
 async def validate_add_controller(
diff --git a/tests/components/homeworks/test_config_flow.py b/tests/components/homeworks/test_config_flow.py
index 6a5ae68e6ab..d00b5a13150 100644
--- a/tests/components/homeworks/test_config_flow.py
+++ b/tests/components/homeworks/test_config_flow.py
@@ -544,8 +544,12 @@ async def test_options_add_remove_light_flow(
     )
 
 
+@pytest.mark.parametrize("keypad_address", ["[02:08:03:01]", "[02:08:03]"])
 async def test_options_add_remove_keypad_flow(
-    hass: HomeAssistant, mock_config_entry: MockConfigEntry, mock_homeworks: MagicMock
+    hass: HomeAssistant,
+    mock_config_entry: MockConfigEntry,
+    mock_homeworks: MagicMock,
+    keypad_address: str,
 ) -> None:
     """Test options flow to add and remove a keypad."""
     mock_config_entry.add_to_hass(hass)
@@ -566,7 +570,7 @@ async def test_options_add_remove_keypad_flow(
     result = await hass.config_entries.options.async_configure(
         result["flow_id"],
         user_input={
-            CONF_ADDR: "[02:08:03:01]",
+            CONF_ADDR: keypad_address,
             CONF_NAME: "Hall Keypad",
         },
     )
@@ -592,7 +596,7 @@ async def test_options_add_remove_keypad_flow(
                 ],
                 "name": "Foyer Keypad",
             },
-            {"addr": "[02:08:03:01]", "buttons": [], "name": "Hall Keypad"},
+            {"addr": keypad_address, "buttons": [], "name": "Hall Keypad"},
         ],
         "port": 1234,
     }
@@ -612,7 +616,7 @@ async def test_options_add_remove_keypad_flow(
     assert result["step_id"] == "remove_keypad"
     assert result["data_schema"].schema["index"].options == {
         "0": "Foyer Keypad ([02:08:02:01])",
-        "1": "Hall Keypad ([02:08:03:01])",
+        "1": f"Hall Keypad ({keypad_address})",
     }
 
     result = await hass.config_entries.options.async_configure(
@@ -625,7 +629,7 @@ async def test_options_add_remove_keypad_flow(
             {"addr": "[02:08:01:01]", "name": "Foyer Sconces", "rate": 1.0},
         ],
         "host": "192.168.0.1",
-        "keypads": [{"addr": "[02:08:03:01]", "buttons": [], "name": "Hall Keypad"}],
+        "keypads": [{"addr": keypad_address, "buttons": [], "name": "Hall Keypad"}],
         "port": 1234,
     }
     await hass.async_block_till_done()