maximunited / imou_life / 25343466441

90%

push

github

fix: add missing hysteresis and state persistence in battery coordinator

Found 3 actual code bugs while fixing integration tests:

1. Battery-based sleep mode was missing hysteresis logic
   - Sleep mode would exit immediately when battery recovered slightly
   - Added hysteresis: stay asleep until battery > threshold + 10%
   - Prevents rapid on/off cycling

2. set_power_mode() wasn't persisting state internally
   - Called device API but didn't update self._power_mode
   - State would be lost after setting

3. set_led_indicators() wasn't persisting state internally
   - Called device API but didn't update self._led_indicators
   - State would be lost after setting

Also fixed integration tests:
- test_battery_based_sleep_activation: Now passes with hysteresis
- test_power_mode_changes_propagate: Fixed to check async method
- test_led_indicators_toggle: Fixed to check async method
- test_battery_data_caching: Fixed to test coordinator caching, not method caching
- Added async_refresh() call before checking coordinator.data

Integration tests: 9 passing (was 5), 16 failing (was 20)

5 of 5 new or added lines in 1 file covered. (100.0%)

1452 of 1651 relevant lines covered (87.95%)

3.52 hits per line