evlli/hass-core
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix unit of measurement for P1 Montior (#57495)

Browse source
This commit is contained in:
Klaas Schoute 2021-10-22 18:24:12 +02:00 • committed by GitHub
parent a3b3c4ebad
commit efbe8a8689
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 20 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

21
homeassistant/components/p1_monitor/sensor.py
View file

@ -19,7 +19,6 @@ from homeassistant.const import (
DEVICE_CLASS_CURRENT, DEVICE_CLASS_CURRENT,
DEVICE_CLASS_ENERGY, DEVICE_CLASS_ENERGY,
DEVICE_CLASS_GAS, DEVICE_CLASS_GAS,
DEVICE_CLASS_MONETARY,
DEVICE_CLASS_POWER, DEVICE_CLASS_POWER,
DEVICE_CLASS_VOLTAGE, DEVICE_CLASS_VOLTAGE,
ELECTRIC_CURRENT_AMPERE, ELECTRIC_CURRENT_AMPERE,
@ -195,32 +194,32 @@ SENSORS: dict[
key="gas_consumption_price", key="gas_consumption_price",
name="Gas Consumption Price", name="Gas Consumption Price",
entity_registry_enabled_default=False, entity_registry_enabled_default=False,
device_class=DEVICE_CLASS_MONETARY, state_class=STATE_CLASS_MEASUREMENT,
native_unit_of_measurement=CURRENCY_EURO, native_unit_of_measurement=f"{CURRENCY_EURO}/{VOLUME_CUBIC_METERS}",
), ),
SensorEntityDescription( SensorEntityDescription(
key="energy_consumption_price_low", key="energy_consumption_price_low",
name="Energy Consumption Price - Low", name="Energy Consumption Price - Low",
device_class=DEVICE_CLASS_MONETARY, state_class=STATE_CLASS_MEASUREMENT,
native_unit_of_measurement=CURRENCY_EURO, native_unit_of_measurement=f"{CURRENCY_EURO}/{ENERGY_KILO_WATT_HOUR}",
), ),
SensorEntityDescription( SensorEntityDescription(
key="energy_consumption_price_high", key="energy_consumption_price_high",
name="Energy Consumption Price - High", name="Energy Consumption Price - High",
device_class=DEVICE_CLASS_MONETARY, state_class=STATE_CLASS_MEASUREMENT,
native_unit_of_measurement=CURRENCY_EURO, native_unit_of_measurement=f"{CURRENCY_EURO}/{ENERGY_KILO_WATT_HOUR}",
), ),
SensorEntityDescription( SensorEntityDescription(
key="energy_production_price_low", key="energy_production_price_low",
name="Energy Production Price - Low", name="Energy Production Price - Low",
device_class=DEVICE_CLASS_MONETARY, state_class=STATE_CLASS_MEASUREMENT,
native_unit_of_measurement=CURRENCY_EURO, native_unit_of_measurement=f"{CURRENCY_EURO}/{ENERGY_KILO_WATT_HOUR}",
), ),
SensorEntityDescription( SensorEntityDescription(
key="energy_production_price_high", key="energy_production_price_high",
name="Energy Production Price - High", name="Energy Production Price - High",
device_class=DEVICE_CLASS_MONETARY, state_class=STATE_CLASS_MEASUREMENT,
native_unit_of_measurement=CURRENCY_EURO, native_unit_of_measurement=f"{CURRENCY_EURO}/{ENERGY_KILO_WATT_HOUR}",
), ),
), ),
} }

15
tests/components/p1_monitor/test_sensor.py
View file

@ -15,7 +15,6 @@ from homeassistant.const import (
CURRENCY_EURO, CURRENCY_EURO,
DEVICE_CLASS_CURRENT, DEVICE_CLASS_CURRENT,
DEVICE_CLASS_ENERGY, DEVICE_CLASS_ENERGY,
DEVICE_CLASS_MONETARY,
DEVICE_CLASS_POWER, DEVICE_CLASS_POWER,
DEVICE_CLASS_VOLTAGE, DEVICE_CLASS_VOLTAGE,
ELECTRIC_CURRENT_AMPERE, ELECTRIC_CURRENT_AMPERE,
@ -158,8 +157,11 @@ async def test_settings(
assert entry.unique_id == f"{entry_id}_settings_energy_consumption_price_low" assert entry.unique_id == f"{entry_id}_settings_energy_consumption_price_low"
assert state.state == "0.20522" assert state.state == "0.20522"
assert state.attributes.get(ATTR_FRIENDLY_NAME) == "Energy Consumption Price - Low" assert state.attributes.get(ATTR_FRIENDLY_NAME) == "Energy Consumption Price - Low"
assert state.attributes.get(ATTR_DEVICE_CLASS) == DEVICE_CLASS_MONETARY assert state.attributes.get(ATTR_STATE_CLASS) == STATE_CLASS_MEASUREMENT
assert state.attributes.get(ATTR_UNIT_OF_MEASUREMENT) == CURRENCY_EURO assert (
state.attributes.get(ATTR_UNIT_OF_MEASUREMENT)
== f"{CURRENCY_EURO}/{ENERGY_KILO_WATT_HOUR}"
)
state = hass.states.get("sensor.monitor_energy_production_price_low") state = hass.states.get("sensor.monitor_energy_production_price_low")
entry = entity_registry.async_get("sensor.monitor_energy_production_price_low") entry = entity_registry.async_get("sensor.monitor_energy_production_price_low")
@ -168,8 +170,11 @@ async def test_settings(
assert entry.unique_id == f"{entry_id}_settings_energy_production_price_low" assert entry.unique_id == f"{entry_id}_settings_energy_production_price_low"
assert state.state == "0.20522" assert state.state == "0.20522"
assert state.attributes.get(ATTR_FRIENDLY_NAME) == "Energy Production Price - Low" assert state.attributes.get(ATTR_FRIENDLY_NAME) == "Energy Production Price - Low"
assert state.attributes.get(ATTR_DEVICE_CLASS) == DEVICE_CLASS_MONETARY assert state.attributes.get(ATTR_STATE_CLASS) == STATE_CLASS_MEASUREMENT
assert state.attributes.get(ATTR_UNIT_OF_MEASUREMENT) == CURRENCY_EURO assert (
state.attributes.get(ATTR_UNIT_OF_MEASUREMENT)
== f"{CURRENCY_EURO}/{ENERGY_KILO_WATT_HOUR}"
)
assert entry.device_id assert entry.device_id
device_entry = device_registry.async_get(entry.device_id) device_entry = device_registry.async_get(entry.device_id)