modelcontextprotocol/swift-sdk
Watch
1
0
Fork
You've already forked swift-sdk
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Make CallTool.Result.isError optional (#11)

Browse source
This commit is contained in:
Mattt 2025-03-14 06:24:46 -07:00 • committed by GitHub
commit a62e51a61c
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 23 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

6
Tests/MCPTests/RoundtripTests.swift
View file

@ -52,13 +52,13 @@ struct RoundtripTests {
}
guard let a = request.arguments?["a"]?.intValue,
let b = request.arguments?["b"]?.intValue
let b = request.arguments?["b"]?.intValue
else {
return CallTool.Result(
content: [.text("Did not receive valid arguments")], isError: true)
}
return CallTool.Result(content: [.text("\(a + b)")], isError: false)
return CallTool.Result(content: [.text("\(a + b)")])
}
let client = Client(name: "TestClient", version: "1.0")
@ -96,7 +96,7 @@ struct RoundtripTests {
let callToolTask = Task {
let result = try await client.callTool(name: "add", arguments: ["a": 1, "b": 2])
#expect(result.isError == false)
#expect(result.isError == nil)
#expect(result.content == [.text("3")])
}

20
Tests/MCPTests/ToolTests.swift
View file

@ -139,16 +139,16 @@ struct ToolTests {
#expect(params.arguments?["param2"] == .int(42))
}
@Test("CallTool result validation")
@Test("CallTool success result validation")
func testCallToolResult() throws {
let content = [
Tool.Content.text("Result 1"),
Tool.Content.text("Result 2"),
]
let result = CallTool.Result(content: content, isError: false)
let result = CallTool.Result(content: content)
#expect(result.content.count == 2)
#expect(result.isError == false)
#expect(result.isError == nil)
if case .text(let text) = result.content[0] {
#expect(text == "Result 1")
@ -157,6 +157,20 @@ struct ToolTests {
}
}
@Test("CallTool error result validation")
func testCallToolErrorResult() throws {
let errorContent = [Tool.Content.text("Error message")]
let errorResult = CallTool.Result(content: errorContent, isError: true)
#expect(errorResult.content.count == 1)
#expect(errorResult.isError == true)
if case .text(let text) = errorResult.content[0] {
#expect(text == "Error message")
} else {
#expect(Bool(false), "Expected error text content")
}
}
@Test("ToolListChanged notification name validation")
func testToolListChangedNotification() throws {
#expect(ToolListChangedNotification.name == "notifications/tools/list_changed")