Skip to content

Test the perception pipeline orchestration; raise coverage floor to 60% - #16

Open
mandarwagh9 wants to merge 1 commit into
mainfrom
chore/pipeline-service-tests
Open

Test the perception pipeline orchestration; raise coverage floor to 60%#16
mandarwagh9 wants to merge 1 commit into
mainfrom
chore/pipeline-service-tests

Conversation

@mandarwagh9

Copy link
Copy Markdown
Owner

What & why

Builds on the now-clean main. The perception orchestration layer (services.py — the detect→track→fuse→snapshot→broadcast loop that the whole system runs on) was the largest untested core module (~34%). This drives it end-to-end with mock adapters.

Changes

  • test_pipeline_service.py:
    • a tick builds a PerceptionSnapshot (detections/tracks/world-objects/frame) and broadcasts it
    • no-frames is a clean no-op (no snapshot, no broadcast)
    • generation increments per tick
    • PipelineMetrics running average
    • start/stop lifecycle
    • HealthCheckService status reporting
  • CI: raised the backend coverage floor 50% → 60%.

Verification

  • ruff clean · mypy clean (18 files) · 117 backend tests pass (was 111).
  • services.py coverage 34% → 89%; total 60% → 63%.

🤖 Generated with Claude Code

…floor to 60%

Adds the first test driving PerceptionPipelineService end-to-end (detect -> track
-> fuse -> snapshot -> broadcast) with mock adapters, plus metrics, lifecycle, and
health-status coverage. services.py was the largest untested core module.

- test_pipeline_service.py: a tick builds + broadcasts a snapshot; no-frames is a
  no-op; generation increments per tick; PipelineMetrics running average;
  start/stop lifecycle; HealthCheckService status reporting.
- CI: raise the backend coverage floor 50% -> 60%.

services.py coverage 34%->89%, total 60%->63%. ruff/mypy clean, 117 tests pass.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant