chore: use redis-mock for tests

Some issues with library, trying to resolve.

Author: Ovyerus

package.json

@@ -36,7 +36,8 @@
         "chai-as-promised": "^7.1.1",
         "eslint": "^5.15.1",
         "mocha": "^6.0.2",
-        "nyc": "^13.3.0"
+        "nyc": "^13.3.0",
+        "redis-mock": "^0.43.0"
     },
     "dependencies": {
         "redis": "^2.8.0"

src/promisifyRedis.js

@@ -1,15 +1,21 @@
 const {RedisClient} = require('redis');
 
-Object.entries(RedisClient.prototype).filter(v => typeof v[1] === 'function').forEach(([key, func]) => {
-    if (RedisClient.prototype['p' + key]) return;
-
-    RedisClient.prototype['p' + key] = function(...args) {
-        return new Promise((resolve, reject) => {
-            func.call(this, ...args, (err, res) => {
-                /* istanbul ignore next */
-                if (err) reject(err);
-                else resolve(res);
+function promisifyRedisClient(proto) {
+    Object.entries(proto).filter(v => typeof v[1] === 'function').forEach(([key, func]) => {
+        if (proto['p' + key]) return;
+
+        proto['p' + key] = function(...args) {
+            return new Promise((resolve, reject) => {
+                func.call(this, ...args, (err, res) => {
+                    /* istanbul ignore next */
+                    if (err) reject(err);
+                    else resolve(res);
+                });
             });
-        });
-    };
-});
+        };
+    });
+}
+
+promisifyRedisClient(RedisClient.prototype);
+
+module.exports = promisifyRedisClient;

test/ChildWrapper.test.js

@@ -1,7 +1,8 @@
 const {expect, use} = require('chai');
 const chaiAsPromised = require('chai-as-promised');
-const redis = require('redis');
+const redis = require('redis-mock');
 const Redite = require('../');
+const promisifyRedisClient = require('../src/promisifyRedis');
 const {
     promisify,
     DB,
@@ -12,6 +13,7 @@ const {
 } = require('./lib/consts');
 
 use(chaiAsPromised);
+promisifyRedisClient(redis.RedisClient.prototype);
 
 const client = redis.createClient({db: DB});
 const wrapper = new Redite({client});
@@ -172,9 +174,12 @@ describe('ChildWrapper', () => {
                 });
             });
 
-            it('should edit an item in a list without overriding any parts of it', async () => {
+            it.only('should edit an item in a list without overriding any parts of it', async () => {
+                console.log('1');
                 await wrapper.test[0].foo.bar.set(TestHash);
+                console.log('2');
                 await wrapper.test[0].foo.TestHash.set(TestVal);
+                console.log('3');
                 await expect(wrapper.test[0]).to.become({
                     foo: {
                         bar: TestHash,

test/Redite.test.js

@@ -2,14 +2,16 @@
 
 const {expect, use} = require('chai');
 const chaiAsPromised = require('chai-as-promised');
-const redis = require('redis');
+const redis = require('redis-mock');
 const Redite = require('../');
+const promisifyRedisClient = require('../src/promisifyRedis');
 const {
     promisify,
     DB
 } = require('./lib/consts');
 
 use(chaiAsPromised);
+promisifyRedisClient(redis.RedisClient.prototype);
 
 const client = redis.createClient({db: DB});
 const wrapper = new Redite({client});

test/arrayMethods.test.js

@@ -1,7 +1,8 @@
 const {expect, use} = require('chai');
 const chaiAsPromised = require('chai-as-promised');
-const redis = require('redis');
+const redis = require('redis-mock');
 const Redite = require('../');
+const promisifyRedisClient = require('../src/promisifyRedis');
 const {
     promisify,
     DB,
@@ -15,6 +16,7 @@ const {
 } = require('./lib/consts');
 
 use(chaiAsPromised);
+promisifyRedisClient(redis.RedisClient.prototype);
 
 const client = redis.createClient({db: DB});
 const wrapper = new Redite({client});

test/other.test.js

@@ -3,6 +3,8 @@
 const {expect, use} = require('chai');
 const chaiAsPromised = require('chai-as-promised');
 const redis = require('redis');
+const redisMock = require('redis-mock');
+const promisifyRedisClient = require('../src/promisifyRedis');
 const {inspect: {custom}} = require('util');
 const {fork} = require('child_process');
 const Redite = require('../');
@@ -13,8 +15,9 @@ const {
 } = require('./lib/consts');
 
 use(chaiAsPromised);
+promisifyRedisClient(redisMock.RedisClient.prototype);
 
-const client = redis.createClient({db: DB});
+const client = redisMock.createClient({db: DB});
 const wrapper = new Redite({
     client,
     customInspection: true,

yarn.lock

@@ -1972,6 +1972,11 @@ redis-commands@^1.2.0:
   resolved "https://registry.yarnpkg.com/redis-commands/-/redis-commands-1.4.0.tgz#52f9cf99153efcce56a8f86af986bd04e988602f"
   integrity sha512-cu8EF+MtkwI4DLIT0x9P8qNTLFhQD4jLfxLR0cCNkeGzs87FN6879JOJwNQR/1zD7aSYNbU0hgsV9zGY71Itvw==
 
+redis-mock@^0.43.0:
+  version "0.43.0"
+  resolved "https://registry.yarnpkg.com/redis-mock/-/redis-mock-0.43.0.tgz#e05bef36b75f7332d41c13828d63baf7b47bced1"
+  integrity sha512-ayNlCxbeaVp16U5HU220C24lFk7TEzW8hL4RE5SXv/GeKq+P4ek1QqKKbu0K2QkGRdO2bvHE2ymJ/pbBLqyUww==
+
 redis-parser@^2.6.0:
   version "2.6.0"
   resolved "https://registry.yarnpkg.com/redis-parser/-/redis-parser-2.6.0.tgz#52ed09dacac108f1a631c07e9b69941e7a19504b"