Skip to content

Commit f7f6e65

Browse files
committed
removes mock test from non-mock test class
1 parent f72e92b commit f7f6e65

File tree

1 file changed

+1
-28
lines changed

1 file changed

+1
-28
lines changed

mcp/src/test/java/io/modelcontextprotocol/client/transport/HttpClientSseClientTransportTests.java

Lines changed: 1 addition & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,8 @@
77
import java.net.URI;
88
import java.net.http.HttpClient;
99
import java.net.http.HttpRequest;
10-
import java.net.http.HttpResponse;
1110
import java.time.Duration;
1211
import java.util.Map;
13-
import java.util.concurrent.CompletableFuture;
1412
import java.util.concurrent.atomic.AtomicBoolean;
1513
import java.util.concurrent.atomic.AtomicInteger;
1614
import java.util.concurrent.atomic.AtomicReference;
@@ -22,8 +20,7 @@
2220
import org.junit.jupiter.api.BeforeEach;
2321
import org.junit.jupiter.api.Test;
2422
import org.junit.jupiter.api.Timeout;
25-
import org.mockito.ArgumentCaptor;
26-
import org.mockito.Mockito;
23+
2724
import org.testcontainers.containers.GenericContainer;
2825
import org.testcontainers.containers.wait.strategy.Wait;
2926
import reactor.core.publisher.Mono;
@@ -34,9 +31,6 @@
3431

3532
import static org.assertj.core.api.Assertions.assertThat;
3633
import static org.assertj.core.api.Assertions.assertThatCode;
37-
import static org.mockito.ArgumentMatchers.any;
38-
import static org.mockito.Mockito.verify;
39-
import static org.mockito.Mockito.when;
4034

4135
import com.fasterxml.jackson.databind.ObjectMapper;
4236

@@ -370,25 +364,4 @@ void testChainedCustomizations() {
370364
customizedTransport.closeGracefully().block();
371365
}
372366

373-
@Test
374-
@SuppressWarnings("unchecked")
375-
void testResolvingClientEndpoint() {
376-
HttpClient httpClient = Mockito.mock(HttpClient.class);
377-
HttpResponse<Void> httpResponse = Mockito.mock(HttpResponse.class);
378-
CompletableFuture<HttpResponse<Void>> future = new CompletableFuture<>();
379-
future.complete(httpResponse);
380-
when(httpClient.sendAsync(any(HttpRequest.class), any(HttpResponse.BodyHandler.class))).thenReturn(future);
381-
382-
HttpClientSseClientTransport transport = new HttpClientSseClientTransport(httpClient, HttpRequest.newBuilder(),
383-
"http://example.com", "http://example.com/sse", new ObjectMapper());
384-
385-
transport.connect(Function.identity());
386-
387-
ArgumentCaptor<HttpRequest> httpRequestCaptor = ArgumentCaptor.forClass(HttpRequest.class);
388-
verify(httpClient).sendAsync(httpRequestCaptor.capture(), any(HttpResponse.BodyHandler.class));
389-
assertThat(httpRequestCaptor.getValue().uri()).isEqualTo(URI.create("http://example.com/sse"));
390-
391-
transport.closeGracefully().block();
392-
}
393-
394367
}

0 commit comments

Comments
 (0)