Merge pull request reduxjs#72 from emmenko/fix-tests

Adjust tests after API changes

Author: gaearon

src/Redux.js

@@ -40,11 +40,12 @@ export default class Redux {
   }
 
   subscribe(listener) {
-    this.listeners.push(listener);
+    const { listeners } = this;
+    listeners.push(listener);
 
     return function unsubscribe () {
-      const index = this.listeners.indexOf(listener);
-      this.listeners.splice(index, 1);
+      const index = listeners.indexOf(listener);
+      listeners.splice(index, 1);
     };
   }
 }

test/createDispatcher.spec.js

@@ -1,5 +1,6 @@
 import expect from 'expect';
 import { createDispatcher, composeStores } from '../src';
+import thunkMiddleware from '../src/middleware/thunk';
 
 const fakeState = { foo: 'bar' };
 const fakeAction = { type: 'FOO', foo: 'bar' };
@@ -33,7 +34,9 @@ describe('createDispatcher', () => {
 
   it('should handle sync and async dispatches', done => {
     const spy = expect.createSpy(() => {});
-    const dispatcher = createDispatcher(composeStores({ fakeStore }));
+    const dispatcher = createDispatcher(
+      composeStores({ fakeStore }),
+      getState => [thunkMiddleware(getState)]);
     expect(dispatcher).toBeA('function');
 
     const dispatchFn = dispatcher(fakeState, spy);

test/exports.spec.js

@@ -5,7 +5,7 @@ describe('Redux', () => {
 
   it('should export necessary components', () => {
     const imports = Object.keys(redux);
-    expect(imports.length).toBe(8);
+    expect(imports.length).toBe(9);
 
     expect(imports).toContain('createRedux');
     expect(imports).toContain('createDispatcher');
@@ -16,6 +16,7 @@ describe('Redux', () => {
     expect(imports).toContain('provide');
     expect(imports).toContain('connect');
 
+    expect(imports).toContain('compose');
     expect(imports).toContain('composeStores');
     expect(imports).toContain('bindActionCreators');
   });