Skip to content

Commit

Permalink
test: Update test_call_tool to use localhost URL and assert server st…
Browse files Browse the repository at this point in the history
…atus response. This change improves test reliability by ensuring it checks the correct server response instead of an external URL.
  • Loading branch information
onuratakan committed Jan 7, 2025
1 parent 07b0834 commit 67fbb94
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 22 deletions.
20 changes: 0 additions & 20 deletions tests/server/test_mcp_client.py

This file was deleted.

4 changes: 2 additions & 2 deletions tests/server/test_tools_api.py
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ async def test_call_tool():
"args": ["mcp-server-fetch"],
"env": None,
"tool_name": "fetch",
"arguments": {"url": "https://google.com"},
"arguments": {"url": "http://localhost:8086/status"},
},
)
assert response.status_code == 200
Expand All @@ -107,4 +107,4 @@ async def test_call_tool():
assert content["type"] == "text"
assert "text" in content
assert isinstance(content["text"], str)
assert "Contents of https://google.com/:" in content["text"]
assert "Server is running" in content["text"]

0 comments on commit 67fbb94

Please sign in to comment.