Skip to content

Commit 2f90ccb

Browse files
committed
feat(test): Increase request timeout to 60 seconds in test configurations
1 parent ae19789 commit 2f90ccb

9 files changed

+16
-14
lines changed

src/test/java/com/github/codeboyzhou/mcp/declarative/McpServersTest.java

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,8 @@ class McpServersTest {
3535

3636
McpServers servers = McpServers.run(McpServersTest.class, new String[] {});
3737

38+
Duration requestTimeout = Duration.ofSeconds(60);
39+
3840
@Test
3941
void testStartStdioServer_shouldSucceed() {
4042
TestMcpStdioServer.main(new String[] {}); // just for jacoco coverage report
@@ -46,10 +48,10 @@ void testStartStdioServer_shouldSucceed() {
4648
.args("-cp", classpath, TestMcpStdioServer.class.getName())
4749
.build();
4850

49-
StdioClientTransport stdioClientTransport =
51+
StdioClientTransport transport =
5052
new StdioClientTransport(serverParameters, McpJsonMapper.getDefault());
5153

52-
try (McpSyncClient client = McpClient.sync(stdioClientTransport).build()) {
54+
try (McpSyncClient client = McpClient.sync(transport).requestTimeout(requestTimeout).build()) {
5355
verify(client);
5456
}
5557
}
@@ -63,7 +65,7 @@ void testStartSseServer_shouldSucceed() {
6365
.name("mcp-server")
6466
.version("1.0.0")
6567
.instructions("test")
66-
.requestTimeout(Duration.ofSeconds(10))
68+
.requestTimeout(requestTimeout)
6769
.baseUrl("http://localhost:" + port)
6870
.port(port)
6971
.sseEndpoint("/sse")
@@ -77,7 +79,7 @@ void testStartSseServer_shouldSucceed() {
7779

7880
servers.startSseServer(serverInfo);
7981

80-
try (McpSyncClient client = McpClient.sync(transport).build()) {
82+
try (McpSyncClient client = McpClient.sync(transport).requestTimeout(requestTimeout).build()) {
8183
verify(client);
8284
}
8385
}
@@ -91,7 +93,7 @@ void testStartStreamableServer_shouldSucceed() {
9193
.name("mcp-server")
9294
.version("1.0.0")
9395
.instructions("test")
94-
.requestTimeout(Duration.ofSeconds(10))
96+
.requestTimeout(requestTimeout)
9597
.port(port)
9698
.mcpEndpoint("/mcp/message")
9799
.build();
@@ -103,7 +105,7 @@ void testStartStreamableServer_shouldSucceed() {
103105

104106
servers.startStreamableServer(serverInfo);
105107

106-
try (McpSyncClient client = McpClient.sync(transport).build()) {
108+
try (McpSyncClient client = McpClient.sync(transport).requestTimeout(requestTimeout).build()) {
107109
verify(client);
108110
}
109111
}

src/test/java/com/github/codeboyzhou/mcp/declarative/test/TestMcpStdioServer.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ public static void main(String[] args) {
1212
.name("mcp-server")
1313
.version("1.0.0")
1414
.instructions("test")
15-
.requestTimeout(Duration.ofSeconds(30))
15+
.requestTimeout(Duration.ofSeconds(60))
1616
.build();
1717
McpServers.run(TestMcpStdioServer.class, args).startStdioServer(info);
1818
}

src/test/resources/mcp-server.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ mode: STREAMABLE
33
name: mcp-server
44
version: 1.0.0
55
type: SYNC
6-
request-timeout: 30000
6+
request-timeout: 60000
77
capabilities:
88
resource: true
99
prompt: true

src/test/resources/test-mcp-server-disabled.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ mode: STREAMABLE
33
name: mcp-server
44
version: 1.0.0
55
type: SYNC
6-
request-timeout: 30000
6+
request-timeout: 60000
77
capabilities:
88
resource: true
99
prompt: true

src/test/resources/test-mcp-server-enable-http-sse-mode.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ mode: SSE
33
name: mcp-server
44
version: 1.0.0
55
type: SYNC
6-
request-timeout: 30000
6+
request-timeout: 60000
77
capabilities:
88
resource: true
99
prompt: true

src/test/resources/test-mcp-server-enable-stdio-mode.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ mode: STDIO
33
name: mcp-server
44
version: 1.0.0
55
type: SYNC
6-
request-timeout: 30000
6+
request-timeout: 60000
77
capabilities:
88
resource: true
99
prompt: true

src/test/resources/test-mcp-server-enable-streamable-http-mode.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ mode: STREAMABLE
33
name: mcp-server
44
version: 1.0.0
55
type: SYNC
6-
request-timeout: 30000
6+
request-timeout: 60000
77
capabilities:
88
resource: true
99
prompt: true

src/test/resources/test-mcp-server-enable-unknown-mode.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ mode: UNKNOWN
33
name: mcp-server
44
version: 1.0.0
55
type: SYNC
6-
request-timeout: 30000
6+
request-timeout: 60000
77
capabilities:
88
resource: true
99
prompt: true

src/test/resources/test-mcp-server-with-profile.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ mode: STREAMABLE
44
name: mcp-server
55
version: 1.0.0
66
type: SYNC
7-
request-timeout: 30000
7+
request-timeout: 60000
88
capabilities:
99
prompt: true
1010
tool: true

0 commit comments

Comments
 (0)