Skip to content

Fixed CLI tests that depended on specific version of server-everything#1010

Closed
BobDickinson wants to merge 1 commit into
modelcontextprotocol:mainfrom
BobDickinson:fix-cli-tests-update-server-names
Closed

Fixed CLI tests that depended on specific version of server-everything#1010
BobDickinson wants to merge 1 commit into
modelcontextprotocol:mainfrom
BobDickinson:fix-cli-tests-update-server-names

Conversation

@BobDickinson

Copy link
Copy Markdown
Contributor

Summary

CLI tests depended on specific functionality of server-everything, which has changed, breaking a number of tests.

This PR:

  • Pins server-everything (in CI and tests)
  • Fixes all tests to work with current pinned version
  • Fixes a problem with undetected failures (isError: true payloads).

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Documentation update
  • Refactoring (no functional changes)
  • Test updates
  • Build/CI improvements

Testing

  • Tested in UI mode
  • Tested in CLI mode
  • Tested with STDIO transport
  • Tested with SSE transport
  • Tested with Streamable HTTP transport
  • Added/updated automated tests
  • Manual testing performed

Test Results and/or Instructions

All tests now pass

Checklist

  • Code follows the style guidelines (ran npm run prettier-fix)
  • Self-review completed
  • Code is commented where necessary
  • Documentation updated (README, comments, etc.)

Breaking Changes

No

… version, fixed problem with undetected failures (isError: true payloads).
@BobDickinson

Copy link
Copy Markdown
Contributor Author

If/when #1012 is accepted/merged, we can close this one.

@olaservo

Copy link
Copy Markdown
Member

Closing since the other refactoring PR was merged.

@olaservo olaservo closed this Jan 19, 2026
@BobDickinson BobDickinson deleted the fix-cli-tests-update-server-names branch March 7, 2026 21:57
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.

2 participants