add comments, reorder prints, add default value for REDIS_MAJOR_VERSION

Author: ndyakov

main_test.go

@@ -93,12 +93,15 @@ var _ = BeforeSuite(func() {
 	RCEDocker, _ = strconv.ParseBool(os.Getenv("RCE_DOCKER"))
 
 	REDIS_MAJOR_VERSION, _ = strconv.Atoi(os.Getenv("REDIS_MAJOR_VERSION"))
+	if REDIS_MAJOR_VERSION == 0 {
+		REDIS_MAJOR_VERSION = 7
+	}
 	Expect(REDIS_MAJOR_VERSION).To(BeNumerically(">=", 6))
 	Expect(REDIS_MAJOR_VERSION).To(BeNumerically("<=", 8))
 
 	fmt.Printf("RECluster: %v\n", RECluster)
-	fmt.Printf("REDIS_MAJOR_VERSION: %v\n", REDIS_MAJOR_VERSION)
 	fmt.Printf("RCEDocker: %v\n", RCEDocker)
+	fmt.Printf("REDIS_MAJOR_VERSION: %v\n", REDIS_MAJOR_VERSION)
 	if !RECluster && !RCEDocker {
 
 		redisMain, err = startRedis(redisPort)

search_test.go

@@ -371,10 +371,11 @@ var _ = Describe("RediSearch commands Resp 2", Label("search"), func() {
 		Expect(names).To(ContainElement("John"))
 	})
 
-	// in redis 8, the default scorer is changed BM25
+	// up until redis 8 the default scorer was TFIDF, in redis 8 it is BM25
+	// this test expect redis major version >= 8
 	It("should FTSearch WithScores", Label("search", "ftsearch"), func() {
 		if REDIS_MAJOR_VERSION < 8 {
-			Skip("default scorer is not BM25")
+			Skip("(redis major version < 8) default scorer is not BM25")
 		}
 		text1 := &redis.FieldSchema{FieldName: "description", FieldType: redis.SearchFieldTypeText}
 		val, err := client.FTCreate(ctx, "idx1", &redis.FTCreateOptions{}, text1).Result()
@@ -414,9 +415,11 @@ var _ = Describe("RediSearch commands Resp 2", Label("search"), func() {
 		Expect(*res.Docs[0].Score).To(BeEquivalentTo(float64(0)))
 	})
 
+	// up until redis 8 the default scorer was TFIDF, in redis 8 it is BM25
+	// this test expect redis major version <=7
 	It("should FTSearch WithScores", Label("search", "ftsearch"), func() {
-		if REDIS_MAJOR_VERSION >= 8 {
-			Skip("default scorer is not TFIDF")
+		if REDIS_MAJOR_VERSION > 7 {
+			Skip("(redis major version > 7) default scorer is not TFIDF")
 		}
 		text1 := &redis.FieldSchema{FieldName: "description", FieldType: redis.SearchFieldTypeText}
 		val, err := client.FTCreate(ctx, "idx1", &redis.FTCreateOptions{}, text1).Result()