Merge pull request #48 from jeff-kilbride/feature/undefined_json

Feature/undefined json

Author: PuKoren

index.js

@@ -72,12 +72,8 @@ function redisStore(args) {
       }
 
       if (opts.parse) {
-
         try {
-          // allow undefined only if allowed by isCacheableValue
-          if(! ( (result === undefined || result === 'undefined') && typeof args.isCacheableValue === 'function' && args.isCacheableValue(result))) {
-            result = JSON.parse(result);
-          }
+          result = JSON.parse(result);
         } catch (e) {
           return cb && cb(e);
         }
@@ -187,7 +183,7 @@ function redisStore(args) {
       options = {};
     }
 
-    if (!value && !self.isCacheableValue(value)) {
+    if (!self.isCacheableValue(value)) {
       return cb(new Error('value cannot be ' + value));
     }
 
@@ -199,7 +195,7 @@ function redisStore(args) {
       if (err) {
         return cb && cb(err);
       }
-      var val = JSON.stringify(value) || 'undefined';
+      var val = JSON.stringify(value) || '"undefined"';
       if (ttl) {
         conn.setex(key, ttl, val, handleResponse(conn, cb));
       } else {

test/lib/redis-store-spec.js

@@ -5,6 +5,7 @@ var assert = require('assert');
 
 var redisCache;
 var customRedisCache;
+var customRedisCache2;
 
 before(function () {
   redisCache = require('cache-manager').caching({
@@ -28,6 +29,19 @@ before(function () {
       return redisCache.store.isCacheableValue(val);
     }
   });
+
+  customRedisCache2 = require('cache-manager').caching({
+    store: redisStore,
+    host: config.redis.host,
+    port: config.redis.port,
+    db: config.redis.db,
+    ttl: config.redis.ttl,
+    isCacheableValue: function (val) {
+      // disallow FooBarString
+      if (val === 'FooBarString') return false;
+      return redisCache.store.isCacheableValue(val);
+    }
+  });
 });
 
 describe ('initialization', function () {
@@ -137,6 +151,19 @@ describe('set', function () {
     });
   });
 
+  it('should not store a value disallowed by isCacheableValue', function (done) {
+    assert.strictEqual(customRedisCache2.store.isCacheableValue('FooBarString'), false);
+    customRedisCache2.set('foobar', 'FooBarString', function (err) {
+      try {
+        assert.notEqual(err, null);
+        assert.equal(err.message, 'value cannot be FooBarString');
+        done();
+      } catch (e) {
+        done(e);
+      }
+    });
+  });
+
 });
 
 describe('get', function () {