Skip to content

Commit d5154b4

Browse files
KJonlineclaude
andcommitted
fix: wrap updateInterval new_interval in timedelta(seconds=) to match SessionConfig type
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 2461256 commit d5154b4

2 files changed

Lines changed: 6 additions & 3 deletions

File tree

src/helper/compat_aliases.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77

88
from __future__ import annotations
99

10+
from datetime import timedelta
1011
from typing import Any
1112

1213
from .hivedataclasses import Device
@@ -140,5 +141,5 @@ async def updateData(self, device: Device): # pylint: disable=invalid-name
140141

141142
async def updateInterval(self, new_interval: int): # pylint: disable=invalid-name
142143
"""Backwards-compatible alias for Home Assistant Scan Interval."""
143-
self.config.scan_interval = new_interval # type: ignore[attr-defined]
144+
self.config.scan_interval = timedelta(seconds=new_interval) # type: ignore[attr-defined]
144145
return True

tests/unit/test_compat_aliases.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -362,10 +362,12 @@ class TestSessionCompatMixinUpdateInterval:
362362
"""updateInterval must actually update config.scan_interval."""
363363

364364
async def test_update_interval_sets_scan_interval(self):
365-
"""updateInterval(300) must set self.config.scan_interval = 300."""
365+
"""updateInterval(300) must set self.config.scan_interval to timedelta(seconds=300)."""
366+
from datetime import timedelta
367+
366368
session = _make_concrete_session()
367369
await session.updateInterval(300)
368-
assert session.config.scan_interval == 300
370+
assert session.config.scan_interval == timedelta(seconds=300)
369371

370372
async def test_update_interval_returns_true(self):
371373
"""updateInterval must return True on success."""

0 commit comments

Comments
 (0)