diff options
-rw-r--r-- | src/networking.c | 4 | ||||
-rw-r--r-- | tests/unit/introspection.tcl | 6 |
2 files changed, 5 insertions, 5 deletions
diff --git a/src/networking.c b/src/networking.c index 3b66dd59e..e0716b8a4 100644 --- a/src/networking.c +++ b/src/networking.c @@ -2909,12 +2909,12 @@ void clientSetinfoCommand(client *c) { } else if (!strcasecmp(attr,"lib-ver")) { destvar = &c->lib_ver; } else { - addReplyStatusFormat(c,"Unrecognized option '%s'", attr); + addReplyErrorFormat(c,"Unrecognized option '%s'", attr); return; } if (validateClientAttr(val)==C_ERR) { - addReplyStatusFormat(c, + addReplyErrorFormat(c, "%s cannot contain spaces, newlines or special characters.", attr); return; } diff --git a/tests/unit/introspection.tcl b/tests/unit/introspection.tcl index c8bc3eb89..4452de6b1 100644 --- a/tests/unit/introspection.tcl +++ b/tests/unit/introspection.tcl @@ -333,9 +333,9 @@ start_server {tags {"introspection"}} { test {CLIENT SETINFO invalid args} { assert_error {*wrong number of arguments*} {r CLIENT SETINFO lib-name} - assert_match {*cannot contain spaces*} [r CLIENT SETINFO lib-name "redis py"] - assert_match {*newlines*} [r CLIENT SETINFO lib-name "redis.py\n"] - assert_match {*Unrecognized*} [r CLIENT SETINFO badger hamster] + assert_error {*cannot contain spaces*} {r CLIENT SETINFO lib-name "redis py"} + assert_error {*newlines*} {r CLIENT SETINFO lib-name "redis.py\n"} + assert_error {*Unrecognized*} {r CLIENT SETINFO badger hamster} # test that all of these didn't affect the previously set values r client info } {*lib-name=redis.py lib-ver=1.2.3*} |