Merge pull request #170 from psfrankie/master

fixed setProps execption swallowing issue and added regression test

Author: lelandrichardson

src/ReactWrapperComponent.jsx

@@ -34,7 +34,7 @@ export default function createWrapperComponent(node, options = {}) {
 
     setChildProps(newProps) {
       const props = objectAssign({}, this.state.props, newProps);
-      return new Promise(resolve => this.setState({ props }, resolve));
+      this.setState({ props });
     },
 
     setChildContext(context) {

src/__tests__/ReactWrapper-spec.js

@@ -508,6 +508,35 @@ describeWithDOM('mount', () => {
       expect(wrapper.props().d).to.equal('e');
     });
 
+    it('should throw if an exception occurs during render', () => {
+      class Trainwreck extends React.Component {
+        render() {
+          const { user } = this.props;
+          return (
+            <div>
+              {user.name.givenName}
+            </div>
+          );
+        }
+      }
+
+      const validUser = {
+        name: {
+          givenName: 'Brian',
+        },
+      };
+
+      const wrapper = mount(<Trainwreck user={validUser} />);
+
+      const setInvalidProps = () => {
+        wrapper.setProps({
+          user: {},
+        });
+      };
+
+      expect(setInvalidProps).to.throw();
+    });
+
   });
 
   describe('.setContext(newContext)', () => {
@@ -536,7 +565,6 @@ describeWithDOM('mount', () => {
     });
   });
 
-
   describe('.mount()', () => {
     it('should call componentWillUnmount()', () => {
       const willMount = sinon.spy();