Merge pull request #24 from eschwartz/fix-wrap-missing-callback

Fix wrap missing callback

Author: four43

main.js

@@ -79,6 +79,9 @@ CrispCache.prototype.get = function (key, options, callback) {
         callback = options;
         options = {};
     }
+    if (!callback) {
+        throw new Error('Unable to retrieve "' + key + '" from cache: no callback provided');
+    }
 
     if (this.cache[key] === undefined || options.forceFetch) {
         //Cache miss.

test/crisp-cache-test.js

@@ -973,7 +973,7 @@ describe("CrispCache", function () {
                 cb(null, 'RETURN VAL');
             };
             var fetchCb = sinon.spy(function(fetchInfo) {
-               return "Fetching key: " + fetchInfo.key;
+                return "Fetching key: " + fetchInfo.key;
             });
             var hitCb = sinon.spy(function(fetchInfo) {
                 return "Hit key: " + fetchInfo.key;
@@ -1131,7 +1131,7 @@ describe("CrispCache", function () {
             assert.deepEqual(getOptions.callCount, 2, 'getOptions should be called once for every result');
         });
 
-        it(' should complain if `getOptions` throws an error', function(done) {
+        it('should complain if `getOptions` throws an error', function(done) {
             var cached = CrispCache.wrap(function(x, cb) { cb(null, 'foo') }, {
                 createKey: function(x) { return x; },
                 parseKey: function(x) { return [x]; },
@@ -1146,5 +1146,13 @@ describe("CrispCache", function () {
             });
         });
 
+        it('should complain if a wrapped function is called without a callback', function(done) {
+            const cached = CrispCache.wrap(function() {});
+
+            assert.throws(function() { cached() }, /callback/);
+
+            done();
+        });
+
     });
 });