From dcb9b055e733911c65b4ab9de00b5fcd6162598c Mon Sep 17 00:00:00 2001 From: Andreas Marek Date: Fri, 16 May 2025 13:35:55 +1000 Subject: [PATCH] use always a new thread for the tck verification as the TCK uses thread locals to track things, which can cause tests failure in rare cases --- .../CompletionStageMappingPublisherTckVerificationTest.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/test/groovy/graphql/execution/reactive/tck/CompletionStageMappingPublisherTckVerificationTest.java b/src/test/groovy/graphql/execution/reactive/tck/CompletionStageMappingPublisherTckVerificationTest.java index 9c58e3fd6e..f68c7d3fab 100644 --- a/src/test/groovy/graphql/execution/reactive/tck/CompletionStageMappingPublisherTckVerificationTest.java +++ b/src/test/groovy/graphql/execution/reactive/tck/CompletionStageMappingPublisherTckVerificationTest.java @@ -10,6 +10,7 @@ import java.time.Duration; import java.util.concurrent.CompletableFuture; import java.util.concurrent.CompletionStage; +import java.util.concurrent.Executors; import java.util.function.Function; /** @@ -31,7 +32,7 @@ public long maxElementsFromPublisher() { @Override public Publisher createPublisher(long elements) { Publisher publisher = Flowable.range(0, (int) elements); - Function> mapper = i -> CompletableFuture.supplyAsync(() -> i + "!"); + Function> mapper = i -> CompletableFuture.supplyAsync(() -> i + "!", Executors.newSingleThreadExecutor()); return new CompletionStageMappingPublisher<>(publisher, mapper); }