Skip to content

test(cron): cover CLI helper functions - #35814

Closed
xuezhaolan wants to merge 1 commit into
NousResearch:mainfrom
xuezhaolan:lan/test-cron-cli-35727
Closed

test(cron): cover CLI helper functions#35814
xuezhaolan wants to merge 1 commit into
NousResearch:mainfrom
xuezhaolan:lan/test-cron-cli-35727

Conversation

@xuezhaolan

Copy link
Copy Markdown
Contributor

Summary

  • add direct unit coverage for cron_list, cron_status, cron_tick, and cron_create
  • assert gateway-running and gateway-missing output paths without shelling out
  • keep existing command lifecycle coverage intact while updating edit/create arg fixtures for current CLI fields

Test Plan

  • python -m pytest tests/hermes_cli/test_cron.py -q -o addopts=
  • git diff --check

Closes #35727

@alt-glitch alt-glitch added type/test Test coverage or test infrastructure P3 Low — cosmetic, nice to have comp/cron Cron scheduler and job management labels May 31, 2026
@teknium1

teknium1 commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

This coverage has landed on main via #59510 (commit 3ba5ba8) — your original commit was cherry-picked with co-author credit preserved. Thanks @xuezhaolan for the contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cron Cron scheduler and job management P3 Low — cosmetic, nice to have type/test Test coverage or test infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

hermes_cli/cron.py background ticker lacks dedicated unit tests

3 participants