Skip to content

Add targeted SDK boundary test coverage#64

Merged
KeeganEad merged 1 commit into
mainfrom
keegan/improve-test-coverage
May 21, 2026
Merged

Add targeted SDK boundary test coverage#64
KeeganEad merged 1 commit into
mainfrom
keegan/improve-test-coverage

Conversation

@KeeganEad
Copy link
Copy Markdown
Contributor

Summary

  • Adds controlled Rare API integration tests for client-bound request shaping, metadata pinning, import owner precedence, and media upload handshakes.
  • Adds package export smoke coverage for built @rareprotocol/rare-cli/{client,contracts,utils} subpaths plus a typecheck fixture for public exports.
  • Adds focused unit coverage for Uniswap Trade API request/response boundaries, Liquid Edition deploy receipt edge cases, and shared SDK validation helpers.

Validation

  • npm run lint
  • npm run test:typecheck
  • npx vitest run test/unit/swap/uniswap-api.test.ts test/unit/sdk/validation-core.test.ts test/unit/sdk/liquid.test.ts test/integration/sdk/package-exports.test.ts test/integration/sdk/client-api-shaping.test.ts test/unit/commands/swap.test.ts --config vitest.config.ts
  • npx vitest run test/e2e/cli.test.ts --config vitest.config.ts
  • npm test

@KeeganEad KeeganEad changed the title [codex] Add targeted SDK boundary test coverage Add targeted SDK boundary test coverage May 21, 2026
@KeeganEad KeeganEad marked this pull request as ready for review May 21, 2026 22:48
@KeeganEad KeeganEad merged commit d83e906 into main May 21, 2026
@KeeganEad KeeganEad deleted the keegan/improve-test-coverage branch May 21, 2026 22:49
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