Skip to content

Commit 8534220

Browse files
committed
fix: resolve compilation errors after rebase onto upstream main
1 parent 367a552 commit 8534220

File tree

4 files changed

+16
-14
lines changed

4 files changed

+16
-14
lines changed

mcp-core/src/test/java/io/modelcontextprotocol/experimental/tasks/InMemoryTaskStoreTests.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ void tearDown() {
5757
// ------------------------------------------
5858

5959
private CallToolResult createTestResult(String text) {
60-
return CallToolResult.builder().content(List.of(new TextContent(null, null, text))).isError(false).build();
60+
return CallToolResult.builder().content(List.of(new TextContent(null, text))).isError(false).build();
6161
}
6262

6363
/**

mcp-core/src/test/java/io/modelcontextprotocol/server/McpAsyncServerExchangeTests.java

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -704,7 +704,8 @@ void testPingMultipleCalls() {
704704

705705
@Test
706706
void testListTasksWithNullCapabilities() {
707-
McpAsyncServerExchange exchangeWithNullCapabilities = new McpAsyncServerExchange(mockSession, null, clientInfo);
707+
McpAsyncServerExchange exchangeWithNullCapabilities = new McpAsyncServerExchange("testSessionId", mockSession,
708+
null, clientInfo, McpTransportContext.EMPTY);
708709

709710
StepVerifier.create(exchangeWithNullCapabilities.listTasks("cursor")).verifyErrorSatisfies(error -> {
710711
assertThat(error).isInstanceOf(IllegalStateException.class)
@@ -720,8 +721,8 @@ void testListTasksWithoutTasksCapabilities() {
720721
.roots(true)
721722
.build();
722723

723-
McpAsyncServerExchange exchangeWithoutTasks = new McpAsyncServerExchange(mockSession, capabilitiesWithoutTasks,
724-
clientInfo);
724+
McpAsyncServerExchange exchangeWithoutTasks = new McpAsyncServerExchange("testSessionId", mockSession,
725+
capabilitiesWithoutTasks, clientInfo, McpTransportContext.EMPTY);
725726

726727
StepVerifier.create(exchangeWithoutTasks.listTasks("cursor")).verifyErrorSatisfies(error -> {
727728
assertThat(error).isInstanceOf(IllegalStateException.class)
@@ -737,8 +738,8 @@ void testListTasksWithoutListCapability() {
737738
.tasks(McpSchema.ClientCapabilities.ClientTaskCapabilities.builder().build())
738739
.build();
739740

740-
McpAsyncServerExchange exchangeWithoutList = new McpAsyncServerExchange(mockSession, capabilitiesWithoutList,
741-
clientInfo);
741+
McpAsyncServerExchange exchangeWithoutList = new McpAsyncServerExchange("testSessionId", mockSession,
742+
capabilitiesWithoutList, clientInfo, McpTransportContext.EMPTY);
742743

743744
StepVerifier.create(exchangeWithoutList.listTasks("cursor")).verifyErrorSatisfies(error -> {
744745
assertThat(error).isInstanceOf(IllegalStateException.class)
@@ -854,7 +855,8 @@ void testListTasksWithMultiplePages() {
854855

855856
@Test
856857
void testCancelTaskWithNullCapabilities() {
857-
McpAsyncServerExchange exchangeWithNullCapabilities = new McpAsyncServerExchange(mockSession, null, clientInfo);
858+
McpAsyncServerExchange exchangeWithNullCapabilities = new McpAsyncServerExchange("testSessionId", mockSession,
859+
null, clientInfo, McpTransportContext.EMPTY);
858860

859861
StepVerifier
860862
.create(exchangeWithNullCapabilities
@@ -873,8 +875,8 @@ void testCancelTaskWithoutTasksCapabilities() {
873875
.roots(true)
874876
.build();
875877

876-
McpAsyncServerExchange exchangeWithoutTasks = new McpAsyncServerExchange(mockSession, capabilitiesWithoutTasks,
877-
clientInfo);
878+
McpAsyncServerExchange exchangeWithoutTasks = new McpAsyncServerExchange("testSessionId", mockSession,
879+
capabilitiesWithoutTasks, clientInfo, McpTransportContext.EMPTY);
878880

879881
StepVerifier
880882
.create(exchangeWithoutTasks.cancelTask(McpSchema.CancelTaskRequest.builder().taskId("task-1").build()))
@@ -892,8 +894,8 @@ void testCancelTaskWithoutCancelCapability() {
892894
.tasks(McpSchema.ClientCapabilities.ClientTaskCapabilities.builder().build())
893895
.build();
894896

895-
McpAsyncServerExchange exchangeWithoutCancel = new McpAsyncServerExchange(mockSession,
896-
capabilitiesWithoutCancel, clientInfo);
897+
McpAsyncServerExchange exchangeWithoutCancel = new McpAsyncServerExchange("testSessionId", mockSession,
898+
capabilitiesWithoutCancel, clientInfo, McpTransportContext.EMPTY);
897899

898900
StepVerifier
899901
.create(exchangeWithoutCancel.cancelTask(McpSchema.CancelTaskRequest.builder().taskId("task-1").build()))

mcp-test/src/main/java/io/modelcontextprotocol/server/AbstractMcpAsyncServerTests.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -837,7 +837,7 @@ void builderShouldThrowWhenNormalToolAndTaskToolShareSameName() {
837837

838838
assertThatThrownBy(() -> {
839839
prepareAsyncServerBuilder()
840-
.tool(normalTool,
840+
.toolCall(normalTool,
841841
(exchange, args) -> Mono
842842
.just(CallToolResult.builder().content(List.of()).isError(false).build()))
843843
.taskTools(TaskAwareAsyncToolSpecification.builder()

mcp-test/src/main/java/io/modelcontextprotocol/server/AbstractMcpSyncServerTests.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -672,7 +672,7 @@ void testToolWithTaskSupportRequired() {
672672
var server = prepareSyncServerBuilder().serverInfo("test-server", "1.0.0")
673673
.capabilities(TaskTestUtils.DEFAULT_TASK_CAPABILITIES)
674674
.taskStore(taskStore)
675-
.tool(taskTool,
675+
.toolCall(taskTool,
676676
(exchange, args) -> CallToolResult.builder()
677677
.content(List.of(new McpSchema.TextContent("Task completed!")))
678678
.isError(false)
@@ -692,7 +692,7 @@ void testToolWithTaskSupportOptional() {
692692
var server = prepareSyncServerBuilder().serverInfo("test-server", "1.0.0")
693693
.capabilities(TaskTestUtils.DEFAULT_TASK_CAPABILITIES)
694694
.taskStore(taskStore)
695-
.tool(taskTool, (exchange, args) -> CallToolResult.builder().content(List.of()).isError(false).build())
695+
.toolCall(taskTool, (exchange, args) -> CallToolResult.builder().content(List.of()).isError(false).build())
696696
.build();
697697

698698
assertThatCode(server::closeGracefully).doesNotThrowAnyException();

0 commit comments

Comments
 (0)