Skip to content

Commit

Permalink
Adjust has_v2_api so test does not trigger 'Was never awaited' error
Browse files Browse the repository at this point in the history
  • Loading branch information
DCSBL committed Jan 5, 2025
1 parent 9767ff9 commit 72cb79a
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
8 changes: 4 additions & 4 deletions homewizard_energy/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,10 +27,10 @@ async def has_v2_api(host: str, websession: ClientSession | None = None) -> bool
# v2 api is https only and returns a 401 Unauthorized when no key provided,
# no connection can be made if the device is not v2
url = f"https://{host}/api"
async with websession.get(
url, ssl=False, raise_for_status=False, timeout=15
) as res:
return res.status == 401
res = await websession.get(url, ssl=False, raise_for_status=False, timeout=5)
res.close()

return res.status == 401
except Exception: # pylint: disable=broad-except
# all other status/exceptions means the device is not v2 or not reachable at this time
return False
Expand Down
2 changes: 1 addition & 1 deletion tests/test_helpers.py
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ async def test_has_v2_api_false(aresponses):
async def test_has_v2_api_exception():
"""Test if has_v2_api returns False when an exception occurs."""
session = AsyncMock()
session.request = AsyncMock(side_effect=asyncio.TimeoutError())
session.get = AsyncMock(side_effect=asyncio.TimeoutError)

result = await has_v2_api("example.com", session)
assert result is False
Expand Down

0 comments on commit 72cb79a

Please sign in to comment.