Skip to content

Enforce 95% Python SDK coverage in CI#52

Merged
negillett merged 1 commit into
mainfrom
enforce-python-sdk-coverage-95
May 19, 2026
Merged

Enforce 95% Python SDK coverage in CI#52
negillett merged 1 commit into
mainfrom
enforce-python-sdk-coverage-95

Conversation

@negillett
Copy link
Copy Markdown
Member

Summary

  • Gate IntentProof CI: Build and Test at 95% line coverage on
    src/intentproof/ via pytest-cov and scripts/check-coverage.sh.
  • Add end-to-end tests for the local loop (configure, wrap, outbox,
    HTTP export) plus targeted coverage for outbox transactions, async
    wrap, keypair edge cases, and signing verification.

Test plan

  • pip install -e ".[dev]" && pytest -q (98.8% line coverage)
  • bash ./scripts/check-coverage.sh 95
  • CI IntentProof CI: Build and Test green on this PR

@cursor
Copy link
Copy Markdown

cursor Bot commented May 19, 2026

PR Summary

Low Risk
Low risk: changes are limited to CI/test tooling and add test coverage; main impact is stricter CI gating that may fail builds if coverage drops.

Overview
CI now enforces 95% line coverage by adding pytest-cov defaults in pyproject.toml, ignoring coverage artifacts in .gitignore, and updating the GitHub Actions workflow to run scripts/check-coverage.sh after tests.

Test coverage is significantly expanded with new end-to-end local loop tests (configure → wrap → outbox → HTTP export) plus targeted edge/branch tests for async wrapping, canonicalization number/type handling, outbox transaction/chain-state behavior, exporter HTTP response handling, keypair load/write race cases, and signature verification failure modes.

Reviewed by Cursor Bugbot for commit 2b6b84e. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 1a4fa1d. Configure here.

Comment thread tests/test_keys_edges.py Outdated
@negillett negillett force-pushed the enforce-python-sdk-coverage-95 branch from 1a4fa1d to 91d7b95 Compare May 19, 2026 05:14
Add pytest-cov and a check-coverage script to the build workflow.
Expand E2E and integration tests for wrap, outbox, export, and
async paths so line coverage reaches the threshold.

Signed-off-by: Nathan Gillett <nathan@intentproof.io>
@negillett negillett force-pushed the enforce-python-sdk-coverage-95 branch from 91d7b95 to 2b6b84e Compare May 19, 2026 05:15
@negillett negillett merged commit 8c75559 into main May 19, 2026
3 checks passed
@negillett negillett deleted the enforce-python-sdk-coverage-95 branch May 19, 2026 11:39
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