fix: Go 1.18 supports was broken because of slices and maps usage

These packages were only added to Go 1.21

Author: ccoVeille

doctests/query_em_test.go

@@ -5,7 +5,7 @@ package example_commands_test
 import (
 	"context"
 	"fmt"
-	"slices"
+	"sort"
 	"strings"
 
 	"github.com/redis/go-redis/v9"
@@ -279,8 +279,8 @@ func ExampleClient_query_em() {
 	fmt.Println(res3.Total) // >>> 5
 
 	docs := res3.Docs
-	slices.SortFunc(docs, func(a, b redis.Document) int {
-		return strings.Compare(a.ID, b.ID)
+	sort.Slice(docs, func(i, j int) bool {
+		return strings.Compare(docs[i].ID, docs[j].ID) < 0
 	})
 
 	for _, doc := range docs {

doctests/timeseries_tut_test.go

@@ -5,9 +5,7 @@ package example_commands_test
 import (
 	"context"
 	"fmt"
-	"maps"
 	"math"
-	"slices"
 	"sort"
 
 	"github.com/redis/go-redis/v9"
@@ -417,7 +415,11 @@ func ExampleClient_timeseries_query_multi() {
 		panic(err)
 	}
 
-	res28Keys := slices.Collect(maps.Keys(res28))
+	var res28Keys []string
+	for k := range res28 {
+		res28Keys = append(res28Keys, k)
+	}
+
 	sort.Strings(res28Keys)
 
 	for _, k := range res28Keys {
@@ -457,7 +459,10 @@ func ExampleClient_timeseries_query_multi() {
 		panic(err)
 	}
 
-	res29Keys := slices.Collect(maps.Keys(res29))
+	var res29Keys []string
+	for k := range res29 {
+		res29Keys = append(res29Keys, k)
+	}
 	sort.Strings(res29Keys)
 
 	for _, k := range res29Keys {
@@ -505,7 +510,10 @@ func ExampleClient_timeseries_query_multi() {
 		panic(err)
 	}
 
-	res30Keys := slices.Collect(maps.Keys(res30))
+	var res30Keys []string
+	for k := range res30 {
+		res30Keys = append(res30Keys, k)
+	}
 	sort.Strings(res30Keys)
 
 	for _, k := range res30Keys {
@@ -550,7 +558,10 @@ func ExampleClient_timeseries_query_multi() {
 		panic(err)
 	}
 
-	res31Keys := slices.Collect(maps.Keys(res31))
+	var res31Keys []string
+	for k := range res31 {
+		res31Keys = append(res31Keys, k)
+	}
 	sort.Strings(res31Keys)
 
 	for _, k := range res31Keys {
@@ -857,7 +868,10 @@ func ExampleClient_timeseries_aggmulti() {
 		panic(err)
 	}
 
-	res44Keys := slices.Collect(maps.Keys(res44))
+	var res44Keys []string
+	for k := range res44 {
+		res44Keys = append(res44Keys, k)
+	}
 	sort.Strings(res44Keys)
 
 	for _, k := range res44Keys {
@@ -905,7 +919,10 @@ func ExampleClient_timeseries_aggmulti() {
 		panic(err)
 	}
 
-	res45Keys := slices.Collect(maps.Keys(res45))
+	var res45Keys []string
+	for k := range res45 {
+		res45Keys = append(res45Keys, k)
+	}
 	sort.Strings(res45Keys)
 
 	for _, k := range res45Keys {

maintnotifications/hooks.go

@@ -2,7 +2,6 @@ package maintnotifications
 
 import (
 	"context"
-	"slices"
 
 	"github.com/redis/go-redis/v9/internal"
 	"github.com/redis/go-redis/v9/internal/maintnotifications/logs"
@@ -15,6 +14,17 @@ type LoggingHook struct {
 	LogLevel int // 0=Error, 1=Warn, 2=Info, 3=Debug
 }
 
+// slicesContains is an helper function to check if a slice contains a specific string.
+// TODO: Replace with slices.Contains when we move to Go 1.21+
+func slicesContains(slice []string, item string) bool {
+	for _, s := range slice {
+		if s == item {
+			return true
+		}
+	}
+	return false
+}
+
 // PreHook logs the notification before processing and allows modification.
 func (lh *LoggingHook) PreHook(ctx context.Context, notificationCtx push.NotificationHandlerContext, notificationType string, notification []interface{}) ([]interface{}, bool) {
 	if lh.LogLevel >= 2 { // Info level
@@ -24,7 +34,7 @@ func (lh *LoggingHook) PreHook(ctx context.Context, notificationCtx push.Notific
 			connID = conn.GetID()
 		}
 		seqID := int64(0)
-		if slices.Contains(maintenanceNotificationTypes, notificationType) {
+		if slicesContains(maintenanceNotificationTypes, notificationType) {
 			// seqID is the second element in the notification array
 			if len(notification) > 1 {
 				if parsedSeqID, ok := notification[1].(int64); !ok {

osscluster_test.go

@@ -6,7 +6,6 @@ import (
 	"errors"
 	"fmt"
 	"net"
-	"slices"
 	"strconv"
 	"strings"
 	"sync"
@@ -107,6 +106,18 @@ func (s *clusterScenario) Close() error {
 	return nil
 }
 
+// slicesContains is an helper function to check if a slice contains a specific string.
+//
+//	TODO: Replace with slices.Contains when we move to Go 1.21+
+func slicesContains(slice []string, item string) bool {
+	for _, s := range slice {
+		if s == item {
+			return true
+		}
+	}
+	return false
+}
+
 func configureClusterTopology(ctx context.Context, scenario *clusterScenario) error {
 	allowErrs := []string{
 		"ERR Slot 0 is already busy",
@@ -131,8 +142,10 @@ func configureClusterTopology(ctx context.Context, scenario *clusterScenario) er
 	slots := scenario.slots()
 	for pos, master := range scenario.masters() {
 		err := master.ClusterAddSlotsRange(ctx, slots[pos], slots[pos+1]-1).Err()
-		if err != nil && slices.Contains(allowErrs, err.Error()) == false {
-			return err
+		if err != nil {
+			if !slicesContains(allowErrs, err.Error()) {
+				return err
+			}
 		}
 	}