pyimouapi 1.3.1__tar.gz → 1.3.2__tar.gz

{pyimouapi-1.3.1 → pyimouapi-1.3.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pyimouapi
-Version: 1.3.1
+Version: 1.3.2
 Summary: A package for imou open api
 Home-page: https://github.com/Imou-OpenPlatform/Py-Imou-Open-Api
 Author: Imou-OpenPlatform

{pyimouapi-1.3.1 → pyimouapi-1.3.2}/pyimouapi/__init__.py

@@ -1,4 +1,4 @@
-__version__ = "1.3.1"
+__version__ = "1.3.2"
 
 from .device import ImouChannel, ImouDevice, ImouDeviceManager
 from .exceptions import (

{pyimouapi-1.3.1 → pyimouapi-1.3.2}/pyimouapi/device.py

@@ -258,6 +258,10 @@ class ImouDeviceManager:
         self._event_map_locks: dict[str, asyncio.Lock] = {}
         self._event_map_locks_guard = asyncio.Lock()
 
+    async def async_close(self) -> None:
+        """Close the underlying Open API HTTP session."""
+        await self._imou_api_client.async_close()
+
     async def _async_lock_for_product(self, product_id: str) -> asyncio.Lock:
         async with self._event_map_locks_guard:
             lock = self._event_map_locks.get(product_id)
@@ -555,11 +559,13 @@ class ImouDeviceManager:
                 }
             ]
         }
-        return (
-            await self._imou_api_client.async_request_api(
-                API_ENDPOINT_GET_IOT_DEVICE_PROPERTIES, params
-            )
-        ).get(PARAM_DEVICE_LIST, [{}])[0]
+        data = await self._imou_api_client.async_request_api(
+            API_ENDPOINT_GET_IOT_DEVICE_PROPERTIES, params
+        )
+        device_list = data.get(PARAM_DEVICE_LIST) if isinstance(data, dict) else None
+        if not device_list:
+            return {}
+        return device_list[0]
 
     async def async_set_iot_device_properties(
         self, device_id: str, channel_id: str | None, product_id: str, properties: dict
@@ -655,12 +661,15 @@ class ImouDeviceManager:
         if device_detail.get(PARAM_CHANNELS) and imou_device.channels:
             channels_detail = device_detail.get(PARAM_CHANNELS)
             channel_detail_map = {
-                channel_detail.get(PARAM_CHANNEL_ID): channel_detail.get(
+                str(channel_detail.get(PARAM_CHANNEL_ID)): channel_detail.get(
                     PARAM_ABILITY_REFS, "unknown"
                 )
                 for channel_detail in channels_detail
+                if channel_detail.get(PARAM_CHANNEL_ID) is not None
             }
             for channel in imou_device.channels:
-                channel_id = channel.channel_id
-                if channel_id in channel_detail_map:
+                channel_id = (
+                    str(channel.channel_id) if channel.channel_id is not None else None
+                )
+                if channel_id is not None and channel_id in channel_detail_map:
                     channel.set_channel_ability_refs(channel_detail_map[channel_id])

{pyimouapi-1.3.1 → pyimouapi-1.3.2}/pyimouapi/ha_device.py

@@ -254,6 +254,10 @@ class ImouHaDeviceManager:
     def delegate(self):
         return self._delegate
 
+    async def async_close(self) -> None:
+        """Close the underlying Open API HTTP session."""
+        await self._delegate.async_close()
+
     @staticmethod
     def _resolve_device_id(device: ImouHaDevice) -> str:
         device_id = device.device_id
@@ -482,8 +486,11 @@ class ImouHaDeviceManager:
                     self.get_device_status(data[PARAM_ONLINE]),
                 )
             else:
+                device_channel_id = (
+                    str(device.channel_id) if device.channel_id is not None else None
+                )
                 for channel in data[PARAM_CHANNELS]:
-                    if channel[PARAM_CHANNEL_ID] == device.channel_id:
+                    if str(channel[PARAM_CHANNEL_ID]) == device_channel_id:
                         apply_sensor_state(
                             device.sensors,
                             PARAM_STATUS,
@@ -681,9 +688,12 @@ class ImouHaDeviceManager:
                         + "_"
                         + device.parent_product_id
                     )
-                if value_type == "int" and text_value.isdigit():
-                    value = int(text_value)
-                elif value_type == "str" and not isinstance(value_type, str):
+                if value_type == "int":
+                    try:
+                        value = int(text_value)
+                    except (TypeError, ValueError):
+                        value = text_value
+                elif value_type == "str":
                     value = str(text_value)
                 else:
                     value = text_value

{pyimouapi-1.3.1 → pyimouapi-1.3.2}/pyimouapi.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pyimouapi
-Version: 1.3.1
+Version: 1.3.2
 Summary: A package for imou open api
 Home-page: https://github.com/Imou-OpenPlatform/Py-Imou-Open-Api
 Author: Imou-OpenPlatform

{pyimouapi-1.3.1 → pyimouapi-1.3.2}/pyimouapi.egg-info/SOURCES.txt

@@ -14,6 +14,7 @@ pyimouapi.egg-info/SOURCES.txt
 pyimouapi.egg-info/dependency_links.txt
 pyimouapi.egg-info/requires.txt
 pyimouapi.egg-info/top_level.txt
+tests/test_channel_id_and_api_guards.py
 tests/test_collect_property_entities.py
 tests/test_device_summaries.py
 tests/test_event_identifier_map.py

{pyimouapi-1.3.1 → pyimouapi-1.3.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "pyimouapi"
-version = "1.3.1"
+version = "1.3.2"
 description = "A package for imou open api"
 readme = "README.md"
 requires-python = ">=3.11"

{pyimouapi-1.3.1 → pyimouapi-1.3.2}/setup.py

@@ -4,7 +4,7 @@ from setuptools import find_packages, setup
 
 setup(
     name="pyimouapi",
-    version="1.3.1",
+    version="1.3.2",
     packages=find_packages(),
     python_requires=">=3.11",
     install_requires=[

pyimouapi-1.3.2/tests/test_channel_id_and_api_guards.py

@@ -0,0 +1,97 @@
+"""Guards for channelId type mismatch and empty API payloads."""
+
+from unittest.mock import AsyncMock, MagicMock
+
+import pytest
+from pyimouapi.const import (
+    PARAM_ABILITY_REFS,
+    PARAM_CHANNEL_ID,
+    PARAM_CHANNELS,
+    PARAM_DEVICE_LIST,
+    PARAM_ONLINE,
+    PARAM_REF,
+    PARAM_STATE,
+    PARAM_STATUS,
+    PARAM_VALUE_TYPE,
+)
+from pyimouapi.device import ImouChannel, ImouDevice, ImouDeviceManager
+from pyimouapi.ha_device import DeviceStatus, ImouHaDevice, ImouHaDeviceManager
+
+
+@pytest.mark.asyncio
+async def test_online_status_matches_int_channel_id():
+    """API may return channelId as int while HA device stores str."""
+    device = ImouHaDevice("dev1", "Cam", "Imou", "IPC", "1.0")
+    device.set_channel_id("0")
+    device.sensors[PARAM_STATUS][PARAM_STATE] = DeviceStatus.OFFLINE.value
+
+    delegate = MagicMock()
+    delegate.async_get_device_online_status = AsyncMock(
+        return_value={
+            PARAM_ONLINE: "1",
+            PARAM_CHANNELS: [
+                {PARAM_CHANNEL_ID: 0, PARAM_ONLINE: "1"},
+            ],
+        }
+    )
+    manager = ImouHaDeviceManager(delegate)
+    await manager._async_update_status(device)
+
+    assert device.sensors[PARAM_STATUS][PARAM_STATE] == DeviceStatus.ONLINE.value
+
+
+@pytest.mark.asyncio
+async def test_ability_refs_match_int_channel_id():
+    """Channel abilityRefs lookup must tolerate int/str channelId."""
+    channel = ImouChannel("0", "ch0", "1", "ability")
+    device = ImouDevice("dev1", "Cam", "1", "Imou", "IPC")
+    device.set_channels([channel])
+    device.set_product_id("pid1")
+
+    manager = ImouDeviceManager(MagicMock())
+    manager.async_get_iot_device_detail_info = AsyncMock(
+        return_value={
+            PARAM_ABILITY_REFS: "devAbility",
+            PARAM_CHANNELS: [
+                {PARAM_CHANNEL_ID: 0, PARAM_ABILITY_REFS: "chAbility"},
+            ],
+        }
+    )
+    await manager._async_update_device_ability_refs(device)
+
+    assert channel.channel_ability_refs == "chAbility"
+
+
+@pytest.mark.asyncio
+async def test_get_iot_device_properties_empty_list():
+    """Empty deviceList must not raise IndexError."""
+    api = MagicMock()
+    api.async_request_api = AsyncMock(return_value={PARAM_DEVICE_LIST: []})
+    manager = ImouDeviceManager(api)
+
+    result = await manager.async_get_iot_device_properties(
+        "dev1", "0", "pid", ["10001"]
+    )
+    assert result == {}
+
+
+@pytest.mark.asyncio
+async def test_set_text_value_coerces_str_type():
+    """value_type=str must stringify the payload."""
+    device = ImouHaDevice("dev1", "Plug", "Imou", "Plug", "1.0")
+    device.set_product_id("pid1")
+    device.texts["name"] = {
+        PARAM_REF: "20001",
+        PARAM_VALUE_TYPE: "str",
+        PARAM_STATE: "",
+    }
+
+    delegate = MagicMock()
+    delegate.async_set_iot_device_properties = AsyncMock()
+    manager = ImouHaDeviceManager(delegate)
+
+    await manager.async_set_text_value(device, "name", "hello")
+
+    delegate.async_set_iot_device_properties.assert_awaited_with(
+        "dev1", None, "pid1", {"20001": "hello"}
+    )