Added setnx and fixed hsetnx (stipsan#65)

* hsetnx lied when it said it didn't overwrite

* added setnx command

* 1.6.0

* updated compat

compat.md

@@ -125,7 +125,7 @@
 |[set](http://redis.io/commands/SET)|:white_check_mark:|:white_check_mark:|
 |[setbit](http://redis.io/commands/SETBIT)|:white_check_mark:|:x:|
 |[setex](http://redis.io/commands/SETEX)|:white_check_mark:|:x:|
-|[setnx](http://redis.io/commands/SETNX)|:white_check_mark:|:x:|
+|[setnx](http://redis.io/commands/SETNX)|:white_check_mark:|:white_check_mark:|
 |[setrange](http://redis.io/commands/SETRANGE)|:white_check_mark:|:x:|
 |[shutdown](http://redis.io/commands/SHUTDOWN)|:white_check_mark:|:x:|
 |[sinter](http://redis.io/commands/SINTER)|:white_check_mark:|:x:|

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ioredis-mock",
-  "version": "1.5.0",
+  "version": "1.6.0",
   "description": "This library emulates ioredis by performing all operations in-memory.",
   "main": "./lib",
   "scripts": {

src/commands/hsetnx.js

@@ -2,8 +2,12 @@ export function hsetnx(key, hashKey, hashVal) {
   if (!this.data.hasOwnProperty(key)) {
     this.data[key] = {};
   }
-  const exists = this.data[key].hasOwnProperty(hashKey);
-  this.data[key][hashKey] = hashVal;
 
-  return !exists;
+  if (!this.data[key].hasOwnProperty(hashKey)) {
+    this.data[key][hashKey] = hashVal;
+
+    return '1';
+  }
+
+  return '0';
 }

src/commands/index.js

@@ -21,6 +21,7 @@ export * from './incrby';
 export * from './rename';
 export * from './sadd';
 export * from './set';
+export * from './setnx';
 export * from './sismember';
 export * from './smembers';
 export * from './srem';

src/commands/setnx.js

@@ -0,0 +1,9 @@
+export function setnx(key, val) {
+  if (!this.data.hasOwnProperty(key)) {
+    this.data[key] = val;
+
+    return '1';
+  }
+
+  return '0';
+}

test/commands/hsetnx.js

@@ -6,12 +6,16 @@ describe('hsetnx', () => {
   it('should set a key in a hash map if it does not exist already', () => {
     const redis = new MockRedis();
     return redis.hsetnx('emails', 'bruce@wayne.enterprises', '1')
-         .then(userNext => expect(userNext).toBe(true))
+         .then(status => expect(status).toBe('1'))
          .then(() => {
            expect(redis.data.emails['bruce@wayne.enterprises'])
            .toBe('1', 'hash map value failed to persist');
            return redis.hsetnx('emails', 'bruce@wayne.enterprises', '2');
          })
-         .then(userNext => expect(userNext).toBeFalsy('hsetnx no-op failed on existing key'));
+         .then(status => expect(status).toBe('0', 'hsetnx no-op failed on existing key'))
+         .then(() => {
+           expect(redis.data.emails['bruce@wayne.enterprises'])
+           .toBe('1', 'existing hash map value was overwritten');
+         });
   });
 });

test/commands/setnx.js

@@ -0,0 +1,21 @@
+import expect from 'expect';
+
+import MockRedis from '../../src';
+
+describe('setnx', () => {
+  it('should set a key with value if it does not exist already', () => {
+    const redis = new MockRedis();
+    return redis.setnx('foo', 'bar')
+         .then(status => expect(status).toBe('1'))
+         .then(() => {
+           expect(redis.data.foo)
+           .toBe('bar', 'value failed to persist');
+           return redis.setnx('foo', 'baz');
+         })
+         .then(status => expect(status).toBe('0', 'setnx no-op failed on existing key'))
+         .then(() => {
+           expect(redis.data.foo)
+           .toBe('bar', 'existing value was overwritten');
+         });
+  });
+});