Add subscribe to Middleware API

Fixes reduxjs#922
jonnyreeves · Oct 21, 2015 · 8c9dfc0 · 8c9dfc0
1 parent 0d30e9b
commit 8c9dfc0
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/src/utils/applyMiddleware.js b/src/utils/applyMiddleware.js
@@ -24,7 +24,8 @@ export default function applyMiddleware(...middlewares) {
 
     var middlewareAPI = {
       getState: store.getState,
-      dispatch: (action) => dispatch(action)
+      dispatch: (action) => dispatch(action),
+      subscribe: store.subscribe
     };
     chain = middlewares.map(middleware => middleware(middlewareAPI));
     dispatch = compose(...chain)(store.dispatch);

diff --git a/test/utils/applyMiddleware.spec.js b/test/utils/applyMiddleware.spec.js
@@ -21,11 +21,17 @@ describe('applyMiddleware', () => {
 
     expect(spy.calls.length).toEqual(1);
 
-    expect(Object.keys(spy.calls[0].arguments[0])).toEqual([
+    const middlewareAPI = spy.calls[0].arguments[0];
+
+    expect(Object.keys(middlewareAPI)).toEqual([
       'getState',
-      'dispatch'
+      'dispatch',
+      'subscribe'
     ]);
 
+    expect(middlewareAPI.getState).toEqual(store.getState);
+    expect(middlewareAPI.subscribe).toEqual(store.subscribe);
+
     expect(store.getState()).toEqual([ { id: 1, text: 'Use Redux' }, { id: 2, text: 'Flux FTW!' } ]);
   });