From fdea99927ba545364928bfa081c72c8a4337d44b Mon Sep 17 00:00:00 2001 From: Franck Nijhof Date: Sun, 30 Aug 2026 16:29:15 +0200 Subject: [PATCH] Split the Daikin user flow init from its data (#180808) --- tests/components/daikin/test_config_flow.py | 55 +++++++++++++++------ 1 file changed, 39 insertions(+), 16 deletions(-) diff --git a/tests/components/daikin/test_config_flow.py b/tests/components/daikin/test_config_flow.py index 5afe55a7b9d2..c1724e49faea 100644 --- a/tests/components/daikin/test_config_flow.py +++ b/tests/components/daikin/test_config_flow.py @@ -66,10 +66,9 @@ async def test_user(hass: HomeAssistant, mock_daikin) -> None: assert result["type"] is FlowResultType.FORM assert result["step_id"] == "user" - result = await hass.config_entries.flow.async_init( - DOMAIN, - context={"source": SOURCE_USER}, - data={CONF_HOST: HOST}, + result = await hass.config_entries.flow.async_configure( + result["flow_id"], + user_input={CONF_HOST: HOST}, ) assert result["type"] is FlowResultType.CREATE_ENTRY assert result["title"] == HOST @@ -81,9 +80,15 @@ async def test_abort_if_already_setup(hass: HomeAssistant, mock_daikin) -> None: """Test we abort if Daikin is already setup.""" MockConfigEntry(domain="daikin", unique_id=MAC).add_to_hass(hass) result = await hass.config_entries.flow.async_init( - DOMAIN, - context={"source": SOURCE_USER}, - data={CONF_HOST: HOST, KEY_MAC: MAC}, + DOMAIN, context={"source": SOURCE_USER} + ) + + assert result["type"] is FlowResultType.FORM + assert result["step_id"] == "user" + + result = await hass.config_entries.flow.async_configure( + result["flow_id"], + user_input={CONF_HOST: HOST}, ) assert result["type"] is FlowResultType.ABORT @@ -106,9 +111,15 @@ async def test_device_abort(hass: HomeAssistant, mock_daikin, s_effect, reason) mock_daikin.side_effect = s_effect result = await hass.config_entries.flow.async_init( - DOMAIN, - context={"source": SOURCE_USER}, - data={CONF_HOST: HOST, KEY_MAC: MAC}, + DOMAIN, context={"source": SOURCE_USER} + ) + + assert result["type"] is FlowResultType.FORM + assert result["step_id"] == "user" + + result = await hass.config_entries.flow.async_configure( + result["flow_id"], + user_input={CONF_HOST: HOST}, ) assert result["type"] is FlowResultType.FORM assert result["errors"] == {"base": reason} @@ -118,9 +129,15 @@ async def test_device_abort(hass: HomeAssistant, mock_daikin, s_effect, reason) async def test_api_password_abort(hass: HomeAssistant) -> None: """Test device abort.""" result = await hass.config_entries.flow.async_init( - DOMAIN, - context={"source": SOURCE_USER}, - data={CONF_HOST: HOST, CONF_API_KEY: "aa", CONF_PASSWORD: "aa"}, + DOMAIN, context={"source": SOURCE_USER} + ) + + assert result["type"] is FlowResultType.FORM + assert result["step_id"] == "user" + + result = await hass.config_entries.flow.async_configure( + result["flow_id"], + user_input={CONF_HOST: HOST, CONF_API_KEY: "aa", CONF_PASSWORD: "aa"}, ) assert result["type"] is FlowResultType.FORM assert result["errors"] == {"base": "api_password"} @@ -159,9 +176,15 @@ async def test_discovery_zeroconf( MockConfigEntry(domain="daikin", unique_id=unique_id).add_to_hass(hass) result = await hass.config_entries.flow.async_init( - DOMAIN, - context={"source": SOURCE_USER, "unique_id": unique_id}, - data={CONF_HOST: HOST}, + DOMAIN, context={"source": SOURCE_USER, "unique_id": unique_id} + ) + + assert result["type"] is FlowResultType.FORM + assert result["step_id"] == "user" + + result = await hass.config_entries.flow.async_configure( + result["flow_id"], + user_input={CONF_HOST: HOST}, ) assert result["type"] is FlowResultType.ABORT