Add heat pump supply pressure sensor in ViCare integration (#136265)

homeassistant/components/vicare/sensor.py

@@ -30,6 +30,7 @@ from homeassistant.const import (
     EntityCategory,
     UnitOfEnergy,
     UnitOfPower,
+    UnitOfPressure,
     UnitOfTemperature,
     UnitOfTime,
     UnitOfVolume,
@@ -836,6 +837,16 @@ GLOBAL_SENSORS: tuple[ViCareSensorEntityDescription, ...] = (
             "forcedlevelfour",
         ],
     ),
+    ViCareSensorEntityDescription(
+        key="supply_pressure",
+        translation_key="supply_pressure",
+        device_class=SensorDeviceClass.PRESSURE,
+        native_unit_of_measurement=UnitOfPressure.BAR,
+        state_class=SensorStateClass.MEASUREMENT,
+        entity_category=EntityCategory.DIAGNOSTIC,
+        value_getter=lambda api: api.getSupplyPressure(),
+        unit_getter=lambda api: api.getSupplyPressureUnit(),
+    ),
 )
 
 CIRCUIT_SENSORS: tuple[ViCareSensorEntityDescription, ...] = (

homeassistant/components/vicare/strings.json

@@ -455,6 +455,9 @@
           "silent": "Silent",
           "forcedlevelfour": "Boost"
         }
+      },
+      "supply_pressure": {
+        "name": "Supply pressure"
       }
     },
     "water_heater": {

tests/components/vicare/test_sensor.py

@@ -16,15 +16,25 @@ from tests.common import MockConfigEntry, snapshot_platform
 
 
 @pytest.mark.usefixtures("entity_registry_enabled_by_default")
-async def test_all_heating_entities(
+@pytest.mark.parametrize(
+    ("fixture_type", "fixture_data"),
+    [
+        ("type:boiler", "vicare/Vitodens300W.json"),
+        ("type:heatpump", "vicare/Vitocal250A.json"),
+        ("type:ventilation", "vicare/ViAir300F.json"),
+    ],
+)
+async def test_all_entities(
     hass: HomeAssistant,
+    fixture_type: str,
+    fixture_data: str,
     snapshot: SnapshotAssertion,
     mock_config_entry: MockConfigEntry,
     entity_registry: er.EntityRegistry,
 ) -> None:
     """Test all entities."""
     fixtures: list[Fixture] = [
-        Fixture({"type:boiler"}, "vicare/Vitodens300W.json"),
+        Fixture({fixture_type}, fixture_data),
     ]
     with (
         patch(f"{MODULE}.login", return_value=MockPyViCare(fixtures)),
@@ -35,24 +45,6 @@ async def test_all_heating_entities(
     await snapshot_platform(hass, entity_registry, snapshot, mock_config_entry.entry_id)
 
 
-@pytest.mark.usefixtures("entity_registry_enabled_by_default")
-async def test_all_ventilation_entities(
-    hass: HomeAssistant,
-    snapshot: SnapshotAssertion,
-    mock_config_entry: MockConfigEntry,
-    entity_registry: er.EntityRegistry,
-) -> None:
-    """Test all entities."""
-    fixtures: list[Fixture] = [Fixture({"type:ventilation"}, "vicare/ViAir300F.json")]
-    with (
-        patch(f"{MODULE}.login", return_value=MockPyViCare(fixtures)),
-        patch(f"{MODULE}.PLATFORMS", [Platform.SENSOR]),
-    ):
-        await setup_integration(hass, mock_config_entry)
-
-    await snapshot_platform(hass, entity_registry, snapshot, mock_config_entry.entry_id)
-
-
 @pytest.mark.usefixtures("entity_registry_enabled_by_default")
 async def test_room_sensors(
     hass: HomeAssistant,