Skip to content

Commit

Permalink
The testWindowTimespan() is time-sensitive
Browse files Browse the repository at this point in the history
The `FluxAggregatorMessageHandlerTests.testWindowTimespan()`
relies on the time-based window function of `100` millis.
Looks like a delay of `Thread.sleep(20)` may cause a resources race
condition and window is released only with one item.

* Disable `testWindowTimespan()` since there is no guarantee with what
timing settings it is going to work stably

**Cherry-pick to `5.5.x`**
  • Loading branch information
artembilan committed May 23, 2022
1 parent 4203a23 commit 4cc9b30
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@
import java.util.stream.Collectors;
import java.util.stream.IntStream;

import org.junit.jupiter.api.Disabled;
import org.junit.jupiter.api.Test;

import org.springframework.integration.IntegrationMessageHeaderAccessor;
Expand Down Expand Up @@ -157,6 +158,7 @@ void testCustomCombineFunction() {
}

@Test
@Disabled("Time-sensitive")
void testWindowTimespan() {
QueueChannel resultChannel = new QueueChannel();
FluxAggregatorMessageHandler fluxAggregatorMessageHandler = new FluxAggregatorMessageHandler();
Expand Down

0 comments on commit 4cc9b30

Please sign in to comment.