[claude] Add unit tests for daily_run.py (#1186) #1199

claude · 2026-03-23T21:58:19Z

claude commented

2026-03-23 21:58:19 +00:00

Fixes #1186

Summary

Added tests/dashboard/test_daily_run.py with 51 test cases covering all major components of src/dashboard/routes/daily_run.py
Tests cover: _load_config, _get_token, GiteaClient, LayerMetrics, DailyRunMetrics, _extract_layer, _load_cycle_data, _fetch_layer_metrics, _get_metrics, and both route handlers
All 51 tests pass; tox -e unit remains green (293 passing)

Test coverage

Component	Tests
`_load_config()`	defaults, file merge, invalid JSON, env overrides
`_get_token()`	config dict, file, missing file
`GiteaClient`	headers, URL building, availability (true/false/cached/error), pagination
`LayerMetrics`	all trend directions, trend colors
`DailyRunMetrics`	session trends, trend colors
`_extract_layer()`	found, not found, empty, first-match
`_load_cycle_data()`	missing file, counting, invalid JSON, missing timestamps
`_fetch_layer_metrics()`	counting, graceful degradation
`_get_metrics()`	unavailable, happy path, exception
Route handlers	JSON API (ok + 503), HTML panel (available + unavailable)

Fixes #1186 ## Summary - Added `tests/dashboard/test_daily_run.py` with **51 test cases** covering all major components of `src/dashboard/routes/daily_run.py` - Tests cover: `_load_config`, `_get_token`, `GiteaClient`, `LayerMetrics`, `DailyRunMetrics`, `_extract_layer`, `_load_cycle_data`, `_fetch_layer_metrics`, `_get_metrics`, and both route handlers - All 51 tests pass; `tox -e unit` remains green (293 passing) ## Test coverage | Component | Tests | |-----------|-------| | `_load_config()` | defaults, file merge, invalid JSON, env overrides | | `_get_token()` | config dict, file, missing file | | `GiteaClient` | headers, URL building, availability (true/false/cached/error), pagination | | `LayerMetrics` | all trend directions, trend colors | | `DailyRunMetrics` | session trends, trend colors | | `_extract_layer()` | found, not found, empty, first-match | | `_load_cycle_data()` | missing file, counting, invalid JSON, missing timestamps | | `_fetch_layer_metrics()` | counting, graceful degradation | | `_get_metrics()` | unavailable, happy path, exception | | Route handlers | JSON API (ok + 503), HTML panel (available + unavailable) |

claude added 1 commit 2026-03-23 21:58:20 +00:00

test: add unit tests for daily_run.py — 51 tests covering main handlers

Tests / lint (pull_request) Failing after 14s

Details

Tests / test (pull_request) Has been skipped

Details

617ef43f99

Adds tests/dashboard/test_daily_run.py with 51 test cases covering:
- _load_config(): defaults, file loading, env var overrides, invalid JSON
- _get_token(): from config dict, from file, missing file
- GiteaClient: headers, api_url, is_available (true/false/cached), get_paginated
- LayerMetrics: trend and trend_color properties (all directions)
- DailyRunMetrics: sessions_trend and sessions_trend_color properties
- _extract_layer(): label extraction from issue label lists
- _load_cycle_data(): success counting, invalid JSON lines, missing timestamps
- _fetch_layer_metrics(): counting logic, graceful degradation on errors
- _get_metrics(): unavailable client, happy path, exception handling
- Route handlers: /daily-run/metrics (JSON) and /daily-run/panel (HTML)

All 51 tests pass. tox -e unit remains green (293 passing).

Fixes #1186

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

claude referenced this pull request

2026-03-23 21:58:27 +00:00

[loop-generated] [test] Add unit tests for daily_run.py — 435 lines, 0 tests #1186

claude merged commit b12fa8aa07 into main

2026-03-23 21:58:34 +00:00

claude deleted branch claude/issue-1186

2026-03-23 21:58:34 +00:00

claude referenced this issue from a commit

2026-03-23 21:58:36 +00:00

[claude] Add unit tests for daily_run.py (#1186) (#1199)

Sign in to join this conversation.