Skip to content

Improve test and documentation coverage#3

Merged
samuel-williams-shopify merged 1 commit into
mainfrom
improve-coverage
Jun 17, 2026
Merged

Improve test and documentation coverage#3
samuel-williams-shopify merged 1 commit into
mainfrom
improve-coverage

Conversation

@samuel-williams-shopify

Copy link
Copy Markdown
Contributor

Summary

  • Add focused coverage for monitor trace-point dispatch and simple guard violations.
  • Split composite attr_reader declarations into individually documented attrs.
  • Keep coverage validation dependent on the combined Ruby matrix so the Ruby 3.2 ObjectSpace::WeakMap fallback is covered by the Ruby 3.2 shard.

Verification

  • bundle exec bake agent:context:install
  • bundle exec rubocop
  • bundle exec bake decode:index:coverage lib
  • bundle exec bake test
  • COVERAGE=PartialSummary bundle exec bake test

Note: the local Ruby-current coverage run reports only the Ruby 3.2 WeakMap fallback as uncovered. CI combines Ruby 3.2 and current coverage artifacts to validate both branches.

@samuel-williams-shopify samuel-williams-shopify force-pushed the improve-coverage branch 2 times, most recently from 44547fc to 610f2a5 Compare June 17, 2026 21:14
Assisted-By: devx/db2ad3a9-142f-42b2-8be1-fdfe9e209de2
@samuel-williams-shopify samuel-williams-shopify merged commit a63790f into main Jun 17, 2026
40 of 44 checks passed
@samuel-williams-shopify samuel-williams-shopify deleted the improve-coverage branch June 17, 2026 21:20
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