[claude] Add unit tests for health.py (#945) #1002

claude · 2026-03-22T22:57:50Z

claude commented

2026-03-22 22:57:50 +00:00

Fixes #945

Summary

Adds 41 unit tests for src/dashboard/routes/health.py
Covers all 5 endpoints: /health, /health/status, /health/sovereignty, /health/components, /health/snapshot
Tests helper functions: _calculate_overall_score, _generate_recommendations
Tests subsystem probes: Ollama (sync + async with TTL cache), SQLite, Lightning
Tests graceful degradation when Ollama is down or snapshot module fails
All 41 tests pass; no pre-existing tests broken

Fixes #945 ## Summary - Adds 41 unit tests for `src/dashboard/routes/health.py` - Covers all 5 endpoints: `/health`, `/health/status`, `/health/sovereignty`, `/health/components`, `/health/snapshot` - Tests helper functions: `_calculate_overall_score`, `_generate_recommendations` - Tests subsystem probes: Ollama (sync + async with TTL cache), SQLite, Lightning - Tests graceful degradation when Ollama is down or snapshot module fails - All 41 tests pass; no pre-existing tests broken

claude added 1 commit 2026-03-22 22:57:50 +00:00

test: add unit tests for health.py route module

Tests / lint (pull_request) Failing after 3s

Details

Tests / test (pull_request) Has been skipped

Details

ed1270969a

Adds 41 unit tests covering:
- Pydantic models (DependencyStatus, SovereigntyReport, HealthStatus)
- Helper functions (_calculate_overall_score, _generate_recommendations)
- Ollama sync probe (_check_ollama_sync) with mock HTTP responses
- Async Ollama check with TTL cache behavior (hit, miss, expiry)
- Legacy bool wrapper (check_ollama)
- SQLite and Lightning dependency checks
- All five endpoints (/health, /health/status, /health/sovereignty,
  /health/components, /health/snapshot)
- Graceful degradation when Ollama is down or snapshot module fails

Fixes #945

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

claude referenced this pull request

2026-03-22 22:57:56 +00:00

LHF: Write unit tests for health.py #945

Timmy merged commit b5a65b9d10 into main

2026-03-23 15:10:54 +00:00

Timmy deleted branch claude/issue-945

2026-03-23 15:10:55 +00:00

Timmy referenced this issue from a commit

2026-03-23 15:10:56 +00:00

[claude] Add unit tests for health.py (#945) (#1002)

Sign in to join this conversation.