Revert "Add warning when bindActionCreators encounters non-function p…

…roperty (#2279)" (#2473)

This reverts commit 1b154e0.

src/bindActionCreators.js

@@ -1,5 +1,3 @@
-import warning from './utils/warning'
-
 function bindActionCreator(actionCreator, dispatch) {
   return (...args) => dispatch(actionCreator(...args))
 }
@@ -44,8 +42,6 @@ export default function bindActionCreators(actionCreators, dispatch) {
     const actionCreator = actionCreators[key]
     if (typeof actionCreator === 'function') {
       boundActionCreators[key] = bindActionCreator(actionCreator, dispatch)
-    } else {
-      warning(`bindActionCreators expected a function actionCreator for key '${key}', instead received type '${typeof actionCreator}'.`)
     }
   }
   return boundActionCreators

test/bindActionCreators.spec.js

@@ -17,8 +17,6 @@ describe('bindActionCreators', () => {
   })
 
   it('wraps the action creators with the dispatch function', () => {
-    const _console = console
-    global.console = { error: jest.fn() }
     const boundActionCreators = bindActionCreators(actionCreators, store.dispatch)
     expect(
       Object.keys(boundActionCreators)
@@ -33,13 +31,9 @@ describe('bindActionCreators', () => {
     expect(store.getState()).toEqual([
       { id: 1, text: 'Hello' }
     ])
-    expect(console.error).toHaveBeenCalled()
-    global.console = _console
   })
 
   it('skips non-function values in the passed object', () => {
-    const _console = console
-    global.console = { error: jest.fn() }
     const boundActionCreators = bindActionCreators({
       ...actionCreators,
       foo: 42,
@@ -53,9 +47,6 @@ describe('bindActionCreators', () => {
     ).toEqual(
       Object.keys(actionCreatorFunctions)
     )
-    // 6 instead of 5 because of `__esModule: true` property from importing `actionCreators`
-    expect(console.error.mock.calls.length).toBe(6)
-    global.console = _console
   })
 
   it('supports wrapping a single function only', () => {