Merge pull request #179 from mattybow/master

subscribe moved from Connector constructor to componentDidMount

Author: gaearon

src/components/createConnector.js

@@ -38,10 +38,13 @@ export default function createConnector(React) {
     constructor(props, context) {
       super(props, context);
 
-      this.unsubscribe = context.redux.subscribe(::this.handleChange);
       this.state = this.selectState(props, context);
     }
 
+    componentDidMount() {
+      this.unsubscribe = this.context.redux.subscribe(::this.handleChange);
+    }
+
     componentWillReceiveProps(nextProps) {
       if (nextProps.select !== this.props.select) {
         // Force the state slice recalculation

test/components/Connector.spec.js

@@ -202,5 +202,30 @@ describe('React', () => {
         );
       }).toThrow(/select/);
     });
+
+    it('does not throw error when `renderToString` is called on server', () => {
+      const { renderToString } = React;
+      const redux = createRedux({ string: stringBuilder });
+      class TestComp extends Component {
+        componentWillMount() {
+          // simulate response action on data returning
+          redux.dispatch({ type: 'APPEND', body: 'a'});
+        }
+        render() {
+          return (<div>{this.props.string}</div>);
+        }
+      }
+      const el = (
+        <Provider redux={redux}>
+          {() => (
+            <Connector select={state => ({ string: state.string })}>
+              {({ string }) => <TestComp string={string} />}
+            </Connector>
+          )}
+        </Provider>
+      );
+      expect(() => renderToString(el)).toNotThrow();
+
+    });
   });
 });