Add e2e tests for remote MCP servers #2444
Merged
+368
−0
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
This commit adds comprehensive end-to-end tests for remote MCP server functionality, verifying that `thv run` works correctly with remote servers as documented in the user guide. The tests cover: - Starting remote servers from the registry (mcp-spec) - Starting remote servers with explicit URLs - Verifying the `remote` flag is set correctly in workload metadata - Testing MCP protocol operations (initialize, ping, list tools) - Calling tools on remote servers (SearchModelContextProtocol) - Server lifecycle management (stop, restart, logs) All tests use JSON output parsing for robust validation and follow existing e2e test patterns. Tests are labeled with "mcp" to run in the appropriate CI worker group. Tests verify: - No containers are created for remote servers - Package field shows "remote" - Tool type is "remote" - Remote boolean flag is true - Transport defaults to streamable-http - Full MCP protocol compliance 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2444 +/- ##
==========================================
+ Coverage 55.34% 55.36% +0.01%
==========================================
Files 277 277
Lines 26451 26451
==========================================
+ Hits 14639 14644 +5
+ Misses 10467 10457 -10
- Partials 1345 1350 +5 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
eleftherias
approved these changes
Nov 4, 2025
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.
What
Adds end-to-end tests for remote MCP server functionality to verify
thv runworks correctly with remote servers.Why
We document how to run remote MCP servers in our user guide, but we had no automated tests to verify this functionality actually works. This PR adds comprehensive e2e tests using the
mcp-specremote server from the registry.Changes
test/e2e/remote_mcp_server_test.goTesting
All 9 tests pass locally:
Tests are labeled with
"mcp"and will run in thee2e-tests-core (mcp)CI job.