From 8de26e508554f79ee1d5b6439d4fbf2600557dd5 Mon Sep 17 00:00:00 2001 From: "J. Nick Koston" Date: Sun, 3 Apr 2022 07:42:29 -0600 Subject: [PATCH] Exclude static humidifier attributes from being recorded in the database (#69193) --- .../components/humidifier/recorder.py | 16 ++++++ tests/components/humidifier/test_recorder.py | 49 +++++++++++++++++++ 2 files changed, 65 insertions(+) create mode 100644 homeassistant/components/humidifier/recorder.py create mode 100644 tests/components/humidifier/test_recorder.py diff --git a/homeassistant/components/humidifier/recorder.py b/homeassistant/components/humidifier/recorder.py new file mode 100644 index 00000000000..53df96605d6 --- /dev/null +++ b/homeassistant/components/humidifier/recorder.py @@ -0,0 +1,16 @@ +"""Integration platform for recorder.""" +from __future__ import annotations + +from homeassistant.core import HomeAssistant, callback + +from . import ATTR_AVAILABLE_MODES, ATTR_MAX_HUMIDITY, ATTR_MIN_HUMIDITY + + +@callback +def exclude_attributes(hass: HomeAssistant) -> set[str]: + """Exclude static attributes from being recorded in the database.""" + return { + ATTR_MIN_HUMIDITY, + ATTR_MAX_HUMIDITY, + ATTR_AVAILABLE_MODES, + } diff --git a/tests/components/humidifier/test_recorder.py b/tests/components/humidifier/test_recorder.py new file mode 100644 index 00000000000..c7505574b98 --- /dev/null +++ b/tests/components/humidifier/test_recorder.py @@ -0,0 +1,49 @@ +"""The tests for humidifier recorder.""" +from __future__ import annotations + +from datetime import timedelta + +from homeassistant.components import humidifier +from homeassistant.components.humidifier import ( + ATTR_AVAILABLE_MODES, + ATTR_MAX_HUMIDITY, + ATTR_MIN_HUMIDITY, +) +from homeassistant.components.recorder.models import StateAttributes, States +from homeassistant.components.recorder.util import session_scope +from homeassistant.const import ATTR_FRIENDLY_NAME +from homeassistant.core import State +from homeassistant.setup import async_setup_component +from homeassistant.util import dt as dt_util + +from tests.common import async_fire_time_changed, async_init_recorder_component +from tests.components.recorder.common import async_wait_recording_done_without_instance + + +async def test_exclude_attributes(hass): + """Test humidifier registered attributes to be excluded.""" + await async_init_recorder_component(hass) + await async_setup_component( + hass, humidifier.DOMAIN, {humidifier.DOMAIN: {"platform": "demo"}} + ) + await hass.async_block_till_done() + async_fire_time_changed(hass, dt_util.utcnow() + timedelta(minutes=5)) + await hass.async_block_till_done() + await async_wait_recording_done_without_instance(hass) + + def _fetch_states() -> list[State]: + with session_scope(hass=hass) as session: + native_states = [] + for db_state, db_state_attributes in session.query(States, StateAttributes): + state = db_state.to_native() + state.attributes = db_state_attributes.to_native() + native_states.append(state) + return native_states + + states: list[State] = await hass.async_add_executor_job(_fetch_states) + assert len(states) > 1 + for state in states: + assert ATTR_MIN_HUMIDITY not in state.attributes + assert ATTR_MAX_HUMIDITY not in state.attributes + assert ATTR_AVAILABLE_MODES not in state.attributes + assert ATTR_FRIENDLY_NAME in state.attributes