Skip to content

Commit 68b4407

Browse files
committed
test(timeseries): fix duplicatePolicy check
1 parent b8fb31f commit 68b4407

File tree

1 file changed

+12
-2
lines changed

1 file changed

+12
-2
lines changed

timeseries_commands_test.go

+12-2
Original file line numberDiff line numberDiff line change
@@ -269,11 +269,21 @@ var _ = Describe("RedisTimeseries commands", Label("timeseries"), func() {
269269
if client.Options().Protocol == 2 {
270270
Expect(resultInfo["labels"].([]interface{})[0]).To(BeEquivalentTo([]interface{}{"Time", "Series"}))
271271
Expect(resultInfo["retentionTime"]).To(BeEquivalentTo(10))
272-
Expect(resultInfo["duplicatePolicy"]).To(BeEquivalentTo(redis.Nil))
272+
if RedisVersion >= 8 {
273+
Expect(resultInfo["duplicatePolicy"]).To(BeEquivalentTo("block"))
274+
} else {
275+
// Older versions of Redis had a bug where the duplicate policy was not set correctly
276+
Expect(resultInfo["duplicatePolicy"]).To(BeEquivalentTo(redis.Nil))
277+
}
273278
} else {
274279
Expect(resultInfo["labels"].(map[interface{}]interface{})["Time"]).To(BeEquivalentTo("Series"))
275280
Expect(resultInfo["retentionTime"]).To(BeEquivalentTo(10))
276-
Expect(resultInfo["duplicatePolicy"]).To(BeEquivalentTo(redis.Nil))
281+
if RedisVersion >= 8 {
282+
Expect(resultInfo["duplicatePolicy"]).To(BeEquivalentTo("block"))
283+
} else {
284+
// Older versions of Redis had a bug where the duplicate policy was not set correctly
285+
Expect(resultInfo["duplicatePolicy"]).To(BeEquivalentTo(redis.Nil))
286+
}
277287
}
278288
opt = &redis.TSAlterOptions{DuplicatePolicy: "min"}
279289
resultAlter, err = client.TSAlter(ctx, "1", opt).Result()

0 commit comments

Comments
 (0)