From 4f8c7fe05b823466621287cc078712865efb0d37 Mon Sep 17 00:00:00 2001 From: "J. Nick Koston" Date: Sun, 3 Apr 2022 07:43:44 -0600 Subject: [PATCH] Exclude static number attributes from being recorded in the database (#69194) --- homeassistant/components/number/recorder.py | 17 ++++++++ tests/components/number/test_recorder.py | 46 +++++++++++++++++++++ 2 files changed, 63 insertions(+) create mode 100644 homeassistant/components/number/recorder.py create mode 100644 tests/components/number/test_recorder.py diff --git a/homeassistant/components/number/recorder.py b/homeassistant/components/number/recorder.py new file mode 100644 index 00000000000..39418a48878 --- /dev/null +++ b/homeassistant/components/number/recorder.py @@ -0,0 +1,17 @@ +"""Integration platform for recorder.""" +from __future__ import annotations + +from homeassistant.core import HomeAssistant, callback + +from . import ATTR_MAX, ATTR_MIN, ATTR_MODE, ATTR_STEP + + +@callback +def exclude_attributes(hass: HomeAssistant) -> set[str]: + """Exclude static attributes from being recorded in the database.""" + return { + ATTR_MIN, + ATTR_MAX, + ATTR_STEP, + ATTR_MODE, + } diff --git a/tests/components/number/test_recorder.py b/tests/components/number/test_recorder.py new file mode 100644 index 00000000000..26eca4eb4da --- /dev/null +++ b/tests/components/number/test_recorder.py @@ -0,0 +1,46 @@ +"""The tests for number recorder.""" +from __future__ import annotations + +from datetime import timedelta + +from homeassistant.components import number +from homeassistant.components.number import ATTR_MAX, ATTR_MIN, ATTR_MODE, ATTR_STEP +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 number registered attributes to be excluded.""" + await async_init_recorder_component(hass) + await async_setup_component( + hass, number.DOMAIN, {number.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 not in state.attributes + assert ATTR_MAX not in state.attributes + assert ATTR_STEP not in state.attributes + assert ATTR_MODE not in state.attributes + assert ATTR_FRIENDLY_NAME in state.attributes