fix: correct unit getter for heating.gas.consumption.summary.dhw (#380)

fix: correct unit getter for heating.gas.consumption.summary.dhw (#9) * Update PyViCareGazBoiler.py * Update test_Vitodens100W.py * Update PyViCareGazBoiler.py
openviess · Sep 6, 2024 · 950fd65 · 950fd65
1 parent e6e189e
commit 950fd65
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/PyViCare/PyViCareGazBoiler.py b/PyViCare/PyViCareGazBoiler.py
@@ -180,7 +180,7 @@ def getGasSummaryConsumptionHeatingLastYear(self):
     # Gas consumption for Domestic Hot Water data:
     @handleNotSupported
     def getGasSummaryConsumptionDomesticHotWaterUnit(self):
-        return self.service.getProperty("heating.gas.consumption.summary.dhw")["properties"]["day"]["unit"]
+        return self.service.getProperty("heating.gas.consumption.summary.dhw")["properties"]["unit"]["value"]
 
     @handleNotSupported
     def getGasSummaryConsumptionDomesticHotWaterCurrentDay(self):

diff --git a/tests/test_Vitodens100W.py b/tests/test_Vitodens100W.py
@@ -32,3 +32,6 @@ def test_getPowerSummaryConsumptionHeatingCurrentDay(self):
 
     def test_getPowerSummaryConsumptionDomesticHotWaterCurrentYear(self):
         self.assertEqual(self.device.getPowerSummaryConsumptionDomesticHotWaterCurrentYear(), 18)
+
+    def test_getGasSummaryConsumptionDomesticHotWaterUnit(self):
+        self.assertEqual(self.device.getGasSummaryConsumptionDomesticHotWaterUnit(), "cubicMeter")