Add sensor tests to PVOutput (#63400)

.coveragerc

@@ -869,7 +869,6 @@ omit =
     homeassistant/components/pushbullet/sensor.py
     homeassistant/components/pushover/notify.py
     homeassistant/components/pushsafer/notify.py
-    homeassistant/components/pvoutput/sensor.py
     homeassistant/components/pyload/sensor.py
     homeassistant/components/qbittorrent/sensor.py
     homeassistant/components/qnap/sensor.py

homeassistant/components/pvoutput/sensor.py

@@ -195,7 +195,7 @@ class PVOutputSensorEntity(CoordinatorEntity, SensorEntity):
         self._attr_unique_id = f"{system_id}_{description.key}"
         self._attr_device_info = DeviceInfo(
             configuration_url=f"https://pvoutput.org/list.jsp?sid={system_id}",
-            identifiers={(DOMAIN, system_id)},
+            identifiers={(DOMAIN, str(system_id))},
             manufacturer="PVOutput",
             model=system.inverter_brand,
             name=system.system_name,

tests/components/pvoutput/test_sensor.py

@@ -0,0 +1,138 @@
+"""Tests for the sensors provided by the PVOutput integration."""
+from homeassistant.components.pvoutput.const import DOMAIN
+from homeassistant.components.sensor import (
+    ATTR_STATE_CLASS,
+    SensorDeviceClass,
+    SensorStateClass,
+)
+from homeassistant.const import (
+    ATTR_DEVICE_CLASS,
+    ATTR_FRIENDLY_NAME,
+    ATTR_ICON,
+    ATTR_UNIT_OF_MEASUREMENT,
+    ELECTRIC_POTENTIAL_VOLT,
+    ENERGY_KILO_WATT_HOUR,
+    ENERGY_WATT_HOUR,
+    POWER_KILO_WATT,
+    POWER_WATT,
+    TEMP_CELSIUS,
+)
+from homeassistant.core import HomeAssistant
+from homeassistant.helpers import device_registry as dr, entity_registry as er
+
+from tests.common import MockConfigEntry
+
+
+async def test_sensors(
+    hass: HomeAssistant,
+    init_integration: MockConfigEntry,
+) -> None:
+    """Test the PVOutput sensors."""
+    entity_registry = er.async_get(hass)
+    device_registry = dr.async_get(hass)
+
+    state = hass.states.get("sensor.energy_consumed")
+    entry = entity_registry.async_get("sensor.energy_consumed")
+    assert entry
+    assert state
+    assert entry.unique_id == "12345_energy_consumption"
+    assert entry.entity_category is None
+    assert state.state == "1000"
+    assert state.attributes.get(ATTR_DEVICE_CLASS) == SensorDeviceClass.ENERGY
+    assert state.attributes.get(ATTR_FRIENDLY_NAME) == "Energy Consumed"
+    assert state.attributes.get(ATTR_STATE_CLASS) is SensorStateClass.TOTAL_INCREASING
+    assert state.attributes.get(ATTR_UNIT_OF_MEASUREMENT) == ENERGY_WATT_HOUR
+    assert ATTR_ICON not in state.attributes
+
+    state = hass.states.get("sensor.energy_generated")
+    entry = entity_registry.async_get("sensor.energy_generated")
+    assert entry
+    assert state
+    assert entry.unique_id == "12345_energy_generation"
+    assert entry.entity_category is None
+    assert state.state == "500"
+    assert state.attributes.get(ATTR_DEVICE_CLASS) == SensorDeviceClass.ENERGY
+    assert state.attributes.get(ATTR_FRIENDLY_NAME) == "Energy Generated"
+    assert state.attributes.get(ATTR_STATE_CLASS) is SensorStateClass.TOTAL_INCREASING
+    assert state.attributes.get(ATTR_UNIT_OF_MEASUREMENT) == ENERGY_WATT_HOUR
+    assert ATTR_ICON not in state.attributes
+
+    state = hass.states.get("sensor.efficiency")
+    entry = entity_registry.async_get("sensor.efficiency")
+    assert entry
+    assert state
+    assert entry.unique_id == "12345_normalized_output"
+    assert entry.entity_category is None
+    assert state.state == "0.5"
+    assert state.attributes.get(ATTR_FRIENDLY_NAME) == "Efficiency"
+    assert state.attributes.get(ATTR_STATE_CLASS) is SensorStateClass.MEASUREMENT
+    assert (
+        state.attributes.get(ATTR_UNIT_OF_MEASUREMENT)
+        == f"{ENERGY_KILO_WATT_HOUR}/{POWER_KILO_WATT}"
+    )
+    assert ATTR_DEVICE_CLASS not in state.attributes
+    assert ATTR_ICON not in state.attributes
+
+    state = hass.states.get("sensor.power_consumed")
+    entry = entity_registry.async_get("sensor.power_consumed")
+    assert entry
+    assert state
+    assert entry.unique_id == "12345_power_consumption"
+    assert entry.entity_category is None
+    assert state.state == "2500"
+    assert state.attributes.get(ATTR_DEVICE_CLASS) == SensorDeviceClass.POWER
+    assert state.attributes.get(ATTR_FRIENDLY_NAME) == "Power Consumed"
+    assert state.attributes.get(ATTR_STATE_CLASS) is SensorStateClass.MEASUREMENT
+    assert state.attributes.get(ATTR_UNIT_OF_MEASUREMENT) == POWER_WATT
+    assert ATTR_ICON not in state.attributes
+
+    state = hass.states.get("sensor.power_generated")
+    entry = entity_registry.async_get("sensor.power_generated")
+    assert entry
+    assert state
+    assert entry.unique_id == "12345_power_generation"
+    assert entry.entity_category is None
+    assert state.state == "1500"
+    assert state.attributes.get(ATTR_DEVICE_CLASS) == SensorDeviceClass.POWER
+    assert state.attributes.get(ATTR_FRIENDLY_NAME) == "Power Generated"
+    assert state.attributes.get(ATTR_STATE_CLASS) is SensorStateClass.MEASUREMENT
+    assert state.attributes.get(ATTR_UNIT_OF_MEASUREMENT) == POWER_WATT
+    assert ATTR_ICON not in state.attributes
+
+    state = hass.states.get("sensor.temperature")
+    entry = entity_registry.async_get("sensor.temperature")
+    assert entry
+    assert state
+    assert entry.unique_id == "12345_temperature"
+    assert entry.entity_category is None
+    assert state.state == "20.2"
+    assert state.attributes.get(ATTR_DEVICE_CLASS) == SensorDeviceClass.TEMPERATURE
+    assert state.attributes.get(ATTR_FRIENDLY_NAME) == "Temperature"
+    assert state.attributes.get(ATTR_STATE_CLASS) is SensorStateClass.MEASUREMENT
+    assert state.attributes.get(ATTR_UNIT_OF_MEASUREMENT) == TEMP_CELSIUS
+    assert ATTR_ICON not in state.attributes
+
+    state = hass.states.get("sensor.voltage")
+    entry = entity_registry.async_get("sensor.voltage")
+    assert entry
+    assert state
+    assert entry.unique_id == "12345_voltage"
+    assert entry.entity_category is None
+    assert state.state == "220.5"
+    assert state.attributes.get(ATTR_DEVICE_CLASS) == SensorDeviceClass.VOLTAGE
+    assert state.attributes.get(ATTR_FRIENDLY_NAME) == "Voltage"
+    assert state.attributes.get(ATTR_STATE_CLASS) is SensorStateClass.MEASUREMENT
+    assert state.attributes.get(ATTR_UNIT_OF_MEASUREMENT) == ELECTRIC_POTENTIAL_VOLT
+    assert ATTR_ICON not in state.attributes
+
+    assert entry.device_id
+    device_entry = device_registry.async_get(entry.device_id)
+    assert device_entry
+    assert device_entry.identifiers == {(DOMAIN, "12345")}
+    assert device_entry.manufacturer == "PVOutput"
+    assert device_entry.model == "Super Inverters Inc."
+    assert device_entry.name == "Frenck's Solar Farm"
+    assert device_entry.configuration_url == "https://pvoutput.org/list.jsp?sid=12345"
+    assert device_entry.entry_type is None
+    assert device_entry.sw_version is None
+    assert device_entry.hw_version is None