fixed mobxjs#216: findDOMNode might fail with react-test-renderer

mweststrate · mweststrate · commit e2942f271341 · 2017-03-10T13:30:50.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # MobX-React Changelog
 
+### 4.1.3
+
+* Fixed `ReactDOM.findDOMNode` exception when using react-test-runner, #216
+
 ### 4.1.2
 
 * Exceptions caught during render are now rethrown with proper stack, fixes #206
diff --git a/src/observer.js b/src/observer.js
@@ -18,8 +18,16 @@ export const componentByNodeRegistery = typeof WeakMap !== "undefined" ? new Wea
 export const renderReporter = new EventEmitter();
 
 function findDOMNode(component) {
-  if (ReactDOM)
-    return ReactDOM.findDOMNode(component);
+  if (ReactDOM) {
+    try {
+      return ReactDOM.findDOMNode(component);
+    } catch (e) {
+      // findDOMNode will throw in react-test-renderer, see:
+      // See https://github.com/mobxjs/mobx-react/issues/216
+      // Is there a better heuristic?
+      return null;
+    }
+  }
   return null;
 }
 
