Add some important `getDOMNode` tests back.
cpojer committed
Mar 10, 2015 at 17:27 UTC
413e96d0cd1b58ed95d45166d14db77c057c67b2
2 files changed
+14
src/browser/ui/__tests__/ReactDOMComponent-test.js
+1
@@ -586,6 +586,7 @@ describe('ReactDOMComponent', function() {
586
expect(
587
ReactBrowserEventEmitter.getListener(rootNodeID, 'onClick')
588
).toBe(callback);
589
+ expect(rootNode).toBe(instance.getDOMNode());
590
591
React.unmountComponentAtNode(container);
592
src/core/__tests__/ReactCompositeComponent-test.js
+13
@@ -154,6 +154,7 @@ describe('ReactCompositeComponent', function() {
154
var anchor = instance.getAnchor();
155
var actualDOMAnchorNode = React.findDOMNode(anchor);
156
expect(actualDOMAnchorNode.className).toBe('');
157
+ expect(actualDOMAnchorNode).toBe(anchor.getDOMNode());
158
});
159
160
it('should auto bind methods and values correctly', function() {
@@ -926,11 +927,21 @@ describe('ReactCompositeComponent', function() {
927
expect(React.findDOMNode(comp.refs.static0).textContent).toBe('A');
928
expect(React.findDOMNode(comp.refs.static1).textContent).toBe('B');
929
930
+ expect(React.findDOMNode(comp.refs.static0))
931
+ .toBe(comp.refs.static0.getDOMNode());
932
+ expect(React.findDOMNode(comp.refs.static1))
933
+ .toBe(comp.refs.static1.getDOMNode());
934
+
935
// When flipping the order, the refs should update even though the actual
936
// contents do not
937
comp.setProps({flipped: true});
938
expect(React.findDOMNode(comp.refs.static0).textContent).toBe('B');
939
expect(React.findDOMNode(comp.refs.static1).textContent).toBe('A');
940
+
941
+ expect(React.findDOMNode(comp.refs.static0))
942
+ .toBe(comp.refs.static0.getDOMNode());
943
+ expect(React.findDOMNode(comp.refs.static1))
944
+ .toBe(comp.refs.static1.getDOMNode());
945
});
946
947
it('should allow access to findDOMNode in componentWillUnmount', function() {
@@ -939,9 +950,11 @@ describe('ReactCompositeComponent', function() {
950
var Component = React.createClass({
951
componentDidMount: function() {
952
a = React.findDOMNode(this);
953
+ expect(a).toBe(this.getDOMNode());
954
},
955
componentWillUnmount: function() {
956
b = React.findDOMNode(this);
957
+ expect(b).toBe(this.getDOMNode());
958
},
959
render: function() {
960
return <div />;