fix: improve Redis cluster setup reliability and test coverage

appleboy · appleboy · commit 9e4080bd91f6 · 2025-01-18T11:39:01.000+08:00
- Add a wait condition for the Redis cluster setup to ensure it waits for the log message "Sending CLUSTER MEET messages to join the cluster"

Signed-off-by: appleboy &lt;appleboy.tw@gmail.com&gt;
diff --git a/redis_test.go b/redis_test.go
@@ -44,6 +44,7 @@ func setupRedisCluserContainer(ctx context.Context, t *testing.T) (testcontainer
 			"6383/tcp",
 			"6384/tcp",
 		},
+		WaitingFor: wait.ForLog("Sending CLUSTER MEET messages to join the cluster"),
 	}
 	redisC, err := testcontainers.GenericContainer(ctx, testcontainers.GenericContainerRequest{
 		ContainerRequest: req,

Original file line number	Diff line number	Diff line change
`@@ -44,6 +44,7 @@ func setupRedisCluserContainer(ctx context.Context, t *testing.T) (testcontainer`
`44`	`44`	`"6383/tcp",`
`45`	`45`	`"6384/tcp",`
`46`	`46`	`},`
	`47`	`+ WaitingFor: wait.ForLog("Sending CLUSTER MEET messages to join the cluster"),`
`47`	`48`	`}`
`48`	`49`	`redisC, err := testcontainers.GenericContainer(ctx, testcontainers.GenericContainerRequest{`
`49`	`50`	`ContainerRequest: req,`