brianegge / garbage_bin / 22100082806

57%

Build Type

push

github

Committed by web-flow

Commit Message

Add tests for save and sync_local_to_remote (#243)

* Add tests for save and sync_local_to_remote

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Fix crash on MQTT broker restart by allowing auto-reconnect

The on_disconnect callback was setting RUNNING=False, which killed the
entire process whenever the MQTT broker dropped connections (e.g. during
HA restarts). Now paho's built-in auto-reconnect handles broker outages
and the shutdown publish is wrapped in try/except for robustness.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Use tmp_path with chmod instead of hardcoded paths in save tests

Replace /nonexistent/... paths with tmp_path directories made
read-only via chmod(0o555) for deterministic cross-environment
test behavior. Extract repeated date directory string into a
DATE_DIR constant.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Add tests for graceful_shutdown to improve patch coverage

Extract shutdown logic into a testable graceful_shutdown function and
add tests for both the happy path and MQTT publish failure during
shutdown.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Run Details

10 of 12 new or added lines in 1 file covered. (83.33%)

19 existing lines in 1 file now uncovered.

342 of 514 relevant lines covered (66.54%)

0.67 hits per line