Skip to content

Commit 1c7225e

Browse files
committed
feat(test): adds findAvailablePort for test server...
1 parent f980793 commit 1c7225e

File tree

4 files changed

+10
-8
lines changed

4 files changed

+10
-8
lines changed

mcp-spring/mcp-spring-webflux/src/test/java/io/modelcontextprotocol/server/TestUtil.java renamed to mcp-spring/mcp-spring-webflux/src/test/java/io/modelcontextprotocol/TestUtil.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
/*
22
* Copyright 2025 - 2025 the original author or authors.
33
*/
4-
package io.modelcontextprotocol.server;
4+
package io.modelcontextprotocol;
55

66
import java.io.IOException;
77
import java.net.InetSocketAddress;

mcp-spring/mcp-spring-webflux/src/test/java/io/modelcontextprotocol/WebFluxSseIntegrationTests.java

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,9 @@
4848
import static org.awaitility.Awaitility.await;
4949
import static org.mockito.Mockito.mock;
5050

51-
public class WebFluxSseIntegrationTests {
51+
class WebFluxSseIntegrationTests {
5252

53-
private static final int PORT = 8182;
53+
private static final int PORT = TestUtil.findAvailablePort();
5454

5555
private static final String CUSTOM_SSE_ENDPOINT = "/somePath/sse";
5656

@@ -129,7 +129,7 @@ void testCreateMessageWithoutSamplingCapabilities(String clientType) {
129129

130130
@ParameterizedTest(name = "{0} : {displayName} ")
131131
@ValueSource(strings = { "httpclient", "webflux" })
132-
void testCreateMessageSuccess(String clientType) throws InterruptedException {
132+
void testCreateMessageSuccess(String clientType) {
133133

134134
// Client
135135
var clientBuilder = clientBuilders.get(clientType);
@@ -188,8 +188,7 @@ void testCreateMessageSuccess(String clientType) throws InterruptedException {
188188

189189
CallToolResult response = mcpClient.callTool(new McpSchema.CallToolRequest("tool1", Map.of()));
190190

191-
assertThat(response).isNotNull();
192-
assertThat(response).isEqualTo(callResponse);
191+
assertThat(response).isNotNull().isEqualTo(callResponse);
193192

194193
mcpClient.close();
195194
mcpServer.close();
@@ -417,8 +416,7 @@ void testToolCallSuccess(String clientType) {
417416

418417
CallToolResult response = mcpClient.callTool(new McpSchema.CallToolRequest("tool1", Map.of()));
419418

420-
assertThat(response).isNotNull();
421-
assertThat(response).isEqualTo(callResponse);
419+
assertThat(response).isNotNull().isEqualTo(callResponse);
422420

423421
mcpClient.close();
424422
mcpServer.close();

mcp-spring/mcp-spring-webflux/src/test/java/io/modelcontextprotocol/server/WebFluxSseMcpAsyncServerTests.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@
55
package io.modelcontextprotocol.server;
66

77
import com.fasterxml.jackson.databind.ObjectMapper;
8+
9+
import io.modelcontextprotocol.TestUtil;
810
import io.modelcontextprotocol.server.transport.WebFluxSseServerTransportProvider;
911
import io.modelcontextprotocol.spec.McpServerTransportProvider;
1012
import org.junit.jupiter.api.Timeout;

mcp-spring/mcp-spring-webflux/src/test/java/io/modelcontextprotocol/server/WebFluxSseMcpSyncServerTests.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@
55
package io.modelcontextprotocol.server;
66

77
import com.fasterxml.jackson.databind.ObjectMapper;
8+
9+
import io.modelcontextprotocol.TestUtil;
810
import io.modelcontextprotocol.server.transport.WebFluxSseServerTransportProvider;
911
import io.modelcontextprotocol.spec.McpServerTransportProvider;
1012
import org.junit.jupiter.api.Timeout;

0 commit comments

Comments
 (0)