Use free ports for cluster tests.

This ensures that the cluster tests won't compete with each other for the same ports.
amazon-contributing · Jun 26, 2024 · 5c47d9b · 5c47d9b
1 parent 5e8e532
commit 5c47d9b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/redis/tests/support/mod.rs b/redis/tests/support/mod.rs
@@ -499,7 +499,7 @@ impl TestContext {
         let millisecond = Duration::from_millis(1);
         let mut retries = 0;
         loop {
-            match client.get_connection() {
+            match client.get_connection(None) {
                 Err(err) => {
                     if err.is_connection_refusal() {
                         sleep(millisecond);

diff --git a/redis/tests/test_cluster_async.rs b/redis/tests/test_cluster_async.rs
@@ -3514,10 +3514,13 @@ mod cluster_async {
             drop(cluster);
 
             // recreate cluster
-            let _cluster = RedisCluster::new(RedisClusterConfiguration {
-                ports: ports.clone(),
-                ..Default::default()
-            });
+            let _cluster = TestClusterContext::new_with_config_and_builder(
+                RedisClusterConfiguration {
+                    ports: ports.clone(),
+                    ..Default::default()
+                },
+                |builder| builder.retries(2),
+            );
 
             let cmd = cmd("PING");
             // explicitly route to all primaries and request all succeeded