From 9c338a5981f12cb2a20625901b1a9fbe736e1c65 Mon Sep 17 00:00:00 2001 From: michael-grunder Date: Tue, 27 Sep 2022 13:49:10 -0700 Subject: Fix PUSH handler tests for Redis >= 7.0.5 Redis updated how invalidation push messages are sent to the client after a FLUSHDB or FLUSHALL command. See: redis/redis#8935 --- test.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/test.c b/test.c index 67723f2..977a89b 100644 --- a/test.c +++ b/test.c @@ -954,7 +954,7 @@ static void test_resp3_push_handler(redisContext *c) { reply = redisCommand(c, "SET key:0 val:0"); /* We need another command because depending on the version of Redis, the * notification may be delivered after the command's reply. */ - test_cond(reply != NULL); + assert(reply != NULL); freeReplyObject(reply); reply = redisCommand(c, "PING"); test_cond(reply != NULL && reply->type == REDIS_REPLY_STATUS && pc.str == 1); @@ -962,6 +962,9 @@ static void test_resp3_push_handler(redisContext *c) { test("We properly handle a NIL invalidation payload: "); reply = redisCommand(c, "FLUSHDB"); + assert(reply != NULL); + freeReplyObject(reply); + reply = redisCommand(c, "PING"); test_cond(reply != NULL && reply->type == REDIS_REPLY_STATUS && pc.nil == 1); freeReplyObject(reply); -- cgit v1.2.3