From a4cef2970341c08e5c16a2406fbf532fc8053d12 Mon Sep 17 00:00:00 2001 From: Rauno Freiberg Date: Mon, 16 Apr 2018 17:44:17 +0300 Subject: [PATCH] tests: add regression test for reading ReactCurrentOwner stateNode (#12412) * tests: add regression test for reading ReactCurrentOwner stateNode * tests: replace expect with just rendering the component --- .../react-dom/src/__tests__/ReactDOM-test.js | 20 +++++++++++++++++++ 1 file changed, 20 insertions(+) diff --git a/packages/react-dom/src/__tests__/ReactDOM-test.js b/packages/react-dom/src/__tests__/ReactDOM-test.js index 0ca365afd8a90..46b9f5cabe16d 100644 --- a/packages/react-dom/src/__tests__/ReactDOM-test.js +++ b/packages/react-dom/src/__tests__/ReactDOM-test.js @@ -377,6 +377,26 @@ describe('ReactDOM', () => { } }); + it('should not crash calling findDOMNode inside a functional component', () => { + const container = document.createElement('div'); + + class Component extends React.Component { + render() { + return
; + } + } + + const instance = ReactTestUtils.renderIntoDocument(); + const App = () => { + ReactDOM.findDOMNode(instance); + return
; + }; + + if (__DEV__) { + ReactDOM.render(, container); + } + }); + it('throws in DEV if jsdom is destroyed by the time setState() is called', () => { class App extends React.Component { state = {x: 1};