hermes - 💡(How to fix) Fix hermes_cli/cron.py background ticker lacks dedicated unit tests [1 pull requests]

Problem

hermes_cli/cron.py implements a background cron ticker (_start_cron_ticker) that schedules periodic tasks in the gateway. It is production code but has no dedicated unit tests covering job scheduling, tick interval handling, stop events, or adapter integration.

Expected Outcome

The cron ticker module has unit tests covering: job registration, tick firing at correct intervals, graceful shutdown via stop_event, and error handling when an adapter is unavailable.

Suggested Approach

Create tests/hermes_cli/test_cron.py:

• test_cron_ticker_fires_at_interval — mock time, verify job runs after interval
• test_cron_ticker_stop_event — stop_event set, ticker exits cleanly
• test_cron_ticker_with_adapter — adapter is called on each tick
• test_cron_ticker_handles_adapter_error — adapter raises, ticker continues

Suggested Files

• hermes_cli/cron.py (source)
• tests/hermes_cli/test_cron.py (new test file)