feat(cache): caching function is now a separate configuration

BREAKING CHANGE: caching function should be a function not related to the store

lib/index.js

@@ -6,9 +6,10 @@ export default function cache (config = {}) {
   }
 
   const store = config.store;
+  const cache = config.cache;
 
-  if (!store.cache || typeof store.cache !== 'function') {
-    throw new Error('Store does not have a cache function.');
+  if (!config.cache || typeof config.cache !== 'function') {
+    throw new Error('Cache middleware need a cache function.');
   }
 
   return (req, next, service) => {
@@ -19,7 +20,7 @@ export default function cache (config = {}) {
     }
 
     const promise = next()
-    promise.then(store.cache)
+    promise.then(cache)
 
     return promise;
   }

test/index.js

@@ -4,36 +4,44 @@ import superapiCache from '../lib/index.js'
 import test from 'blue-tape'
 import { spy } from 'sinon'
 
-test('middleware configuration', t => {
+test('middleware configuration need a store', t => {
   t.throws(() => {
     superapiCache()
   }, /Cache middleware need to be provided a store/, 'should throw if no store property found')
 
+  t.end()
+})
+
+test('middleware need a cache function ', t => {
   t.throws(() => {
     superapiCache({
-      store: {
-        foo: 'bar'
-      }
+      store: {}
     })
-  }, /Store does not have a cache function/, 'should throw if invalid store provided')
+  }, /Cache middleware need a cache function/, 'should throw if no caching function provided')
+
+  t.end()
+})
 
+
+test('middleware need a store and a cache function', t => {
   t.doesNotThrow(() => {
     superapiCache({
-      store: {
-        cache: function () {}
-      }
+      store: {},
+      cache: function () {}
     })
-  }, 'should not throw if valid store provided')
+  }, 'should not throw with a valid configuration provided')
+
+  t.end()
+})
   t.end()
 })
 
 
 test('should call cache function by default', t => {
   const cacheFn = spy()
   const handler = superapiCache({
-    store: {
-      cache: cacheFn
-    }
+    store: new MemoryStore(),
+    cache: cacheFn
   })
 
   const next = () => {
@@ -54,9 +62,8 @@ test('should call cache function by default', t => {
 test('disable cache', t => {
   const cacheFn = spy(function () {})
   const handler = superapiCache({
-    store: {
-      cache: cacheFn
-    }
+    store: {},
+    cache: cacheFn
   })
 
   const next = () => {