yeahoffline / redis-mock

Node.js redis client mock
213 stars 111 forks source link

SET with NX flag does not work as expected #198

Open przpl opened 3 years ago

przpl commented 3 years ago

Example code

 this._client.SET("key", "value", "NX", (err, reply) => {
    console.log(reply); // should print "OK" only on the first call
});

NX is "Only set the key if it does not already exist" flag.

On real Redis we will get "OK" on the first call and null on the second, which is expected as the value was set at the first time and cannot be overwritten. Redis-mock will respond with "OK" on the first call, second and any other afterward which is not correct behavior.