Portainer expand reconfigure check (home-assistant#155544)

Author: erwindouna

homeassistant/components/portainer/config_flow.py

@@ -159,6 +159,8 @@ async def async_step_reconfigure(
                 _LOGGER.exception("Unexpected exception")
                 errors["base"] = "unknown"
             else:
+                await self.async_set_unique_id(user_input[CONF_API_TOKEN])
+                self._abort_if_unique_id_configured()
                 return self.async_update_reload_and_abort(
                     reconf_entry,
                     data_updates={

tests/components/portainer/conftest.py

@@ -73,6 +73,7 @@ def mock_config_entry() -> MockConfigEntry:
         domain=DOMAIN,
         title="Portainer test",
         data=MOCK_TEST_CONFIG,
+        unique_id=MOCK_TEST_CONFIG[CONF_API_TOKEN],
         entry_id="portainer_test_entry_123",
         version=2,
     )

tests/components/portainer/snapshots/test_diagnostics.ambr

@@ -21,7 +21,7 @@
       'subentries': list([
       ]),
       'title': 'Portainer test',
-      'unique_id': None,
+      'unique_id': 'test_api_token',
       'version': 2,
     }),
     'coordinator': dict({

tests/components/portainer/test_config_flow.py

@@ -255,6 +255,31 @@ async def test_full_flow_reconfigure(
     assert len(mock_setup_entry.mock_calls) == 1
 
 
+async def test_full_flow_reconfigure_unique_id(
+    hass: HomeAssistant,
+    mock_portainer_client: AsyncMock,
+    mock_setup_entry: MagicMock,
+    mock_config_entry: MockConfigEntry,
+) -> None:
+    """Test the full flow of the config flow, this time with a known unique ID."""
+    mock_config_entry.add_to_hass(hass)
+    result = await mock_config_entry.start_reconfigure_flow(hass)
+    assert result["type"] is FlowResultType.FORM
+    assert result["step_id"] == "reconfigure"
+
+    result = await hass.config_entries.flow.async_configure(
+        result["flow_id"],
+        user_input=MOCK_USER_SETUP,
+    )
+
+    assert result["type"] is FlowResultType.ABORT
+    assert result["reason"] == "already_configured"
+    assert mock_config_entry.data[CONF_API_TOKEN] == "test_api_token"
+    assert mock_config_entry.data[CONF_URL] == "https://127.0.0.1:9000/"
+    assert mock_config_entry.data[CONF_VERIFY_SSL] is True
+    assert len(mock_setup_entry.mock_calls) == 0
+
+
 @pytest.mark.parametrize(
     ("exception", "reason"),
     [