Summary
The runner.py refactoring (Issue #58) extracted several services that currently lack dedicated unit tests:
AIService - threshold adjustment, veto cooldown logicSignalService - signal history storageCramerService - largest service at ~800 lines
Context
From PR #292 review (bot review, 2025-12-23)
The existing integration tests in test_trading_daemon.py cover end-to-end flows, but dedicated unit tests would improve coverage and make it easier to test edge cases.
Priority
Low - existing integration tests provide coverage
Summary
The runner.py refactoring (Issue #58) extracted several services that currently lack dedicated unit tests:
AIService- threshold adjustment, veto cooldown logicSignalService- signal history storageCramerService- largest service at ~800 linesContext
From PR #292 review (bot review, 2025-12-23)
The existing integration tests in
test_trading_daemon.pycover end-to-end flows, but dedicated unit tests would improve coverage and make it easier to test edge cases.Priority
Low - existing integration tests provide coverage