Marc Billow
df5b704f3e
/alarms/vs/0 and /kidslock/vs/0 were the dump's two unbound hrefs -- both are the exact shapes common.UNIVERSAL already models elsewhere (common.ALARMS, common.KIDS_LOCK_VS_FALLBACK), picked individually rather than pulling in all of UNIVERSAL to match this registry's existing hand-picked-common style. The six-vs-three burner count the reporter originally asked about is expected behavior (the board's own /mode/vs/0 options genuinely advertise six OperationState slots on hardware with three physical burners, with no per-device signal to tell real slots from phantom ones) -- already explained on the issue; this commit is scoped to the coverage warning.
56 lines
2.2 KiB
Python
56 lines
2.2 KiB
Python
"""Tests for the TP2X_DA-KS-COOKTOP-000001 gas cooktop (issue #314).
|
|
|
|
The reporter's actual complaint (six advertised burner slots when only
|
|
three are physically present) is expected: the registry declares a
|
|
generous static superset of slots and gates each on the live options array
|
|
per cooktop.py's own comment; there's no per-device signal that
|
|
distinguishes an advertised-but-nonexistent slot from a real one, so the
|
|
extra entities are disabled by the user, not filtered by the integration.
|
|
This file locks in the actual gap from that issue -- /alarms/vs/0 and
|
|
/kidslock/vs/0 -- now bound via common.ALARMS/KIDS_LOCK_VS_FALLBACK.
|
|
"""
|
|
|
|
from custom_components.localthings.registry.adapter import flatten
|
|
from custom_components.localthings.registry.by_type import resolve
|
|
from custom_components.localthings.registry.discovery import discover
|
|
from tests.conftest import _load_device
|
|
|
|
|
|
def _bound():
|
|
resources = _load_device("gas_cooktop_tp2x_ks")
|
|
reg = resolve(resources)
|
|
return discover(resources, reg.capabilities, reg.pattern_capabilities), resources
|
|
|
|
|
|
def test_resolves_to_gas_cooktop_registry():
|
|
resources = _load_device("gas_cooktop_tp2x_ks")
|
|
reg = resolve(resources)
|
|
assert reg is not None and reg.name == "gas_cooktop"
|
|
|
|
|
|
def test_no_unbound_hrefs():
|
|
_, resources = _bound()
|
|
reg = resolve(resources)
|
|
unbound = []
|
|
discover(resources, reg.capabilities, reg.pattern_capabilities, log=unbound.append)
|
|
assert unbound == []
|
|
|
|
|
|
def test_alarm_code_and_child_lock_bound():
|
|
bound, resources = _bound()
|
|
state = flatten(bound, resources)
|
|
assert state["alarm_code"] == "CT_E"
|
|
# x.com.samsung.da.kidsLock == "Ready" reads as unlocked, same polarity
|
|
# as every other board KIDS_LOCK_VS_FALLBACK covers.
|
|
assert state["child_lock"] is True
|
|
|
|
|
|
def test_advertises_six_burner_slots_only_three_of_which_are_real():
|
|
"""Not a bug to fix -- the board's own /mode/vs/0 options genuinely
|
|
list six OperationState slots (issue #314); the reporter's physical
|
|
cooktop only has three. Locks in the current, documented behavior."""
|
|
bound, resources = _bound()
|
|
state = flatten(bound, resources)
|
|
for i in range(6):
|
|
assert f"burner_{i}_state" in state
|