diff options
author | guybe7 <guy.benoish@redislabs.com> | 2022-04-25 12:08:13 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-04-25 13:08:13 +0300 |
commit | df787764e37ba5958d5b902eef3847ec7185b156 (patch) | |
tree | 3011f2be0b1c157ab5a933de8d8b3702474275eb /tests/modules/basics.c | |
parent | 21e39ec461cf412609475054a9755941df9365bf (diff) | |
download | redis-df787764e37ba5958d5b902eef3847ec7185b156.tar.gz |
Fix regression not aborting transaction on error, and re-edit some error responses (#10612)
1. Disk error and slave count checks didn't flag the transactions or counted correctly in command stats (regression from #10372 , 7.0 RC3)
2. RM_Call will reply the same way Redis does, in case of non-exisitng command or arity error
3. RM_WrongArtiy will consider the full command name
4. Use lowercase 'u' in "unknonw subcommand" (to align with "unknown command")
Followup work of #10127
Diffstat (limited to 'tests/modules/basics.c')
-rw-r--r-- | tests/modules/basics.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/modules/basics.c b/tests/modules/basics.c index ecd1b8852..9dbb5a9d4 100644 --- a/tests/modules/basics.c +++ b/tests/modules/basics.c @@ -866,10 +866,10 @@ int TestBasics(RedisModuleCtx *ctx, RedisModuleString **argv, int argc) { if (!TestAssertStringReply(ctx,RedisModule_CallReplyArrayElement(reply, 1),"1234",4)) goto fail; T("foo", "E"); - if (!TestAssertErrorReply(ctx,reply,"ERR Unknown Redis command 'foo'.",32)) goto fail; + if (!TestAssertErrorReply(ctx,reply,"ERR unknown command 'foo', with args beginning with: ",53)) goto fail; T("set", "Ec", "x"); - if (!TestAssertErrorReply(ctx,reply,"ERR Wrong number of args calling Redis command 'set'.",53)) goto fail; + if (!TestAssertErrorReply(ctx,reply,"ERR wrong number of arguments for 'set' command",47)) goto fail; T("shutdown", "SE"); if (!TestAssertErrorReply(ctx,reply,"ERR command 'shutdown' is not allowed on script mode",52)) goto fail; |