Revert "chore: use redis-mock for tests"

This reverts commit 9681e17.

Author: Ovyerus

package.json

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

src/promisifyRedis.js

@@ -1,21 +1,15 @@
 const {RedisClient} = require('redis');
 
-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);
-                });
+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);
             });
-        };
-    });
-}
-
-promisifyRedisClient(RedisClient.prototype);
-
-module.exports = promisifyRedisClient;
+        });
+    };
+});

test/ChildWrapper.test.js

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

test/Redite.test.js

@@ -2,16 +2,14 @@
 
 const {expect, use} = require('chai');
 const chaiAsPromised = require('chai-as-promised');
-const redis = require('redis-mock');
+const redis = require('redis');
 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,8 +1,7 @@
 const {expect, use} = require('chai');
 const chaiAsPromised = require('chai-as-promised');
-const redis = require('redis-mock');
+const redis = require('redis');
 const Redite = require('../');
-const promisifyRedisClient = require('../src/promisifyRedis');
 const {
     promisify,
     DB,
@@ -16,7 +15,6 @@ 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,8 +3,6 @@
 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('../');
@@ -15,9 +13,8 @@ const {
 } = require('./lib/consts');
 
 use(chaiAsPromised);
-promisifyRedisClient(redisMock.RedisClient.prototype);
 
-const client = redisMock.createClient({db: DB});
+const client = redis.createClient({db: DB});
 const wrapper = new Redite({
     client,
     customInspection: true,

yarn.lock

@@ -1974,11 +1974,6 @@ 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"