Skip to content

Commit d0b365d

Browse files
committed
Sort members by rs member id
1 parent 00a9472 commit d0b365d

File tree

1 file changed

+16
-10
lines changed

1 file changed

+16
-10
lines changed

test/e2e/mongodbtests/mongodbtests.go

+16-10
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import (
44
"context"
55
"encoding/json"
66
"fmt"
7+
"sort"
78
"strings"
89
"testing"
910
"time"
@@ -429,16 +430,13 @@ func AutomationConfigHasVoteTagPriorityConfigured(ctx context.Context, mdb *mdbv
429430

430431
return func(t *testing.T) {
431432
currentAc := getAutomationConfig(ctx, t, mdb)
432-
rsMemebers := currentAc.ReplicaSets
433-
434-
for _, m := range rsMemebers[0].Members {
435-
var priorityPtr *string
436-
var priority string
437-
if m.Priority != nil {
438-
priority = fmt.Sprintf("%f", *m.Priority)
439-
priorityPtr = &priority
440-
}
441-
acMemberOptions = append(acMemberOptions, automationconfig.MemberOptions{Votes: m.Votes, Priority: priorityPtr, Tags: m.Tags})
433+
rsMembers := currentAc.ReplicaSets
434+
sort.Slice(rsMembers[0].Members, func(i, j int) bool {
435+
return rsMembers[0].Members[i].Id < rsMembers[0].Members[j].Id
436+
})
437+
438+
for _, m := range rsMembers[0].Members {
439+
acMemberOptions = append(acMemberOptions, automationconfig.MemberOptions{Votes: m.Votes, Priority: floatPtrTostringPtr(m.Priority), Tags: m.Tags})
442440
}
443441
assert.ElementsMatch(t, memberOptions, acMemberOptions)
444442
}
@@ -825,3 +823,11 @@ func AddUserToMongoDBCommunity(ctx context.Context, mdb *mdbv1.MongoDBCommunity,
825823
}
826824
}
827825
}
826+
827+
func floatPtrTostringPtr(floatPtr *float32) *string {
828+
if floatPtr != nil {
829+
stringValue := fmt.Sprintf("%.1f", *floatPtr)
830+
return &stringValue
831+
}
832+
return nil
833+
}

0 commit comments

Comments
 (0)