Add parametrized edge-case tests for extreme metric scenarios#274
Conversation
Campaign objective specifies testing both observer metrics (CollectorMetrics, SystemMetrics) AND tuning metrics (aggregate_family_metrics), but implementation only covers observer metrics. Missing: aggregate_family_metrics edge-case tests. File naming is misleading: test_tuning_metrics_extreme_scenarios.py actually contains observer metric tests, not tuning metric tests. Code quality is excellent (proper headers, docstrings, parametrization, all tests passing, ruff clean, type checking passes), but scope does not match spec. |
CI has not gone green after 20 checks (1 failing: audit: failure). Not merged (red CI) and not closed (work preserved) — needs a human to fix CI. |
Missing ## Overall Plan and ## Current Stage sections caused the custodian-audit CI check to fail on PR #274. Added both required sections; custodian-multi now reports 0 findings locally. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Auto-generated by Operations Center execution.
Goal
Add parametrized edge-case tests for extreme metric scenarios
Definition of done (complete ALL before finishing)
file the task implies (implementation, tests, and docs as applicable). Do
not leave TODOs, stubs, or 'follow-up' gaps; a partial change is rejected
in review.
pass locally. If anything fails, fix it before finishing — do not hand
off a red build.
green. The PR you open should be mergeable as-is.