Skip to content

Add spec-driven test suite and run it in CI#18

Merged
iDavi merged 1 commit into
mainfrom
claude/nifty-gates-i8ytcx
Jun 26, 2026
Merged

Add spec-driven test suite and run it in CI#18
iDavi merged 1 commit into
mainfrom
claude/nifty-gates-i8ytcx

Conversation

@iDavi

@iDavi iDavi commented Jun 26, 2026

Copy link
Copy Markdown
Owner

Implements the testing strategy from issue #1:

  • spec-driven tests for each core feature (crypto, capsule, draft, ics, config, errors) exercised through their public api
  • cli command tests assert the final printed output with the core mocked
  • 90% coverage gate (suite currently sits at ~99%)
  • a tests workflow runs pytest across python 3.10-3.13 on push and PR

Test config and the gate live in pyproject; pytest/pytest-cov are added to the dev extra. Tests bind to public contracts and the documented .mcap wire format, not to private helpers, so refactors stay free.

Closes #1

Claude-Session: https://claude.ai/code/session_01ARYv6iMTZ7UiZnbUL1LnJC

Implements the testing strategy from issue #1:

- spec-driven tests for each core feature (crypto, capsule, draft, ics,
  config, errors) exercised through their public api
- cli command tests assert the final printed output with the core mocked
- 90% coverage gate (suite currently sits at ~99%)
- a `tests` workflow runs pytest across python 3.10-3.13 on push and PR

Test config and the gate live in pyproject; pytest/pytest-cov are added to
the dev extra. Tests bind to public contracts and the documented .mcap wire
format, not to private helpers, so refactors stay free.

Closes #1

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01ARYv6iMTZ7UiZnbUL1LnJC
@iDavi iDavi merged commit 1bb6708 into main Jun 26, 2026
10 checks passed
@iDavi iDavi deleted the claude/nifty-gates-i8ytcx branch June 26, 2026 20:58
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.

Add automated tests

2 participants