bugfix #4292 (#4295)

* withRouter: Return the wrapper component if no router is present fixes #4292

* Add test for without Router case

modules/__tests__/withRouter-test.js

@@ -128,4 +128,17 @@ describe('withRouter', function () {
       done()
     })
   })
+
+  it('should render Component even without Router context', function (done) {
+    const MyComponent = withRouter(({ router }) => {
+      expect(router).toNotExist()
+
+      return <h1>Hello</h1>
+    })
+
+    render((<MyComponent />), node, function () {
+      expect(node.firstChild.textContent).toEqual('Hello')
+      done()
+    })
+  })
 })

modules/withRouter.js

@@ -29,6 +29,10 @@ export default function withRouter(WrappedComponent, options) {
 
     render() {
       const router = this.props.router || this.context.router
+      if (!router) {
+        return <WrappedComponent {...this.props} />
+      }
+
       const { params, location, routes } = router
       const props = { ...this.props, router, params, location, routes }