Skip to content

Commit

Permalink
Adding the disableMyMaster as false in the default test
Browse files Browse the repository at this point in the history
  • Loading branch information
samof76 committed Sep 3, 2024
1 parent ed3acdc commit 97edfad
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions test/integration/redisfailover/creation_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@ func (c *clients) cleanup(stopC chan struct{}) {

func TestRedisFailover(t *testing.T) {
require := require.New(t)
disableMyMaster := false

// Create signal channels.
stopC := make(chan struct{})
Expand Down Expand Up @@ -131,7 +132,9 @@ func TestRedisFailover(t *testing.T) {
require.NoError(err)

// Check that if we create a RedisFailover, it is certainly created and we can get it
ok := t.Run("Check Custom Resource Creation", clients.testCRCreation)
ok := t.Run("Check Custom Resource Creation", func(t *testing.T) {
clients.testCRCreation(t, disableMyMaster)
})
require.True(ok, "the custom resource has to be created to continue")

// Giving time to the operator to create the resources
Expand All @@ -158,7 +161,9 @@ func TestRedisFailover(t *testing.T) {
// Connect to all the Sentinel pods and, asking to the Sentinel running inside them,
// check that all of them are connected to the same Redis node, and also that that node
// is the master.
t.Run("Check Sentinels Checking the Redis Master", clients.testSentinelMonitoring)
t.Run("Check Sentinels Checking the Redis Master", func(t *testing.T) {
clients.testSentinelMonitoring(t, disableMyMaster)
})
}

func TestRedisFailoverWithMasterName(t *testing.T) {
Expand Down

0 comments on commit 97edfad

Please sign in to comment.