Skip to content

[WIP] Update test coverage for tools package#83

Closed
ghost wants to merge 2 commits intoslim/sdk-refactorfrom
unknown repository
Closed

[WIP] Update test coverage for tools package#83
ghost wants to merge 2 commits intoslim/sdk-refactorfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Aug 1, 2025

What I did

  • Test coverage needs to be improved. CONTRIBUTING.md updated with Testing section - which contains commands for quickly understanding current coverage.
  • Increased coverage for tools package to: 58.9% from 36.0% (Note this is only for unit, integration coverage is higher)
  • Previously we had a single tools_test.go for the tools package. Switching to a model where we have separate test files for each like: call_test.go and enable_test.go
  • Note this PR is dependent on Switch to modelcontextprotocol/go-sdk #77 which is currently failing CI checks. Will continue to update with additional coverage.

Related issue

  • Currently unit-test coverage for tools is: 36.0% of statements. See screenshot below for file coverage breakdown:
image

(not mandatory) A picture of a cute animal, if possible in relation to what you did
🐞

@ghost
Copy link
Copy Markdown
Author

ghost commented Aug 5, 2025

Closing this - will re-open with additional coverage after the updated Go framework PR's are merged

@ghost ghost closed this Aug 5, 2025
This pull request was closed.
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