diff --git a/tests/components/automation/test_init.py b/tests/components/automation/test_init.py index 51d7f0a0852..5ec77a43e8c 100644 --- a/tests/components/automation/test_init.py +++ b/tests/components/automation/test_init.py @@ -1380,7 +1380,7 @@ async def test_automation_not_trigger_on_bootstrap(hass): async def test_automation_bad_config_validation( hass: HomeAssistant, caplog, broken_config, problem, details ): - """Test bad trigger configuration which can be detected during validation.""" + """Test bad automation configuration which can be detected during validation.""" assert await async_setup_component( hass, automation.DOMAIN, diff --git a/tests/components/config/test_script.py b/tests/components/config/test_script.py index b8e980c29b9..bd79c000027 100644 --- a/tests/components/config/test_script.py +++ b/tests/components/config/test_script.py @@ -1,11 +1,13 @@ """Tests for config/script.""" from http import HTTPStatus +import json from unittest.mock import patch import pytest from homeassistant.bootstrap import async_setup_component from homeassistant.components import config +from homeassistant.core import HomeAssistant from homeassistant.helpers import entity_registry as er from tests.components.blueprint.conftest import stub_blueprint_populate # noqa: F401 @@ -17,6 +19,89 @@ async def setup_script(hass, script_config, stub_blueprint_populate): # noqa: F assert await async_setup_component(hass, "script", {"script": script_config}) +@pytest.mark.parametrize("script_config", ({},)) +async def test_get_script_config(hass: HomeAssistant, hass_client, hass_config_store): + """Test getting script config.""" + with patch.object(config, "SECTIONS", ["script"]): + await async_setup_component(hass, "config", {}) + + client = await hass_client() + + hass_config_store["scripts.yaml"] = { + "sun": {"alias": "Sun"}, + "moon": {"alias": "Moon"}, + } + + resp = await client.get("/api/config/script/config/moon") + + assert resp.status == HTTPStatus.OK + result = await resp.json() + + assert result == {"alias": "Moon"} + + +@pytest.mark.parametrize("script_config", ({},)) +async def test_update_script_config( + hass: HomeAssistant, hass_client, hass_config_store +): + """Test updating script config.""" + with patch.object(config, "SECTIONS", ["script"]): + await async_setup_component(hass, "config", {}) + + assert sorted(hass.states.async_entity_ids("script")) == [] + + client = await hass_client() + + orig_data = {"sun": {"alias": "Sun"}, "moon": {"alias": "Moon"}} + hass_config_store["scripts.yaml"] = orig_data + + resp = await client.post( + "/api/config/script/config/moon", + data=json.dumps({"alias": "Moon updated", "sequence": []}), + ) + await hass.async_block_till_done() + assert sorted(hass.states.async_entity_ids("script")) == ["script.moon"] + + assert resp.status == HTTPStatus.OK + result = await resp.json() + assert result == {"result": "ok"} + + new_data = hass_config_store["scripts.yaml"] + assert list(new_data["moon"]) == ["alias", "sequence"] + assert new_data["moon"] == {"alias": "Moon updated", "sequence": []} + + +@pytest.mark.parametrize("script_config", ({},)) +async def test_update_remove_key_script_config( + hass: HomeAssistant, hass_client, hass_config_store +): + """Test updating script config while removing a key.""" + with patch.object(config, "SECTIONS", ["script"]): + await async_setup_component(hass, "config", {}) + + assert sorted(hass.states.async_entity_ids("script")) == [] + + client = await hass_client() + + orig_data = {"sun": {"key": "value"}, "moon": {"key": "value"}} + hass_config_store["scripts.yaml"] = orig_data + + resp = await client.post( + "/api/config/script/config/moon", + data=json.dumps({"sequence": []}), + ) + await hass.async_block_till_done() + assert sorted(hass.states.async_entity_ids("script")) == ["script.moon"] + + assert resp.status == HTTPStatus.OK + result = await resp.json() + assert result == {"result": "ok"} + + new_data = hass_config_store["scripts.yaml"] + assert list(new_data["moon"]) == ["sequence"] + assert new_data["moon"] == {"sequence": []} + + @pytest.mark.parametrize( "script_config", ( @@ -26,7 +111,7 @@ async def setup_script(hass, script_config, stub_blueprint_populate): # noqa: F }, ), ) -async def test_delete_script(hass, hass_client, hass_config_store): +async def test_delete_script(hass: HomeAssistant, hass_client, hass_config_store): """Test deleting a script.""" with patch.object(config, "SECTIONS", ["script"]): await async_setup_component(hass, "config", {})