Avoid redundant calls to async_write_ha_state in mqtt lawn_mower (#100795)

Avoid redundant calls to async_write_ha_state
This commit is contained in:
Jan Bouwhuis
2023-09-25 18:03:52 +02:00
committed by GitHub
parent 014fb61743
commit ce02cbefc9
2 changed files with 46 additions and 4 deletions
+38
View File
@@ -24,6 +24,7 @@ from homeassistant.const import (
from homeassistant.core import HomeAssistant, State
from .test_common import (
help_custom_config,
help_test_availability_when_connection_lost,
help_test_availability_without_topic,
help_test_custom_availability_payload,
@@ -47,6 +48,7 @@ from .test_common import (
help_test_setting_attribute_via_mqtt_json_message,
help_test_setting_attribute_with_template,
help_test_setting_blocked_attribute_via_mqtt_json_message,
help_test_skipped_async_ha_write_state,
help_test_unique_id,
help_test_unload_config_entry_with_platform,
help_test_update_with_json_attrs_bad_json,
@@ -886,3 +888,39 @@ async def test_persistent_state_after_reconfig(
# assert the state persistent
state = hass.states.get("lawn_mower.garden")
assert state.state == "docked"
@pytest.mark.parametrize(
"hass_config",
[
help_custom_config(
lawn_mower.DOMAIN,
DEFAULT_CONFIG,
(
{
"activity_state_topic": "activity-state-topic",
"availability_topic": "availability-topic",
"json_attributes_topic": "json-attributes-topic",
},
),
)
],
)
@pytest.mark.parametrize(
("topic", "payload1", "payload2"),
[
("activity-state-topic", "mowing", "paused"),
("availability-topic", "online", "offline"),
("json-attributes-topic", '{"attr1": "val1"}', '{"attr1": "val2"}'),
],
)
async def test_skipped_async_ha_write_state(
hass: HomeAssistant,
mqtt_mock_entry: MqttMockHAClientGenerator,
topic: str,
payload1: str,
payload2: str,
) -> None:
"""Test a write state command is only called when there is change."""
await mqtt_mock_entry()
await help_test_skipped_async_ha_write_state(hass, topic, payload1, payload2)