Skip to content

Commit 3dcbf9e

Browse files
committed
fix (#3220):
* Compatible with known RediSearch issue in test
1 parent c697bf8 commit 3dcbf9e

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

search_test.go

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -644,7 +644,14 @@ var _ = Describe("RediSearch commands Resp 2", Label("search"), func() {
644644
Expect(res.Rows[0].Fields["t2"]).To(BeEquivalentTo("world"))
645645

646646
_, err = client.FTAggregateWithArgs(ctx, "idx_not_exist", "*", &redis.FTAggregateOptions{}).Result()
647-
Expect(err).To(HaveOccurred())
647+
if !RECluster {
648+
Expect(err).To(HaveOccurred())
649+
} else {
650+
//FIXME:This is a known issue, which is fixed in RedisSearch(2.8.10), but the Redis Enterprise(7.4.2) packaged RediSearch 2.8.9
651+
//https://github.com/RediSearch/RediSearch/issues/4272
652+
//https://redis.io/docs/latest/operate/rs/release-notes/rs-7-4-2-releases/rs-7-4-2-54/
653+
Expect(err).NotTo(HaveOccurred())
654+
}
648655
})
649656

650657
It("should FTAggregate apply", Label("search", "ftaggregate"), func() {

0 commit comments

Comments
 (0)