pySmartHashtag 0.12.2__tar.gz → 0.12.3__tar.gz

{pysmarthashtag-0.12.2/pySmartHashtag.egg-info → pysmarthashtag-0.12.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pySmartHashtag
-Version: 0.12.2
+Version: 0.12.3
 Summary: A python library to get information from Smart #1, #3 and #5 web API
 Home-page: https://github.com/dasBasti/pySmartHashtag
 Author: dasBasti

{pysmarthashtag-0.12.2 → pysmarthashtag-0.12.3/pySmartHashtag.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pySmartHashtag
-Version: 0.12.2
+Version: 0.12.3
 Summary: A python library to get information from Smart #1, #3 and #5 web API
 Home-page: https://github.com/dasBasti/pySmartHashtag
 Author: dasBasti

{pysmarthashtag-0.12.2 → pysmarthashtag-0.12.3}/pySmartHashtag.egg-info/SOURCES.txt

@@ -56,6 +56,7 @@ pysmarthashtag/tests/test_journal_control.py
 pysmarthashtag/tests/test_journal_pagination.py
 pysmarthashtag/tests/test_log_sanitizer.py
 pysmarthashtag/tests/test_missing_fields.py
+pysmarthashtag/tests/test_ota_info.py
 pysmarthashtag/tests/test_ssl_context.py
 pysmarthashtag/tests/test_token_lifecycle.py
 pysmarthashtag/tests/test_tracked_vins.py

{pysmarthashtag-0.12.2 → pysmarthashtag-0.12.3}/pySmartHashtag.egg-info/scm_file_list.json

@@ -65,6 +65,7 @@
     "pysmarthashtag/tests/test_journal_pagination.py",
     "pysmarthashtag/tests/test_log_sanitizer.py",
     "pysmarthashtag/tests/test_missing_fields.py",
+    "pysmarthashtag/tests/test_ota_info.py",
     "pysmarthashtag/tests/test_ssl_context.py",
     "pysmarthashtag/tests/test_token_lifecycle.py",
     "pysmarthashtag/tests/test_tracked_vins.py",

pysmarthashtag-0.12.3/pySmartHashtag.egg-info/scm_version.json

@@ -0,0 +1,8 @@
+{
+  "tag": "0.12.3",
+  "distance": 0,
+  "node": "ge7f443b13dbb531926f2cb3f667e5c3e52ce96a9",
+  "dirty": false,
+  "branch": "HEAD",
+  "node_date": "2026-09-20"
+}

{pysmarthashtag-0.12.2 → pysmarthashtag-0.12.3}/pysmarthashtag/account.py

@@ -50,6 +50,13 @@ VALID_UNTIL_OFFSET = datetime.timedelta(seconds=10)
 # blip mid-loop doesn't fail the whole poll.
 _BENIGN_EMPTY_CODE = "8153"
 
+# Cloud's code on the OTA endpoint for vehicles it has no update record
+# for (reported on Smart #5). It is a property of the OTA service, not of
+# the session: refreshing the token does not clear it, so a poll loop that
+# treats it as an unmapped code refreshes the session on every cycle for
+# nothing. OTA data is optional, so callers normalise this to "no OTA info".
+_OTA_NO_DATA_CODE = "1003"
+
 _LOGGER = logging.getLogger(__name__)
 
 
@@ -931,8 +938,18 @@ class SmartAccount:
                     await self.select_active_vehicle(vin)
                     continue
                 except httpx.HTTPStatusError as exc:
-                    # Unmapped code: one refresh in case the session is stale, then surface.
                     code = _cloud_code(exc)
+                    if code == _OTA_NO_DATA_CODE:
+                        # The OTA service has nothing to report for this VIN.
+                        # Optional data, and no session remedy applies — return
+                        # empty rather than burning a refresh on every poll.
+                        _LOGGER.debug(
+                            "OTA endpoint returned cloud code %s for %s; no OTA info available",
+                            code,
+                            sanitize_log_data(vin),
+                        )
+                        return {}
+                    # Unmapped code: one refresh in case the session is stale, then surface.
                     if code is None or refreshed_unmapped:
                         raise
                     refreshed_unmapped = True

pysmarthashtag-0.12.3/pysmarthashtag/tests/test_ota_info.py

@@ -0,0 +1,73 @@
+"""Tests for OTA info fetching.
+
+The OTA endpoint is optional data, so its failures must never cost the
+session. In particular cloud code 1003 ("no OTA record for this VIN",
+observed on Smart #5) used to fall into the unmapped-code branch, which
+refreshed the authentication session on every single poll — 153 times in
+one day in the report — without ever clearing the 1003.
+"""
+
+import httpx
+import pytest
+import respx
+from httpx import Response
+
+from pysmarthashtag.account import SmartAccount
+from pysmarthashtag.const import OTA_SERVER_URL
+from pysmarthashtag.tests.conftest import prepare_account_with_vehicles
+
+_OTA_URL = OTA_SERVER_URL + "app/info/TestVIN0000000001"
+
+
+def _count_refreshes(account: SmartAccount) -> list[int]:
+    """Replace ``authentication.refresh`` with a counting no-op."""
+    calls: list[int] = []
+
+    async def fake_refresh() -> None:
+        calls.append(1)
+
+    account.config.authentication.refresh = fake_refresh
+    return calls
+
+
+@pytest.mark.asyncio
+async def test_ota_1003_returns_empty_without_refresh(smart_fixture: respx.Router) -> None:
+    """Code 1003 is normalised to "no OTA info" — no refresh, no retry."""
+    account = await prepare_account_with_vehicles()
+    refreshes = _count_refreshes(account)
+
+    route = smart_fixture.get(_OTA_URL).mock(
+        return_value=Response(200, json={"code": "1003", "message": "no ota info"})
+    )
+    # respx reuses the route registered by the fixture, so its counter already
+    # carries the OTA call made while setting the account up.
+    before = route.call_count
+
+    assert await account.get_vehicle_ota_info("TestVIN0000000001") == {}
+    assert refreshes == [], "1003 must not refresh the session"
+    assert route.call_count - before == 1, "1003 must not be retried"
+
+
+@pytest.mark.asyncio
+async def test_ota_1003_does_not_break_refresh_cycle(smart_fixture: respx.Router) -> None:
+    """A permanent 1003 still leaves every other vehicle sensor populated."""
+    smart_fixture.get(_OTA_URL).mock(return_value=Response(200, json={"code": "1003", "message": "no ota info"}))
+
+    account = await prepare_account_with_vehicles()
+
+    vehicle = account.vehicles["TestVIN0000000001"]
+    assert vehicle.vin == "TestVIN0000000001"
+    assert "ota" not in vehicle.data
+
+
+@pytest.mark.asyncio
+async def test_ota_unmapped_code_still_refreshes_once(smart_fixture: respx.Router) -> None:
+    """Regression guard: other unmapped codes keep the one-refresh remedy."""
+    account = await prepare_account_with_vehicles()
+    refreshes = _count_refreshes(account)
+
+    smart_fixture.get(_OTA_URL).mock(return_value=Response(200, json={"code": "9999", "message": "something new"}))
+
+    with pytest.raises(httpx.HTTPStatusError):
+        await account.get_vehicle_ota_info("TestVIN0000000001")
+    assert len(refreshes) == 1, "unmapped codes still get exactly one refresh"

pysmarthashtag-0.12.2/pySmartHashtag.egg-info/scm_version.json

@@ -1,8 +0,0 @@
-{
-  "tag": "0.12.2",
-  "distance": 0,
-  "node": "g800eaf2bed8475decf036d35ccdea3178897c327",
-  "dirty": false,
-  "branch": "HEAD",
-  "node_date": "2026-08-15"
-}