| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377 |
- """Tests for the active Tuya LAN rediscovery sweeper."""
- import logging
- from unittest.mock import AsyncMock
- import pytest
- from homeassistant.const import CONF_HOST
- from pytest_homeassistant_custom_component.common import MockConfigEntry
- from custom_components.tuya_local.const import (
- CONF_DEVICE_ID,
- CONF_LOCAL_KEY,
- CONF_POLL_ONLY,
- CONF_PROTOCOL_VERSION,
- CONF_TYPE,
- DATA_DISCOVERY,
- DOMAIN,
- )
- from custom_components.tuya_local.helpers import discovery
- from custom_components.tuya_local.helpers.discovery import (
- TuyaLANRediscovery,
- async_start_discovery,
- async_stop_discovery,
- )
- TESTKEY = ")<jO<@)'P1|kR$Kd"
- DEVID = "bf1234567890abcdef"
- @pytest.fixture(autouse=True)
- def auto_enable_custom_integrations(enable_custom_integrations):
- yield
- def _make_entry(hass, host="192.168.1.10", options=None):
- entry = MockConfigEntry(
- domain=DOMAIN,
- version=13,
- minor_version=20,
- title="thermostat",
- data={
- CONF_DEVICE_ID: DEVID,
- CONF_HOST: host,
- CONF_LOCAL_KEY: TESTKEY,
- CONF_POLL_ONLY: False,
- CONF_PROTOCOL_VERSION: "auto",
- CONF_TYPE: "polytherm_polyalpha_thermostat",
- },
- options=options or {},
- )
- entry.add_to_hass(hass)
- return entry
- def _set_device(hass, returned_state, device_id=DEVID):
- """Register a fake device object in hass.data under the device id."""
- device = type("Dev", (), {"has_returned_state": returned_state})()
- hass.data.setdefault(DOMAIN, {})[device_id] = {"device": device}
- return device
- @pytest.mark.asyncio
- async def test_sweep_updates_unreachable_changed_host(hass, caplog, mocker):
- """An unreachable device gets relocated, its host updated, and it's logged at WARNING."""
- entry = _make_entry(hass, host="192.168.1.10")
- _set_device(hass, returned_state=False)
- mocker.patch(
- "custom_components.tuya_local.helpers.discovery._find_device",
- return_value={"ip": "192.168.1.55", "id": DEVID},
- )
- with caplog.at_level(
- logging.WARNING, logger="custom_components.tuya_local.helpers.discovery"
- ):
- await TuyaLANRediscovery(hass)._async_sweep()
- await hass.async_block_till_done()
- assert entry.data[CONF_HOST] == "192.168.1.55"
- # The IP change must be visible even when the entry runs at WARNING.
- assert "192.168.1.55" in caplog.text
- assert "192.168.1.10" in caplog.text
- @pytest.mark.asyncio
- async def test_sweep_skips_reachable_device(hass, mocker):
- """A device that is returning state is never scanned."""
- entry = _make_entry(hass, host="192.168.1.10")
- _set_device(hass, returned_state=True)
- find = mocker.patch(
- "custom_components.tuya_local.helpers.discovery._find_device",
- return_value={"ip": "192.168.1.55"},
- )
- await TuyaLANRediscovery(hass)._async_sweep()
- await hass.async_block_till_done()
- find.assert_not_called()
- assert entry.data[CONF_HOST] == "192.168.1.10"
- @pytest.mark.asyncio
- async def test_sweep_no_change_when_ip_same(hass, mocker):
- """If the scan returns the current IP, no entry update happens."""
- entry = _make_entry(hass, host="192.168.1.10")
- _set_device(hass, returned_state=False)
- mocker.patch(
- "custom_components.tuya_local.helpers.discovery._find_device",
- return_value={"ip": "192.168.1.10"},
- )
- update = mocker.spy(hass.config_entries, "async_update_entry")
- await TuyaLANRediscovery(hass)._async_sweep()
- await hass.async_block_till_done()
- update.assert_not_called()
- assert entry.data[CONF_HOST] == "192.168.1.10"
- @pytest.mark.asyncio
- async def test_sweep_handles_not_found(hass, mocker):
- """A scan that finds nothing must not raise or change anything."""
- entry = _make_entry(hass, host="192.168.1.10")
- _set_device(hass, returned_state=False)
- mocker.patch(
- "custom_components.tuya_local.helpers.discovery._find_device",
- return_value={"ip": None},
- )
- update = mocker.spy(hass.config_entries, "async_update_entry")
- await TuyaLANRediscovery(hass)._async_sweep()
- await hass.async_block_till_done()
- update.assert_not_called()
- assert entry.data[CONF_HOST] == "192.168.1.10"
- @pytest.mark.asyncio
- async def test_sweep_updates_host_stored_in_options(hass, mocker):
- """When the effective host lives in options, the update targets options."""
- entry = _make_entry(hass, host="10.0.0.1", options={CONF_HOST: "192.168.1.103"})
- _set_device(hass, returned_state=False)
- mocker.patch(
- "custom_components.tuya_local.helpers.discovery._find_device",
- return_value={"ip": "192.168.1.55"},
- )
- await TuyaLANRediscovery(hass)._async_sweep()
- await hass.async_block_till_done()
- assert entry.options[CONF_HOST] == "192.168.1.55"
- assert entry.data[CONF_HOST] == "192.168.1.55"
- @pytest.mark.asyncio
- async def test_sweep_scans_when_no_device_object(hass, mocker):
- """An entry with no device object yet (failed setup) is still scanned."""
- entry = _make_entry(hass, host="192.168.1.10")
- hass.data.setdefault(DOMAIN, {}) # no device bucket registered
- mocker.patch(
- "custom_components.tuya_local.helpers.discovery._find_device",
- return_value={"ip": "192.168.1.77"},
- )
- await TuyaLANRediscovery(hass)._async_sweep()
- await hass.async_block_till_done()
- assert entry.data[CONF_HOST] == "192.168.1.77"
- @pytest.mark.asyncio
- async def test_start_is_idempotent_and_stop_cancels(hass, mocker):
- """async_start_discovery schedules the sweep + scan intervals; stop cancels both."""
- unsub_sweep = mocker.MagicMock()
- unsub_scan = mocker.MagicMock()
- track = mocker.patch(
- "custom_components.tuya_local.helpers.discovery.async_track_time_interval",
- side_effect=[unsub_sweep, unsub_scan],
- )
- await async_start_discovery(hass)
- rediscovery = hass.data[DOMAIN][DATA_DISCOVERY]
- assert isinstance(rediscovery, TuyaLANRediscovery)
- assert track.call_count == 2
- # Second call must not schedule more intervals (singleton).
- await async_start_discovery(hass)
- assert track.call_count == 2
- async_stop_discovery(hass)
- unsub_sweep.assert_called_once()
- unsub_scan.assert_called_once()
- assert DATA_DISCOVERY not in hass.data[DOMAIN]
- def _fake_config(matches):
- """Minimal stand-in for a device config with a matches_product() method."""
- return type("Cfg", (), {"matches_product": lambda self, pid: matches})()
- def _scan_result(gwid=DEVID, product="keyabc123", ip="192.168.1.10"):
- """A tinytuya.deviceScan-style result: keyed by IP, carrying gwId/productKey."""
- info = {"gwId": gwid, "ip": ip, "version": "3.5"}
- if product is not None:
- info["productKey"] = product
- return {ip: info}
- def _patch_flow_init(hass, mocker):
- """Patch the config-entries flow init with an awaitable mock."""
- return mocker.patch.object(
- hass.config_entries.flow, "async_init", new_callable=AsyncMock
- )
- @pytest.mark.asyncio
- async def test_product_scan_warns_once_on_unmatched_product(hass, caplog, mocker):
- """An unmatched product id is logged at WARNING, once per device per run."""
- _make_entry(hass, host="192.168.1.10")
- mocker.patch(
- "custom_components.tuya_local.helpers.discovery._scan_all",
- return_value=_scan_result(),
- )
- mocker.patch(
- "custom_components.tuya_local.helpers.discovery.get_config",
- return_value=_fake_config(False),
- )
- _patch_flow_init(hass, mocker)
- disc = TuyaLANRediscovery(hass)
- with caplog.at_level(
- logging.WARNING, logger="custom_components.tuya_local.helpers.discovery"
- ):
- await disc._async_discovery_scan()
- await hass.async_block_till_done()
- assert caplog.text.count("keyabc123") == 1
- # A second scan must not warn again for the same device.
- caplog.clear()
- await disc._async_discovery_scan()
- await hass.async_block_till_done()
- assert "keyabc123" not in caplog.text
- @pytest.mark.asyncio
- async def test_product_scan_silent_when_product_matches(hass, caplog, mocker):
- """No warning when the product id is listed in the config."""
- _make_entry(hass, host="192.168.1.10")
- mocker.patch(
- "custom_components.tuya_local.helpers.discovery._scan_all",
- return_value=_scan_result(),
- )
- mocker.patch(
- "custom_components.tuya_local.helpers.discovery.get_config",
- return_value=_fake_config(True),
- )
- _patch_flow_init(hass, mocker)
- with caplog.at_level(
- logging.WARNING, logger="custom_components.tuya_local.helpers.discovery"
- ):
- await TuyaLANRediscovery(hass)._async_discovery_scan()
- await hass.async_block_till_done()
- assert "is not listed" not in caplog.text
- @pytest.mark.asyncio
- async def test_product_scan_skips_when_no_product_id(hass, mocker):
- """If the scan reports no product id, the config is not even looked up."""
- _make_entry(hass, host="192.168.1.10")
- mocker.patch(
- "custom_components.tuya_local.helpers.discovery._scan_all",
- return_value=_scan_result(product=None),
- )
- get_config = mocker.patch(
- "custom_components.tuya_local.helpers.discovery.get_config",
- )
- _patch_flow_init(hass, mocker)
- await TuyaLANRediscovery(hass)._async_discovery_scan()
- await hass.async_block_till_done()
- get_config.assert_not_called()
- @pytest.mark.asyncio
- async def test_product_scan_handles_missing_config(hass, caplog, mocker):
- """A missing config file must not warn or raise."""
- _make_entry(hass, host="192.168.1.10")
- mocker.patch(
- "custom_components.tuya_local.helpers.discovery._scan_all",
- return_value=_scan_result(),
- )
- mocker.patch(
- "custom_components.tuya_local.helpers.discovery.get_config",
- return_value=None,
- )
- _patch_flow_init(hass, mocker)
- with caplog.at_level(
- logging.WARNING, logger="custom_components.tuya_local.helpers.discovery"
- ):
- await TuyaLANRediscovery(hass)._async_discovery_scan()
- await hass.async_block_till_done()
- assert "keyabc123" not in caplog.text
- @pytest.mark.asyncio
- async def test_discovery_raises_flow_for_unknown_device(hass, mocker):
- """An unconfigured device on the LAN starts an integration_discovery flow."""
- mocker.patch(
- "custom_components.tuya_local.helpers.discovery._scan_all",
- return_value=_scan_result(gwid="bfunknown000000000", ip="192.168.1.99"),
- )
- init = _patch_flow_init(hass, mocker)
- await TuyaLANRediscovery(hass)._async_discovery_scan()
- await hass.async_block_till_done()
- init.assert_awaited_once()
- args, kwargs = init.call_args
- assert args[0] == DOMAIN
- assert kwargs["context"]["source"] == "integration_discovery"
- assert kwargs["data"][CONF_DEVICE_ID] == "bfunknown000000000"
- assert kwargs["data"][CONF_HOST] == "192.168.1.99"
- @pytest.mark.asyncio
- async def test_discovery_skips_configured_device(hass, mocker):
- """A device already configured is not offered for discovery again."""
- _make_entry(hass, host="192.168.1.10") # DEVID is configured
- mocker.patch(
- "custom_components.tuya_local.helpers.discovery._scan_all",
- return_value=_scan_result(gwid=DEVID),
- )
- mocker.patch(
- "custom_components.tuya_local.helpers.discovery.get_config",
- return_value=_fake_config(True),
- )
- init = _patch_flow_init(hass, mocker)
- await TuyaLANRediscovery(hass)._async_discovery_scan()
- await hass.async_block_till_done()
- init.assert_not_awaited()
- @pytest.mark.asyncio
- async def test_discovery_raises_flow_only_once_per_device(hass, mocker):
- """Repeated scans do not spawn duplicate flows for the same new device."""
- mocker.patch(
- "custom_components.tuya_local.helpers.discovery._scan_all",
- return_value=_scan_result(gwid="bfunknown000000000", ip="192.168.1.99"),
- )
- init = _patch_flow_init(hass, mocker)
- disc = TuyaLANRediscovery(hass)
- await disc._async_discovery_scan()
- await hass.async_block_till_done()
- await disc._async_discovery_scan()
- await hass.async_block_till_done()
- assert init.await_count == 1
- @pytest.mark.asyncio
- async def test_discovery_scan_handles_empty_result(hass, mocker):
- """An empty scan (e.g. socket error) does nothing and does not raise."""
- _make_entry(hass, host="192.168.1.10")
- mocker.patch(
- "custom_components.tuya_local.helpers.discovery._scan_all",
- return_value={},
- )
- init = _patch_flow_init(hass, mocker)
- await TuyaLANRediscovery(hass)._async_discovery_scan()
- await hass.async_block_till_done()
- init.assert_not_awaited()
- def test_module_exposes_expected_intervals():
- """Guard the cadences against accidental change."""
- assert discovery.SWEEP_INTERVAL.total_seconds() == 60
- assert discovery.SCAN_INTERVAL.total_seconds() == 600
|