🧪 Add unit tests for Labels string parsing (#45)

rnovatorov · google-labs-jules[bot] · web-flow · commit 5a4854df8760 · 2026-04-01T16:02:24.000+02:00
* 🧪 Add unit tests for Labels string parsing and fix edge cases

- Added unit tests for `Labels.parse` covering multiple labels, single labels, empty strings, and multiple spaces.
- Fixed `Labels.parse` to gracefully handle empty strings and redundant whitespace.
- Added a `conftest.py` in the new test directory to mock missing dependencies in restricted environments.

Co-authored-by: rnovatorov &lt;20299819+rnovatorov@users.noreply.github.com&gt;

* 🧪 Remove unnecessary conftest.py mocking

As requested, removed the `conftest.py` file that was mocking dependencies, as they are expected to be installed in the environment.

Co-authored-by: rnovatorov &lt;20299819+rnovatorov@users.noreply.github.com&gt;

* 🧪 Simplify Labels.parse implementation

Switched to using `s.split()` which more cleanly handles empty strings and multiple spaces compared to the previous explicit checks.

Co-authored-by: rnovatorov &lt;20299819+rnovatorov@users.noreply.github.com&gt;

* 🧪 Restore original Labels.parse implementation and update tests

Reverted the simplification of `Labels.parse` to its original state as requested, and updated the unit tests to match this behavior.

Co-authored-by: rnovatorov &lt;20299819+rnovatorov@users.noreply.github.com&gt;

---------

Co-authored-by: google-labs-jules[bot] &lt;161369871+google-labs-jules[bot]@users.noreply.github.com&gt;
diff --git a/tests/unit/test_http/test_api/test_telemetry/test_labels.py b/tests/unit/test_http/test_api/test_telemetry/test_labels.py
@@ -0,0 +1,16 @@
+from enapter.http.api.telemetry.labels import Labels
+
+
+def test_parse_multiple_labels():
+    s = "device=foo telemetry=bar custom=baz"
+    labels = Labels.parse(s)
+    assert labels == {"device": "foo", "telemetry": "bar", "custom": "baz"}
+    assert labels.device == "foo"
+    assert labels.telemetry == "bar"
+
+
+def test_parse_single_label():
+    s = "device=only"
+    labels = Labels.parse(s)
+    assert labels == {"device": "only"}
+    assert labels.device == "only"
