Fix flakey test in HttpClientStreamableHttpTransportErrorHandlingTest (#535)

Kehrlann · web-flow · commit 4d779e139cc4 · 2025-09-04T12:55:26.000+02:00
diff --git a/mcp/src/test/java/io/modelcontextprotocol/client/transport/HttpClientStreamableHttpTransportErrorHandlingTest.java b/mcp/src/test/java/io/modelcontextprotocol/client/transport/HttpClientStreamableHttpTransportErrorHandlingTest.java
@@ -63,7 +63,7 @@ void startServer() throws IOException {
 			if ("DELETE".equals(httpExchange.getRequestMethod())) {
 				httpExchange.sendResponseHeaders(200, 0);
 			}
-			else {
+			else if ("POST".equals(httpExchange.getRequestMethod())) {
 				// Capture session ID from request if present
 				String requestSessionId = httpExchange.getRequestHeaders().getFirst(HttpHeaders.MCP_SESSION_ID);
 				lastReceivedSessionId.set(requestSessionId);

Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,7 @@ void startServer() throws IOException {`
`63`	`63`	`if ("DELETE".equals(httpExchange.getRequestMethod())) {`
`64`	`64`	`httpExchange.sendResponseHeaders(200, 0);`
`65`	`65`	`}`
`66`		`- else {`
	`66`	`+ else if ("POST".equals(httpExchange.getRequestMethod())) {`
`67`	`67`	`// Capture session ID from request if present`
`68`	`68`	`String requestSessionId = httpExchange.getRequestHeaders().getFirst(HttpHeaders.MCP_SESSION_ID);`
`69`	`69`	`lastReceivedSessionId.set(requestSessionId);`