8000 HttpDistTest.maxQueuedRequestsTest fails on runners with limited vCPUs by Pepo48 · Pull Request #48599 · keycloak/keycloak · GitHub
[go: up one dir, main page]

Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7D37
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,8 @@
import java.util.ArrayList;
import java.util.List;
import java.util.concurrent.CompletableFuture;
import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;

import org.keycloak.it.junit5.extension.CLIResult;
import org.keycloak.it.junit5.extension.DistributionTest;
Expand Down Expand Up @@ -50,16 +52,20 @@ public class HttpDistTest {
public void maxQueuedRequestsTest(KeycloakDistribution dist) {
dist.run("start-dev", "--http-max-queued-requests=1", "--http-pool-max-threads=1");

// run requests async
List<CompletableFuture<Integer>> statusCodesFuture = new ArrayList<>();
for (int i = 0; i < 5; i++) {
statusCodesFuture.add(CompletableFuture.supplyAsync(() ->
when().get("/realms/master/test-resources/slow").getStatusCode()));
ExecutorService executor = Executors.newFixedThreadPool(5);
try {
List<CompletableFuture<Integer>> statusCodesFuture = new ArrayList<>();
for (int i = 0; i < 5; i++) {
statusCodesFuture.add(CompletableFuture.supplyAsync(() ->
when().get("/realms/master/test-resources/slow").getStatusCode(), executor));
}
List<Integer> statusCodes = statusCodesFuture.stream().map(CompletableFuture::join).toList();

assertThat("Some of the requests should be properly rejected", statusCodes, hasItem(503));
assertThat("None of the requests should throw an unhandled exception", statusCodes, not(hasItem(500)));
} finally {
executor.shutdown();
}
List<Integer> statusCodes = statusCodesFuture.stream().map(CompletableFuture::join).toList();

assertThat("Some of the requests should be properly rejected", statusCodes, hasItem(503));
assertThat("None of the requests should throw an unhandled exception", statusCodes, not(hasItem(500)));
}

@Test
Expand Down
Loading
0