test: harden coverage to 100% and add CI coverage reporting #20

Merged
scheerenkevin merged 1 commit from feature/test-coverage-hardening into develop 2026-08-03 04:44:40 +00:00
scheerenkevin commented 2026-08-03 04:43:54 +00:00 (Migrated from github.com)

Summary

  • hatch run test-cov (pytest-cov, term-missing + XML) with [tool.coverage] config, fail_under = 90
  • CI now runs test-cov and uploads coverage.xml as a build artifact
  • Closed all coverage gaps found (96% -> 100%, statements and branches): exception __repr__, actual backoff time.sleep() calls, transport-error retry paths, retryable-status-exhausted paths, invalid-JSON-response paths, externally-provided-client-not-closed parity test for GasManagerAdminClient

Verified locally: hatch run lint, hatch run format-check, hatch run test-cov (97 passed, 100% coverage), hatch build.

Closes #10

Note: merges into develop, not the default branch main — issue closed manually after merge per repo process.

## Summary - `hatch run test-cov` (pytest-cov, term-missing + XML) with `[tool.coverage]` config, `fail_under = 90` - CI now runs `test-cov` and uploads `coverage.xml` as a build artifact - Closed all coverage gaps found (96% -> 100%, statements and branches): exception `__repr__`, actual backoff `time.sleep()` calls, transport-error retry paths, retryable-status-exhausted paths, invalid-JSON-response paths, externally-provided-client-not-closed parity test for `GasManagerAdminClient` Verified locally: `hatch run lint`, `hatch run format-check`, `hatch run test-cov` (97 passed, 100% coverage), `hatch build`. ## Related Closes #10 Note: merges into `develop`, not the default branch `main` — issue closed manually after merge per repo process.
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
kevin/alchemy-wallet-client!20
No description provided.