feat: Implement ZREMRANGEBYSCORE (stipsan#539)

Implementing https://redis.io/commands/zremrangebyscore.

compat.md

@@ -172,7 +172,7 @@
 | [zrem](http://redis.io/commands/ZREM)                                 | :white_check_mark: | :white_check_mark: |
 | [zremrangebylex](http://redis.io/commands/ZREMRANGEBYLEX)             | :white_check_mark: |        :x:         |
 | [zremrangebyrank](http://redis.io/commands/ZREMRANGEBYRANK)           | :white_check_mark: | :white_check_mark: |
-| [zremrangebyscore](http://redis.io/commands/ZREMRANGEBYSCORE)         | :white_check_mark: |        :x:         |
+| [zremrangebyscore](http://redis.io/commands/ZREMRANGEBYSCORE)         | :white_check_mark: | :white_check_mark: |
 | [zrevrange](http://redis.io/commands/ZREVRANGE)                       | :white_check_mark: | :white_check_mark: |
 | [zrevrangebylex](http://redis.io/commands/ZREVRANGEBYLEX)             | :white_check_mark: |        :x:         |
 | [zrevrangebyscore](http://redis.io/commands/ZREVRANGEBYSCORE)         | :white_check_mark: | :white_check_mark: |

src/commands/index.js

@@ -104,6 +104,7 @@ export * from './zrange';
 export * from './zrangebyscore';
 export * from './zrem';
 export * from './zremrangebyrank';
+export * from './zremrangebyscore';
 export * from './zrevrange';
 export * from './zrevrangebyscore';
 export * from './zscan';

src/commands/zremrangebyscore.js

@@ -0,0 +1,13 @@
+import { zrevrangebyscore } from './index';
+
+export function zremrangebyscore(key, inputMax, inputMin) {
+  const vals = zrevrangebyscore.call(this, key, inputMax, inputMin);
+
+  const map = this.data.get(key);
+  vals.forEach(val => {
+    map.delete(val);
+  });
+  this.data.set(key, map);
+
+  return vals.length;
+}

test/commands/zremrangebyscore.js

@@ -0,0 +1,89 @@
+import Map from 'es6-map';
+import expect from 'expect';
+
+import MockRedis from '../../src';
+
+describe('zremrangebyscore', () => {
+  const data = {
+    foo: new Map([
+      ['first', { score: 1, value: 'first' }],
+      ['second', { score: 2, value: 'second' }],
+      ['third', { score: 3, value: 'third' }],
+      ['fourth', { score: 4, value: 'fourth' }],
+      ['fifth', { score: 5, value: 'fifth' }],
+    ]),
+  };
+
+  it('should remove using not strict compare', () => {
+    const redis = new MockRedis({ data });
+
+    return redis
+      .zremrangebyscore('foo', 3, 1)
+      .then(res => expect(res).toBe(3))
+      .then(() => {
+        expect(redis.data.get('foo').has('first')).toBe(false);
+        expect(redis.data.get('foo').has('second')).toBe(false);
+        expect(redis.data.get('foo').has('third')).toBe(false);
+        expect(redis.data.get('foo').has('fourth')).toBe(true);
+        expect(redis.data.get('foo').has('fifth')).toBe(true);
+      });
+  });
+
+  it('should return using strict compare', () => {
+    const redis = new MockRedis({ data });
+
+    return redis
+      .zremrangebyscore('foo', 5, '(3')
+      .then(res => expect(res).toEqual(2))
+      .then(() => {
+        expect(redis.data.get('foo').has('first')).toBe(true);
+        expect(redis.data.get('foo').has('second')).toBe(true);
+        expect(redis.data.get('foo').has('third')).toBe(true);
+        expect(redis.data.get('foo').has('fourth')).toBe(false);
+        expect(redis.data.get('foo').has('fifth')).toBe(false);
+      });
+  });
+
+  it('should accept infinity string', () => {
+    const redis = new MockRedis({ data });
+
+    return redis
+      .zremrangebyscore('foo', '+inf', '-inf')
+      .then(res => expect(res).toEqual(5))
+      .then(() => {
+        expect(redis.data.get('foo').has('first')).toBe(false);
+        expect(redis.data.get('foo').has('second')).toBe(false);
+        expect(redis.data.get('foo').has('third')).toBe(false);
+        expect(redis.data.get('foo').has('fourth')).toBe(false);
+        expect(redis.data.get('foo').has('fifth')).toBe(false);
+      });
+  });
+
+  it('should return zero if out-of-range', () => {
+    const redis = new MockRedis({ data });
+
+    return redis
+      .zremrangebyscore('foo', 100, 10)
+      .then(res => expect(res).toEqual(0));
+  });
+
+  it('should return zero if key not found', () => {
+    const redis = new MockRedis({ data });
+
+    return redis
+      .zremrangebyscore('boo', 100, 10)
+      .then(res => expect(res).toEqual(0));
+  });
+
+  it('should return zero if the key contains something other than a list', () => {
+    const redis = new MockRedis({
+      data: {
+        foo: 'not a list',
+      },
+    });
+
+    return redis
+      .zremrangebyscore('foo', 2, 1)
+      .then(res => expect(res).toEqual(0));
+  });
+});