Allow changing the Insteon USB device (#86290)

* Reconfig PLM device

* Test config change

homeassistant/components/insteon/config_flow.py

@@ -49,6 +49,7 @@ STEP_PLM = "plm"
 STEP_HUB_V1 = "hubv1"
 STEP_HUB_V2 = "hubv2"
 STEP_CHANGE_HUB_CONFIG = "change_hub_config"
+STEP_CHANGE_PLM_CONFIG = "change_plm_config"
 STEP_ADD_X10 = "add_x10"
 STEP_ADD_OVERRIDE = "add_override"
 STEP_REMOVE_OVERRIDE = "remove_override"
@@ -213,7 +214,7 @@ class InsteonFlowHandler(config_entries.ConfigFlow, domain=DOMAIN):
         await self.async_set_unique_id(config_entries.DEFAULT_DISCOVERY_UNIQUE_ID)
         return await self.async_step_confirm_usb()
 
-    async def async_step_confirm_usb(self, user_input=None):
+    async def async_step_confirm_usb(self, user_input=None) -> FlowResult:
         """Confirm a USB discovery."""
         if user_input is not None:
             return await self.async_step_plm({CONF_DEVICE: self._device_path})
@@ -240,17 +241,19 @@ class InsteonOptionsFlowHandler(config_entries.OptionsFlow):
         """Init the InsteonOptionsFlowHandler class."""
         self.config_entry = config_entry
 
-    async def async_step_init(self, user_input=None):
+    async def async_step_init(self, user_input=None) -> FlowResult:
         """Init the options config flow."""
         errors = {}
         if user_input is not None:
             change_hub_config = user_input.get(STEP_CHANGE_HUB_CONFIG, False)
+            change_plm_config = user_input.get(STEP_CHANGE_PLM_CONFIG, False)
             device_override = user_input.get(STEP_ADD_OVERRIDE, False)
             x10_device = user_input.get(STEP_ADD_X10, False)
             remove_override = user_input.get(STEP_REMOVE_OVERRIDE, False)
             remove_x10 = user_input.get(STEP_REMOVE_X10, False)
             if _only_one_selected(
                 change_hub_config,
+                change_plm_config,
                 device_override,
                 x10_device,
                 remove_override,
@@ -258,6 +261,8 @@ class InsteonOptionsFlowHandler(config_entries.OptionsFlow):
             ):
                 if change_hub_config:
                     return await self.async_step_change_hub_config()
+                if change_plm_config:
+                    return await self.async_step_change_plm_config()
                 if device_override:
                     return await self.async_step_add_override()
                 if x10_device:
@@ -274,6 +279,8 @@ class InsteonOptionsFlowHandler(config_entries.OptionsFlow):
         }
         if self.config_entry.data.get(CONF_HOST):
             data_schema[vol.Optional(STEP_CHANGE_HUB_CONFIG)] = bool
+        else:
+            data_schema[vol.Optional(STEP_CHANGE_PLM_CONFIG)] = bool
 
         options = {**self.config_entry.options}
         if options.get(CONF_OVERRIDE):
@@ -285,7 +292,7 @@ class InsteonOptionsFlowHandler(config_entries.OptionsFlow):
             step_id="init", data_schema=vol.Schema(data_schema), errors=errors
         )
 
-    async def async_step_change_hub_config(self, user_input=None):
+    async def async_step_change_hub_config(self, user_input=None) -> FlowResult:
         """Change the Hub configuration."""
         if user_input is not None:
             data = {
@@ -306,7 +313,24 @@ class InsteonOptionsFlowHandler(config_entries.OptionsFlow):
             step_id=STEP_CHANGE_HUB_CONFIG, data_schema=data_schema
         )
 
-    async def async_step_add_override(self, user_input=None):
+    async def async_step_change_plm_config(self, user_input=None) -> FlowResult:
+        """Change the PLM configuration."""
+        if user_input is not None:
+            data = {
+                **self.config_entry.data,
+                CONF_DEVICE: user_input[CONF_DEVICE],
+            }
+            self.hass.config_entries.async_update_entry(self.config_entry, data=data)
+            return self.async_create_entry(
+                title="",
+                data={**self.config_entry.options},
+            )
+        data_schema = build_plm_schema(**self.config_entry.data)
+        return self.async_show_form(
+            step_id=STEP_CHANGE_PLM_CONFIG, data_schema=data_schema
+        )
+
+    async def async_step_add_override(self, user_input=None) -> FlowResult:
         """Add a device override."""
         errors = {}
         if user_input is not None:
@@ -322,22 +346,22 @@ class InsteonOptionsFlowHandler(config_entries.OptionsFlow):
             step_id=STEP_ADD_OVERRIDE, data_schema=data_schema, errors=errors
         )
 
-    async def async_step_add_x10(self, user_input=None):
+    async def async_step_add_x10(self, user_input=None) -> FlowResult:
         """Add an X10 device."""
-        errors = {}
+        errors: dict[str, str] = {}
         if user_input is not None:
             options = add_x10_device({**self.config_entry.options}, user_input)
             async_dispatcher_send(self.hass, SIGNAL_ADD_X10_DEVICE, user_input)
             return self.async_create_entry(title="", data=options)
-        schema_defaults = user_input if user_input is not None else {}
+        schema_defaults: dict[str, str] = user_input if user_input is not None else {}
         data_schema = build_x10_schema(**schema_defaults)
         return self.async_show_form(
             step_id=STEP_ADD_X10, data_schema=data_schema, errors=errors
         )
 
-    async def async_step_remove_override(self, user_input=None):
+    async def async_step_remove_override(self, user_input=None) -> FlowResult:
         """Remove a device override."""
-        errors = {}
+        errors: dict[str, str] = {}
         options = self.config_entry.options
         if user_input is not None:
             options = _remove_override(user_input[CONF_ADDRESS], options)
@@ -353,9 +377,9 @@ class InsteonOptionsFlowHandler(config_entries.OptionsFlow):
             step_id=STEP_REMOVE_OVERRIDE, data_schema=data_schema, errors=errors
         )
 
-    async def async_step_remove_x10(self, user_input=None):
+    async def async_step_remove_x10(self, user_input=None) -> FlowResult:
         """Remove an X10 device."""
-        errors = {}
+        errors: dict[str, str] = {}
         options = self.config_entry.options
         if user_input is not None:
             options, housecode, unitcode = _remove_x10(user_input[CONF_DEVICE], options)

homeassistant/components/insteon/strings.json

@@ -52,6 +52,7 @@
       "init": {
         "data": {
           "change_hub_config": "Change the Hub configuration.",
+          "change_plm_config": "Change the PLM configuration.",
           "add_override": "Add a device override.",
           "add_x10": "Add an X10 device.",
           "remove_override": "Remove a device override.",
@@ -67,6 +68,12 @@
           "password": "[%key:common::config_flow::data::password%]"
         }
       },
+      "change_plm_config": {
+        "description": "Change the Insteon PLM connection information. You must restart Home Assistant after making this change. This does not change the configuration of the PLM itself.",
+        "data": {
+          "device": "[%key:common::config_flow::data::usb_path%]"
+        }
+      },
       "add_override": {
         "description": "Add a device override.",
         "data": {

homeassistant/components/insteon/translations/en.json

@@ -80,11 +80,18 @@
                 },
                 "description": "Change the Insteon Hub connection information. You must restart Home Assistant after making this change. This does not change the configuration of the Hub itself. To change the configuration in the Hub use the Hub app."
             },
+            "change_plm_config": {
+                "data": {
+                    "device": "USB Device Path"
+                },
+                "description": "Change the Insteon PLM connection information. You must restart Home Assistant after making this change. This does not change the configuration of the PLM itself."
+            },
             "init": {
                 "data": {
                     "add_override": "Add a device override.",
                     "add_x10": "Add an X10 device.",
                     "change_hub_config": "Change the Hub configuration.",
+                    "change_plm_config": "Change the PLM configuration.",
                     "remove_override": "Remove a device override.",
                     "remove_x10": "Remove an X10 device."
                 }

tests/components/insteon/test_config_flow.py

@@ -12,6 +12,7 @@ from homeassistant.components.insteon.config_flow import (
     STEP_ADD_OVERRIDE,
     STEP_ADD_X10,
     STEP_CHANGE_HUB_CONFIG,
+    STEP_CHANGE_PLM_CONFIG,
     STEP_HUB_V2,
     STEP_REMOVE_OVERRIDE,
     STEP_REMOVE_X10,
@@ -334,6 +335,28 @@ async def test_options_change_hub_config(hass: HomeAssistant):
     assert config_entry.data == {**user_input, CONF_HUB_VERSION: 2}
 
 
+async def test_options_change_plm_config(hass: HomeAssistant):
+    """Test changing PLM config."""
+    config_entry = MockConfigEntry(
+        domain=DOMAIN,
+        entry_id="abcde12345",
+        data=MOCK_USER_INPUT_PLM,
+        options={},
+    )
+
+    config_entry.add_to_hass(hass)
+    result = await _options_init_form(
+        hass, config_entry.entry_id, STEP_CHANGE_PLM_CONFIG
+    )
+
+    user_input = {CONF_DEVICE: "/dev/some_other_device"}
+    result, _ = await _options_form(hass, result["flow_id"], user_input)
+
+    assert result["type"] == data_entry_flow.FlowResultType.CREATE_ENTRY
+    assert config_entry.options == {}
+    assert config_entry.data == user_input
+
+
 async def test_options_add_device_override(hass: HomeAssistant):
     """Test adding a device override."""
     config_entry = MockConfigEntry(