Skip to content

Achieve complete test coverage - #8

Merged
samuel-williams-shopify merged 1 commit into
mainfrom
coverage/complete-test-coverage
Aug 17, 2026
Merged

Achieve complete test coverage#8
samuel-williams-shopify merged 1 commit into
mainfrom
coverage/complete-test-coverage

Conversation

@samuel-williams-shopify

Copy link
Copy Markdown
Contributor

Summary

  • add coverage for client construction, inheritance, representations, and invalid streaming modes
  • cover dispatcher service-name mismatch and missing-handler responses
  • make blockless bidirectional calls close their request stream and preserve the returned response stream
  • reach 100% executable line coverage

Validation

  • COVERAGE=BriefSummary,PartialSummary bundle exec sus test (256/256 lines, 100%)
  • bundle exec bake test
  • bundle exec rubocop
  • bundle exec bake decode:index:coverage lib
  • git diff --check

@samuel-williams-shopify
samuel-williams-shopify merged commit e2ff298 into main Aug 17, 2026
36 of 40 checks passed
@samuel-williams-shopify
samuel-williams-shopify deleted the coverage/complete-test-coverage branch August 17, 2026 00:22
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