Increase coverage for tests that did not set minimum threshold#258
Open
robander wants to merge 5 commits into
Open
Increase coverage for tests that did not set minimum threshold#258robander wants to merge 5 commits into
robander wants to merge 5 commits into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
No code change; this only adds tests that verify existing MCP server code.
These four servers were picked out by Codex as not setting up any minimum bar for passing tests, along with the following comments that caused concern:
Coverage levels by lines-of-code were previously at:
This PR adds codex-generated test cases to each of the four to get over 90%, with a focus on useful or important tests.
PR updated to also reach the 90% coverage bar for
oci-pricing-mcp-server, which did not have a minimum set and reported only 63% coverage. With this update, all of thesrc/oci-*servers pass tests with minimum 90% coverage.Fixes # (issue)
Type of change
New tests, no functional change
How Has This Been Tested?
Adds new tests, those tests and old tests continue to run
Checklist: