jeans/core
jeans/core
1
0
Fork 0
mirror of https://github.com/home-assistant/core.git synced 2025-04-23 00:37:53 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove option to update settings using second config flow in Reolink (#131695)

Browse Source
This commit is contained in:
starkillerOG 2024-12-02 15:01:28 +01:00 committed by GitHub
parent 4b9d89a480
commit 2f644eb61c
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 1 additions and 46 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
homeassistant/components/reolink/config_flow.py
View File

@ -278,7 +278,7 @@ class ReolinkFlowHandler(ConfigFlow, domain=DOMAIN):
return self.async_update_reload_and_abort(
entry=self._get_reconfigure_entry(), data=user_input
)
self._abort_if_unique_id_configured(updates=user_input)
self._abort_if_unique_id_configured()
return self.async_create_entry(
title=str(host.api.nvr_name),

45
tests/components/reolink/test_config_flow.py
View File

@ -286,51 +286,6 @@ async def test_options_flow(hass: HomeAssistant, mock_setup_entry: MagicMock) ->
}
async def test_change_connection_settings(
hass: HomeAssistant, mock_setup_entry: MagicMock
) -> None:
"""Test changing connection settings by issuing a second user config flow."""
config_entry = MockConfigEntry(
domain=DOMAIN,
unique_id=format_mac(TEST_MAC),
data={
CONF_HOST: TEST_HOST,
CONF_USERNAME: TEST_USERNAME,
CONF_PASSWORD: TEST_PASSWORD,
CONF_PORT: TEST_PORT,
CONF_USE_HTTPS: TEST_USE_HTTPS,
},
options={
CONF_PROTOCOL: DEFAULT_PROTOCOL,
},
title=TEST_NVR_NAME,
)
config_entry.add_to_hass(hass)
result = await hass.config_entries.flow.async_init(
DOMAIN, context={"source": config_entries.SOURCE_USER}
)
assert result["type"] is FlowResultType.FORM
assert result["step_id"] == "user"
assert result["errors"] == {}
result = await hass.config_entries.flow.async_configure(
result["flow_id"],
{
CONF_HOST: TEST_HOST2,
CONF_USERNAME: TEST_USERNAME2,
CONF_PASSWORD: TEST_PASSWORD2,
},
)
assert result["type"] is FlowResultType.ABORT
assert result["reason"] == "already_configured"
assert config_entry.data[CONF_HOST] == TEST_HOST2
assert config_entry.data[CONF_USERNAME] == TEST_USERNAME2
assert config_entry.data[CONF_PASSWORD] == TEST_PASSWORD2
async def test_reauth(hass: HomeAssistant, mock_setup_entry: MagicMock) -> None:
"""Test a reauth flow."""
config_entry = MockConfigEntry(