diff --git a/homeassistant/components/cloud/http_api.py b/homeassistant/components/cloud/http_api.py index 7937337f2d3..8d6c4e65e3c 100644 --- a/homeassistant/components/cloud/http_api.py +++ b/homeassistant/components/cloud/http_api.py @@ -201,12 +201,13 @@ class CloudLoginView(HomeAssistantView): # Make sure the pipeline store is loaded, needed because assist_pipeline # is an after dependency of cloud await assist_pipeline.async_setup_pipeline_store(hass) - if (cloud_pipeline_id := cloud_assist_pipeline(hass)) is None: + new_cloud_pipeline_id: str | None = None + if (cloud_assist_pipeline(hass)) is None: if cloud_pipeline := await assist_pipeline.async_create_default_pipeline( hass, DOMAIN, DOMAIN ): - cloud_pipeline_id = cloud_pipeline.id - return self.json({"success": True, "cloud_pipeline": cloud_pipeline_id}) + new_cloud_pipeline_id = cloud_pipeline.id + return self.json({"success": True, "cloud_pipeline": new_cloud_pipeline_id}) class CloudLogoutView(HomeAssistantView): diff --git a/tests/components/cloud/test_http_api.py b/tests/components/cloud/test_http_api.py index b13d0cd9cbf..351caff883f 100644 --- a/tests/components/cloud/test_http_api.py +++ b/tests/components/cloud/test_http_api.py @@ -129,7 +129,7 @@ async def test_login_view(hass: HomeAssistant, cloud_client) -> None: assert req.status == HTTPStatus.OK result = await req.json() - assert result == {"success": True, "cloud_pipeline": "12345"} + assert result == {"success": True, "cloud_pipeline": None} create_pipeline_mock.assert_not_awaited()