Set up integration tests

.github/workflows/tests.yml

@@ -0,0 +1,25 @@
+---
+name: Tests
+on:
+  push:
+  pull_request:
+    branches:
+      - main
+permissions: read-all
+jobs:
+  pytest:
+    name: Run tests
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout Code
+        uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1
+        with:
+          persist-credentials: false
+      - name: Set up Python
+        uses: actions/setup-python@6d4fd22b20465ccc0f7354476588e8df7c9ca0e0
+        with:
+          python-version: "3.13"
+      - name: Install test dependencies
+        run: pip install -r requirements-test.txt
+      - name: Run tests
+        run: pytest tests/ -v

.gitignore

@@ -2,3 +2,4 @@
 .DS_Store
 tekmar-482.bbprojectd
 *.pyc
+.venv-test

requirements-test.txt

@@ -0,0 +1,2 @@
+pytest>=8.0.0
+pytest-homeassistant-custom-component>=0.13.0

setup.cfg

@@ -5,3 +5,6 @@ extend-ignore = E203,E501,E701
 
 [isort]
 profile = black
+
+[tool:pytest]
+asyncio_mode = auto

tests/__init__.py

@@ -0,0 +1 @@
+"""Tests for the Tekmar Gateway 482 integration."""

tests/conftest.py

@@ -0,0 +1,3 @@
+"""Activate pytest-homeassistant-custom-component for all tests."""
+
+pytest_plugins = "pytest_homeassistant_custom_component"

tests/ruff.toml

@@ -0,0 +1,14 @@
+# This extends our general Ruff rules specifically for tests
+extend = "../pyproject.toml"
+
+[lint]
+extend-ignore = [
+  "B904",   # Use raise from to specify exception cause
+  "N806",   # Variable {name} in function should be snake_case
+  "N815",   # Variable {name} in class scope should not be mixedCase
+  "SLF001", # Private member accessed: tests often test internals
+]
+
+[lint.isort]
+known-first-party = ["custom_components", "tests"]
+forced-separate = ["tests"]

tests/test_fields.py

@@ -0,0 +1,70 @@
+"""Tests for the Record class in fields.py.
+
+Test module generated by Claude Code.
+"""
+
+from custom_components.tekmar_482.fields import FieldList, Int8, Int16, Record
+
+FORMAT = FieldList("Example", Int16("address"), Int8("setback"), Int8("setpoint"))
+
+
+def test_create_unpacks_little_endian_values():
+    """Test data bytes are unpacked into named values."""
+    record, extra = Record.create(FORMAT, [0xE9, 0x03, 0x01, 0x8C])
+
+    assert record.values == {"address": 1001, "setback": 1, "setpoint": 140}
+    assert extra == []
+
+
+def test_create_keeps_extra_data():
+    """Test bytes beyond the record format are returned as extra data."""
+    record, extra = Record.create(FORMAT, [0xE9, 0x03, 0x01, 0x8C, 0x55])
+
+    assert record["setpoint"] == 140
+    assert extra == [0x55]
+
+
+def test_create_pads_missing_data_with_zeros():
+    """Test short data is padded with zeros and no data gives all zeros."""
+    short, _ = Record.create(FORMAT, [0xE9])
+    empty, _ = Record.create(FORMAT)
+
+    assert short.values == {"address": 0xE9, "setback": 0, "setpoint": 0}
+    assert empty.values == {"address": 0, "setback": 0, "setpoint": 0}
+
+
+def test_pack_round_trips_and_applies_keyword_values():
+    """Test packing returns the original bytes, updated by keyword values."""
+    record, _ = Record.create(FORMAT, [0xE9, 0x03, 0x01, 0x8C])
+
+    assert record.pack()[0] == [0xE9, 0x03, 0x01, 0x8C]
+    assert record.pack(setpoint=0x50)[0] == [0xE9, 0x03, 0x01, 0x50]
+    assert record["setpoint"] == 0x50
+
+
+def test_set_ignores_unknown_fields():
+    """Test set() only changes fields that are in the record format."""
+    record, _ = Record.create(FORMAT)
+
+    record.set(setback=2, unknown=5)
+
+    assert record.values == {"address": 0, "setback": 2, "setpoint": 0}
+
+
+def test_get_defaults_unknown_fields_to_zero():
+    """Test get() returns 0 for names that are not in the record."""
+    record, _ = Record.create(FORMAT, [0xE9, 0x03, 0x01, 0x8C])
+
+    assert record.get("address", "unknown") == {"address": 1001, "unknown": 0}
+
+
+def test_item_access():
+    """Test dictionary-style access, including the 0 default for unknown keys."""
+    record, _ = Record.create(FORMAT)
+
+    record["setback"] = 3
+
+    assert record["setback"] == 3
+    assert record["unknown"] == 0
+    assert list(record) == ["address", "setback", "setpoint"]
+    assert "address" in record