From 1a288cd5274e39370411fd7c88340d25fef2bc54 Mon Sep 17 00:00:00 2001 From: Artem Bilan Date: Wed, 14 Apr 2021 10:17:13 -0400 Subject: [PATCH] Remove resetConnection from Redis Reactive test Related to https://build.spring.io/browse/INT-SONAR-3564 When we reset shared Redis connection between tests, it may lead to a dead lock on some async command in process when the next test initiate a new connection * Remove `resetConnection()` from the `ReactiveRedisStreamMessageProducerTests` `@After` - looks like it doesn't effect anything in a whole test suite --- .../redis/inbound/ReactiveRedisStreamMessageProducerTests.java | 1 - 1 file changed, 1 deletion(-) diff --git a/spring-integration-redis/src/test/java/org/springframework/integration/redis/inbound/ReactiveRedisStreamMessageProducerTests.java b/spring-integration-redis/src/test/java/org/springframework/integration/redis/inbound/ReactiveRedisStreamMessageProducerTests.java index 2e6fd0ee51..4235fa9f82 100644 --- a/spring-integration-redis/src/test/java/org/springframework/integration/redis/inbound/ReactiveRedisStreamMessageProducerTests.java +++ b/spring-integration-redis/src/test/java/org/springframework/integration/redis/inbound/ReactiveRedisStreamMessageProducerTests.java @@ -97,7 +97,6 @@ public class ReactiveRedisStreamMessageProducerTests extends RedisAvailableTests @After public void tearDown() { this.reactiveRedisStreamProducer.stop(); - RedisAvailableRule.connectionFactory.resetConnection(); } @Test