🐛 Correctly send metrics with value 0

Before this patch, values 0 were not considered a valid value and were
sent as events

Also now we have tests!

.github/workflows/test.yml

@@ -0,0 +1,25 @@
+name: Python package
+
+on:
+  push:
+    branches:
+      - master
+      - main
+  pull_request:
+
+jobs:
+    execute-test:
+        runs-on: ubuntu-latest
+        steps:
+            - uses: actions/checkout@v3
+            - name: Set up Python
+              uses: actions/setup-python@v4
+              with:
+                    python-version: '3.x'
+            - name: Install dependencies
+              run: |
+                python -m pip install --upgrade pip
+                pip install -r requirements_tests.txt
+            - name: Execute test
+              run: |
+                python -m unittest custom_components/datadog_agentless/tests/test_extract_states.py

custom_components/datadog_agentless/__init__.py

@@ -233,7 +233,7 @@ def extract_states(event: Event[EventStateChangedData]) -> list[Tuple[str,float]
     if new_state is None:
         return states
     main_state = _extract_state(new_state, new_state.entity_id, new_state.state, True)
-    if main_state:
+    if main_state is not None:
         states.append((new_state.entity_id, main_state))
     for key, value in new_state.attributes.items():
         fake_id = new_state.entity_id + "_attribute_" + sanitize(key)

custom_components/datadog_agentless/tests/test_extract_states.py

@@ -0,0 +1,61 @@
+from os import path
+import sys
+current_dir = path.dirname(__file__)
+parent_dir = path.dirname(current_dir)
+sys.path.append(".")
+sys.path.append(parent_dir)
+import time
+
+from homeassistant.core import Event, EventStateChangedData, EventOrigin, Context, State
+from custom_components.datadog_agentless import extract_states, _extract_state
+
+import unittest
+import json
+
+def read_context(d: dict) -> Context:
+    return Context(
+            id=d["id"],
+            user_id=d["user_id"],
+            parent_id=d["parent_id"],
+    )
+
+def read_event(event_raw_data: dict) -> Event[EventStateChangedData]:
+        time_fired_timestamp=None
+        if "time_fired" in event_raw_data:
+            time_fired_timestamp = time.mktime(time.strptime(event_raw_data["time_fired"], "%Y-%m-%dT%H:%M:%S.%f%z"))
+        origin=EventOrigin.local
+        if "origin" in event_raw_data:
+            origin=EventOrigin(event_raw_data["origin"])
+        context=None
+        if "context" in event_raw_data:
+            context = read_context(event_raw_data["context"])
+
+        escd : EventStateChangedData = {
+                "old_state": State.from_dict(event_raw_data["data"]["old_state"]),
+                "new_state": State.from_dict(event_raw_data["data"]["new_state"]),
+                "entity_id": event_raw_data["data"]["entity_id"],
+        }
+
+        return Event(
+                event_type="EventStateChangedData",
+                data=escd,
+                context=context,
+                origin=origin,
+                time_fired_timestamp=time_fired_timestamp
+        )
+
+
+
+class TestExtractStates(unittest.TestCase):
+    def test_value_zero(self):
+        event_raw_data = json.loads('{"event_type": "state_changed", "data": {"entity_id": "sensor.offline_zigbee_devices", "old_state": {"entity_id": "sensor.offline_zigbee_devices", "state": "1", "attributes": {"devices": [], "icon": "mdi:zigbee", "friendly_name": "Offline Zigbee Devices", "state_class": "measurement"}, "last_changed": "2025-01-18T09:26:43.652119+00:00", "last_reported": "2025-01-18T09:26:43.745228+00:00", "last_updated": "2025-01-18T09:26:43.652119+00:00", "context": {"id": "01JHWBRWP2220YP0QYBQHGGG02", "parent_id": null, "user_id": null}}, "new_state": {"entity_id": "sensor.offline_zigbee_devices", "state": "0", "attributes": {"devices": [], "icon": "mdi:zigbee", "friendly_name": "Offline Zigbee Devices", "state_class": "measurement"}, "last_changed": "2025-01-18T09:29:27.354286+00:00", "last_reported": "2025-01-18T09:29:27.354757+00:00", "last_updated": "2025-01-18T09:29:27.354286+00:00", "context": {"id": "01JHWBXWHRPRVT7P9SHSBBXZ4Q", "parent_id": null, "user_id": null}}}, "origin": "LOCAL", "time_fired": "2025-01-18T09:29:27.354286+00:00", "context": {"id": "01JHWBXWHRPRVT7P9SHSBBXZ4Q", "parent_id": null, "user_id": null}}')
+        event = read_event(event_raw_data)
+
+        states = extract_states(event)
+        self.assertEqual(len(states), 1)
+        _, state = states[0]
+        self.assertEqual(state, 0)
+
+
+if __name__ == "__main__":
+    unittest.main()