Add tests and add promise support to keys

Author: dabroek

.babelrc

@@ -8,6 +8,16 @@
     ]
   ],
   "plugins": [
-    "external-helpers"
-  ]
+    "external-helpers",
+  ],
+  "env": {
+    "test": {
+      "presets": [
+        "es2015"
+      ],
+      "plugins": [
+        "transform-runtime"
+      ]
+    }
+  }
 }

dist/index.js

@@ -5,11 +5,7 @@ function _interopDefault (ex) { return (ex && (typeof ex === 'object') && 'defau
 var Redis = _interopDefault(require('redis'));
 
 var redisStore = function redisStore() {
-  for (var _len = arguments.length, args = Array(_len), _key = 0; _key < _len; _key++) {
-    args[_key] = arguments[_key];
-  }
-
-  var redisCache = Redis.createClient.apply(Redis, args);
+  var redisCache = Redis.createClient.apply(Redis, arguments);
   var storeArgs = redisCache.options;
 
   return {
@@ -66,13 +62,26 @@ var redisStore = function redisStore() {
     reset: function reset(cb) {
       return redisCache.flushdb(handleResponse(cb));
     },
-    keys: function keys(cb) {
-      return redisCache.keys(handleResponse(cb));
+    keys: function keys(pattern, cb) {
+      return new Promise(function (resolve, reject) {
+        if (typeof pattern === 'function') {
+          cb = pattern;
+          pattern = '*';
+        }
+
+        if (!cb) {
+          cb = function cb(err, result) {
+            return err ? reject(err) : resolve(result);
+          };
+        }
+
+        redisCache.keys(pattern, handleResponse(cb));
+      });
     },
     ttl: function ttl(key, cb) {
       return redisCache.ttl(key, handleResponse(cb));
     },
-    isCacheableValue: args.isCacheableValue || function (value) {
+    isCacheableValue: storeArgs.is_cacheable_value || function (value) {
       return value !== undefined && value !== null;
     }
   };

index.js

@@ -50,9 +50,22 @@ const redisStore = (...args) => {
       redisCache.del(key, handleResponse(cb));
     },
     reset: cb => redisCache.flushdb(handleResponse(cb)),
-    keys: cb => redisCache.keys(handleResponse(cb)),
+    keys: (pattern, cb) => (
+      new Promise((resolve, reject) => {
+        if (typeof pattern === 'function') {
+          cb = pattern;
+          pattern = '*';
+        }
+
+        if (!cb) {
+          cb = (err, result) => (err ? reject(err) : resolve(result));
+        }
+
+        redisCache.keys(pattern, handleResponse(cb));
+      })
+    ),
     ttl: (key, cb) => redisCache.ttl(key, handleResponse(cb)),
-    isCacheableValue: args.isCacheableValue || (value => value !== undefined && value !== null),
+    isCacheableValue: storeArgs.is_cacheable_value || (value => value !== undefined && value !== null),
   };
 };
 

package.json

@@ -2,7 +2,7 @@
   "name": "cache-manager-redis-store",
   "author": "Matthijs Dabroek <[email protected]>",
   "description": "Redis store for node-cache-manager module",
-  "version": "1.1.0",
+  "version": "1.2.0",
   "license": "MIT",
   "main": "dist/index.js",
   "repository": {
@@ -11,15 +11,20 @@
   },
   "scripts": {
     "prepublish": "npm prune",
-    "test": "node test.js",
+    "test": "jest",
     "build": "rollup -c"
   },
   "dependencies": {
     "redis": "^2.7.1"
   },
   "devDependencies": {
+    "babel-cli": "^6.24.1",
+    "babel-jest": "^19.0.0",
     "babel-plugin-external-helpers": "^6.22.0",
+    "babel-plugin-transform-runtime": "^6.23.0",
     "babel-preset-es2015": "^6.24.1",
+    "cache-manager": "^2.4.0",
+    "jest": "^19.0.2",
     "rollup": "^0.41.6",
     "rollup-plugin-babel": "^2.7.1"
   },