fix: correct calculation dc link voltage sensor state

Correct calculation of dc link voltage sensor state. Seems the values where reversed fixes #364
henricm · Jul 8, 2023 · a4a6417 · a4a6417
1 parent 56df5e9
commit a4a6417
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/custom_components/ferroamp/sensor.py b/custom_components/ferroamp/sensor.py
@@ -756,7 +756,7 @@ def update_state_from_events(self, events):
         if neg is None and pos is None:
             return False
         else:
-            self._attr_native_value = int(neg / count + pos / count)
+            self._attr_native_value = round(float(pos / count - neg / count), 2)
             self._attr_extra_state_attributes = dict(neg=round(float(neg / count), 2),
                                                      pos=round(float(pos / count), 2))
             return True

diff --git a/tests/test_sensor.py b/tests/test_sensor.py
@@ -428,7 +428,7 @@ async def test_setting_ehub_sensor_values_via_mqtt_message(hass, mqtt_mock):
     }
 
     state = hass.states.get("sensor.ferroamp_dc_link_voltage")
-    assert state.state == "0"
+    assert state.state == "768.27"
     assert state.attributes == {
         'device_class': 'voltage',
         'friendly_name': 'EnergyHub DC Link Voltage',