mirror of
https://github.com/fusion44/blitz_api.git
synced 2026-08-13 11:52:45 +02:00
b8677cbbf1
On VM installs (e.g. Proxmox) the RaspiBlitz monitor scripts don't
populate keys like system_cpu_load / system_ram_mb / system_temp_celsius,
so redis returns empty strings. get_hardware_info did int('')/float('')
and raised ValueError, which killed the _handle_gather_hardware_info
background task ('Task exception was never retrieved') - hardware SSE
updates then stopped until restart.
- parse the hardware Redis values via _safe_int/_safe_float, returning
zero values instead of crashing (and guard the vram-percent division
by zero)
- wrap the gatherer loop body in try/except so a single failure can no
longer kill the task
Fixes #271
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
33 lines
1.2 KiB
Python
33 lines
1.2 KiB
Python
"""
|
|
Regression test for blitz_api#271.
|
|
|
|
On VM setups (e.g. Proxmox) the RaspiBlitz monitor scripts don't populate
|
|
the hardware Redis keys, so system_cpu_load / system_ram_mb / etc. are empty
|
|
strings. get_hardware_info did int('')/float('') and raised ValueError,
|
|
which killed the _handle_gather_hardware_info background task. It should
|
|
return zero values instead of crashing.
|
|
"""
|
|
|
|
from app.system.impl import raspiblitz as rb
|
|
|
|
|
|
async def test_get_hardware_info_tolerates_empty_redis_values(monkeypatch):
|
|
monkeypatch.setattr(
|
|
rb.RaspiBlitzSystem, "_check_shell_scripts_status", lambda self: None
|
|
)
|
|
system = rb.RaspiBlitzSystem()
|
|
|
|
async def fake_redis_get(key):
|
|
return "" # every hardware key empty (VM scenario)
|
|
|
|
monkeypatch.setattr(rb, "redis_get", fake_redis_get)
|
|
|
|
info = await system.get_hardware_info() # must not raise
|
|
|
|
assert info["cpu_overall_percent"] == 0
|
|
assert info["cpu_per_cpu_percent"] == []
|
|
assert info["vram_total_bytes"] == 0
|
|
assert info["vram_available_bytes"] == 0
|
|
assert info["vram_used_bytes"] == 0
|
|
assert info["vram_usage_percent"] == 0
|
|
assert info["temperatures_celsius"]["system_temp"] == 0
|