kdeyev / pyonwater / 24429629977

98%

Build Type

push

github

Committed by web-flow

Commit Message

test: expand coverage for export-range, client, models, and units (#58)

* fix: type annotations and public helpers in export-range code

Resolves type/lint issues in the export-range implementation added in PR #52:

- Add explicit dict[str, str | int] annotation to the params dict in
  read_historical_data_range_export() to resolve partially-unknown type
- cast(str, result['url']) to resolve unknown argument type for
  normalize_export_path()
- Rewrite flow_value narrowing to eliminate type: ignore[arg-type]
  suppression while keeping identical runtime behavior
- Rename _normalize_export_path, _parse_export_csv, _parse_export_datetime
  to drop the leading underscore — these are utility methods that tests
  legitimately call, and pylint/pyright both flag accessing _ members
  from outside the class

* test: expand coverage for export-range, csv parsing, and meter cache

Phase 1 (9 tests in test_meter_reader.py):
- Export range validation: invalid days_to_load, max_retries, poll_interval
- Export range error paths: bad JSON, missing task_id, missing URL
- Poll task: error state and timeout exhausted
- normalize_export_path: invalid URL scheme

Phase 2 (6 tests in test_meter_reader.py):
- parse_export_csv: empty string, space-separated column names, no Flow column
- read_historical_data_one_day: quoted-empty response, null response, missing timeseries key

Phase 3 (2 functions / 7 cases in test_meter_reader.py):
- AggregationLevel: parametrize over all 6 enum values, verify POST body aggregate param
- RequestUnits: verify units value forwarded to POST body

Phase 4 (3 tests in test_meter.py):
- Meter cache keeps newer data when older data arrives on second read
- Meter cache not updated when timestamp and count are identical
- include_today=False passes correct end-date to export endpoint

* test: broaden client, model, and units coverage

* test: fix remaining lint issues in tests

* test: align coverage branch with v0.3.32 auth behavior

* chore: bu... (continued)

Coverage Stats

433 of 0 new or added lines in 0 files covered. (NaN%)

1 existing line in 1 file now uncovered.

2145 of 2178 relevant lines covered (98.48%)

0.98 hits per line