brianegge / garbage_bin / 23641105129

71%

Build Type

push

github

Committed by web-flow

Commit Message

Auto-republish discovery configs when Home Assistant restarts (#272)

* Auto-republish discovery configs when Home Assistant restarts

Extract discovery config publishing into publish_discovery() and subscribe
to homeassistant/status. When HA sends "online", all entity discovery
configs are re-published so HA picks them up without restarting the container.

https://claude.ai/code/session_019WowacijwJhgd6JfgCZB3g

* Use userdata callback arg instead of private _userdata attribute

Address review feedback: use the userdata parameter passed by paho to
on_message instead of accessing client._userdata. Also remove brittle
exact call_count assertion in test.

https://claude.ai/code/session_019WowacijwJhgd6JfgCZB3g

* Increase test coverage for publish_discovery and on_message

Add payload validation test, empty devices test, end-to-end on_message
test, and retain flag assertions to improve patch coverage.

https://claude.ai/code/session_019WowacijwJhgd6JfgCZB3g

* Add main() setup test to cover MQTT wiring and discovery call

The git diff reordered get_section and main(), causing codecov to see
all main() lines as new additions needing coverage. This test mocks
heavy deps (YOLO, sdnotify, paho) and verifies the setup: userdata
wiring, homeassistant/status subscription, and publish_discovery call.

https://claude.ai/code/session_019WowacijwJhgd6JfgCZB3g

---------

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

Coverage Stats

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

28 existing lines in 1 file now uncovered.

522 of 738 relevant lines covered (70.73%)

0.71 hits per line