Dispatch should be passed by default

Author: gaearon

src/components/createConnect.js

@@ -6,7 +6,7 @@ import wrapActionCreators from '../utils/wrapActionCreators';
 import invariant from 'invariant';
 
 const emptySelector = () => ({});
-const emptyBinder = () => ({});
+const defaultBinder = dispatch => ({ dispatch });
 const identityMerge = (slice, actionsCreators, props) => ({
   ...props,
   ...slice,
@@ -23,7 +23,7 @@ export default function createConnect(React) {
 
   return function connect(
     select,
-    dispatchBinder = emptyBinder,
+    dispatchBinder = defaultBinder,
     mergeHandler = identityMerge
   ) {
     const shouldSubscribe = select ? true : false;

test/components/connect.spec.js

@@ -53,7 +53,7 @@ describe('React', () => {
       expect(container.context.store).toBe(store);
     });
 
-    it('should pass the state and the props given component', () => {
+    it('should pass state and props to the given component', () => {
       const store = createStore(() => ({
         foo: 'bar',
         baz: 42,
@@ -244,15 +244,12 @@ describe('React', () => {
       expect(decorated.subscribed).toBe(true);
     });
 
-    it('should not subscribe to stores if select argument is null', () => {
+    it('should not subscribe to stores if select argument is falsy', () => {
       const store = createStore(() => ({
         foo: 'bar'
       }));
 
-      @connect(
-        null,
-        dispatch => ({ dispatch })
-      )
+      @connect()
       class Container extends Component {
         render() {
           return <div {...this.props} />;