Merge pull request #20 from tswaters/master

Catch unhandled exceptions during render

src/renderer/renderPass.js

@@ -33,10 +33,14 @@ const renderPass = (context, element, staticMarkup = false) => {
     </AsyncRenderer>
   )
   let result;
-  if (staticMarkup) {
-    result = renderToStaticMarkup(component);
-  } else {
-    result = renderToString(component );
+  try {
+    if (staticMarkup) {
+      result = renderToStaticMarkup(component);
+    } else {
+      result = renderToString(component );
+    }
+  } catch (e) {
+    return context.reject(e);
   }
 
   if (!context.hasModules && !context.hasStates) {

test/tests/renderer/includes/FailureApp.jsx

@@ -0,0 +1,12 @@
+import * as React from 'react';
+import Module from '../../../../src/components/Module';
+
+const FailureApp = (props) => (
+  <div>
+    <Module name="foo" module={() => System.import('./FailureFoo')}>
+      { module => module ? <module.default/> : null}
+    </Module>
+  </div>
+);
+
+export default FailureApp;

test/tests/renderer/includes/FailureFoo.jsx

@@ -0,0 +1,7 @@
+import React, {PureComponent} from 'react'
+
+export default class FailureFoo extends PureComponent {
+  render () {
+    throw new Error('aw snap!')
+  }
+}

test/tests/renderer/renderToStaticMarkup.js

@@ -2,6 +2,7 @@ import * as React from 'react';
 import { expect } from 'chai';
 import renderToStaticMarkup from '../../../src/renderer/renderToStaticMarkup';
 import App from './includes/App';
+import FailureApp from './includes/FailureApp';
 
 describe('renderToStaticMarkup', () => {
   it('should do render to static markup', function (done) {
@@ -14,4 +15,13 @@ describe('renderToStaticMarkup', () => {
         done();
       })
   });
+  it('should handle errors properly', () => {
+    renderToStaticMarkup(<FailureApp/>)
+      .then(() => {
+        throw new Error('should not be called')
+      })
+      .catch(e => {
+        expect(e.message).to.equal('aw snap!')
+      })
+  })
 });

test/tests/renderer/renderToString.js

@@ -2,6 +2,7 @@ import * as React from 'react';
 import { expect } from 'chai';
 import renderToString from '../../../src/renderer/renderToString';
 import App from './includes/App';
+import FailureApp from './includes/FailureApp';
 
 describe('renderToString', () => {
   it('should do render to string with module loaded and state fetched', function (done) {
@@ -21,4 +22,13 @@ describe('renderToString', () => {
           })
       })
   });
+  it('should handle errors properly', () => {
+    renderToString(<FailureApp/>)
+      .then(() => {
+        throw new Error('should not be called')
+      })
+      .catch(e => {
+        expect(e.message).to.equal('aw snap!')
+      })
+  })
 });