[SET] - Invalid set is returning success

yeahoffline / redis-mock

Node.js redis client mock

213 stars 111 forks source link

[SET] - Invalid set is returning success #129

Closed andrewfreitas closed 3 years ago

andrewfreitas commented 4 years ago

Sendind a set command with invalid keys or invalid values are returning success. It need to return an error.

For example, a set command passing a null key is returning success.

kamiljano commented 3 years ago

looks like there even was a test to make sure that we do just that... validated with the redis module and confirmed that this is not the wanted behaviour. This should be fixed, along with a bunch of other closely related things over here: https://github.com/yeahoffline/redis-mock/pull/149