Skip to content

Commit 10b6fe0

Browse files
committed
Fixed auto-closeable side effect with Junit 5.8
1 parent c60acf5 commit 10b6fe0

File tree

1 file changed

+6
-3
lines changed
  • subprojects/testcontainers-redis-junit-jupiter/src/main/java/com/redis/testcontainers/junit/jupiter

1 file changed

+6
-3
lines changed

subprojects/testcontainers-redis-junit-jupiter/src/main/java/com/redis/testcontainers/junit/jupiter/RedisTestContext.java

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,15 +35,15 @@ public RedisServer getServer() {
3535

3636
public AbstractRedisClient getClient() {
3737
if (client == null) {
38-
this.client = server.isCluster() ? RedisModulesClusterClient.create(server.getRedisURI())
39-
: RedisModulesClient.create(server.getRedisURI());
38+
String uri = server.getRedisURI();
39+
client = server.isCluster() ? RedisModulesClusterClient.create(uri) : RedisModulesClient.create(uri);
4040
}
4141
return client;
4242
}
4343

4444
public StatefulRedisModulesConnection<String, String> getConnection() {
4545
if (connection == null) {
46-
this.connection = connection();
46+
connection = connection();
4747
}
4848
return connection;
4949
}
@@ -85,13 +85,16 @@ public RedisModulesReactiveCommands<String, String> reactive() {
8585
public void close() {
8686
if (pubSubConnection != null) {
8787
pubSubConnection.close();
88+
pubSubConnection = null;
8889
}
8990
if (connection != null) {
9091
connection.close();
92+
connection = null;
9193
}
9294
if (client != null) {
9395
client.shutdown();
9496
client.getResources().shutdown();
97+
client = null;
9598
}
9699
}
97100

0 commit comments

Comments
 (0)