Skip to content

Commit 0fb2308

Browse files
committed
fix when processes should be started
1 parent 5ada947 commit 0fb2308

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

main_test.go

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ var _ = BeforeSuite(func() {
8989
RCEDocker, _ = strconv.ParseBool(os.Getenv("RCE_DOCKER"))
9090
fmt.Printf("RECluster: %v\n", RECluster)
9191
fmt.Printf("RCEDocker: %v\n", RCEDocker)
92-
if !RECluster || !RCEDocker {
92+
if !RECluster && !RCEDocker {
9393

9494
redisMain, err = startRedis(redisPort)
9595
Expect(err).NotTo(HaveOccurred())
@@ -130,17 +130,15 @@ var _ = BeforeSuite(func() {
130130
redisAddr = rediStackAddr
131131
// populate cluster node information
132132
Expect(configureClusterTopology(ctx, cluster)).NotTo(HaveOccurred())
133-
134133
}
135134
})
136135

137136
var _ = AfterSuite(func() {
138-
if !RECluster {
139-
Expect(cluster.Close()).NotTo(HaveOccurred())
137+
// This should result in no errors when there were no processes started from the tests
138+
Expect(cluster.Close()).NotTo(HaveOccurred())
140139

141-
for _, p := range processes {
142-
Expect(p.Close()).NotTo(HaveOccurred())
143-
}
140+
for _, p := range processes {
141+
Expect(p.Close()).NotTo(HaveOccurred())
144142
}
145143
processes = nil
146144
})

0 commit comments

Comments
 (0)