beatbot-cloud 0.4.2__tar.gz → 0.4.4__tar.gz

{beatbot_cloud-0.4.2/src/beatbot_cloud.egg-info → beatbot_cloud-0.4.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: beatbot-cloud
-Version: 0.4.2
+Version: 0.4.4
 Summary: Asynchronous Python client for the Beatbot cloud API
 Author: Beatbot Robotics
 License-Expression: Apache-2.0

{beatbot_cloud-0.4.2 → beatbot_cloud-0.4.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "beatbot-cloud"
-version = "0.4.2"
+version = "0.4.4"
 description = "Asynchronous Python client for the Beatbot cloud API"
 readme = "README.md"
 requires-python = ">=3.11"

{beatbot_cloud-0.4.2 → beatbot_cloud-0.4.4}/src/beatbot_cloud/client.py

@@ -92,7 +92,11 @@ class BeatbotClient:
         except (TimeoutError, ClientError) as err:
             raise BeatbotConnectionError(str(err)) from err
 
-        body = await response.text()
+        try:
+            body = await response.text()
+        except (TimeoutError, ClientError) as err:
+            raise BeatbotConnectionError(str(err)) from err
+
         if response.status in (HTTPStatus.UNAUTHORIZED, HTTPStatus.FORBIDDEN):
             raise BeatbotAuthenticationError(f"Unauthorized: {response.status}")
         if response.status >= HTTPStatus.BAD_REQUEST:
@@ -161,10 +165,10 @@ class BeatbotClient:
             product_category=device.get("productCategory") or "",
             name=device.get("name") or "",
             model=device.get("model") or "",
-            work_status=0,
+            work_status=None,
             work_mode=0,
-            error_code=0,
-            battery_level=0,
+            error_code=None,
+            battery_level=None,
             versions=versions,
             is_online=bool(device.get("isOnline", False)),
             work_mode_options=BeatbotClient._parse_work_mode_options(capabilities),

{beatbot_cloud-0.4.2 → beatbot_cloud-0.4.4}/src/beatbot_cloud/models.py

@@ -40,10 +40,10 @@ class BeatbotDeviceData:
     device_id: str
     product_id: str
     product_category: str
-    work_status: int
+    work_status: int | None
     work_mode: int
-    error_code: int
-    battery_level: int
+    error_code: int | None
+    battery_level: int | None
     versions: list[FirmwareVersion]
     is_online: bool
     child_lock: bool = False

{beatbot_cloud-0.4.2 → beatbot_cloud-0.4.4/src/beatbot_cloud.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: beatbot-cloud
-Version: 0.4.2
+Version: 0.4.4
 Summary: Asynchronous Python client for the Beatbot cloud API
 Author: Beatbot Robotics
 License-Expression: Apache-2.0

{beatbot_cloud-0.4.2 → beatbot_cloud-0.4.4}/tests/test_client.py

@@ -122,6 +122,28 @@ async def test_timeout_is_connection_error():
         await api._request("GET", "/test")
 
 
+@pytest.mark.parametrize(
+    ("error", "message"),
+    [
+        (ClientError("body read failed"), "body read failed"),
+        (TimeoutError("body read timed out"), "body read timed out"),
+    ],
+    ids=["client-error", "timeout"],
+)
+async def test_body_read_error_is_connection_error(error, message):
+    """Reading the response body cannot escape as a raw aiohttp error."""
+    response = Response("{}")
+
+    async def failing_text():
+        raise error
+
+    response.text = failing_text
+    api, _ = client(response)
+
+    with pytest.raises(BeatbotConnectionError, match=message):
+        await api._request("GET", "/test")
+
+
 async def test_async_access_token_provider():
     async def access_token():
         return "rotated-token"
@@ -298,3 +320,21 @@ async def test_actions():
         {"interfaceInfo": "select.work_mode", "label": "quick"},
         {"interfaceInfo": "switch.child_lock", "label": "on"},
     ]
+
+
+async def test_discovery_runtime_values_remain_unknown_until_reported() -> None:
+    """Distinguish missing state from explicitly reported zero values."""
+    api, _ = client(envelope({"devices": [{"deviceId": "d1", "isOnline": True}]}))
+    (device,) = await api.get_devices()
+    assert device.work_status is None
+    assert device.battery_level is None
+    assert device.error_code is None
+
+    device.apply_state({"vacuum.state": 2})
+    assert device.work_status == 2
+    assert device.battery_level is None
+    assert device.error_code is None
+
+    device.apply_state({"vacuum.battery": 0, "sensor.error": 0})
+    assert device.battery_level == 0
+    assert device.error_code == 0