mirror of
https://github.com/facebook/react.git
synced 2025-11-01 09:12:30 +00:00
Remove obsolete tests with new tree-walking
This commit is contained in:
1 parent
19e2cf5ad5
commit
5becbe3f08
2 files changed
-83
No files matched your search
@@ -196,31 +196,6 @@ describe('ReactMount', function() {
|
||||
});
|
||||
}
|
||||
|
||||
it('warns when using two copies of React before throwing', function() {
|
||||
jest.resetModuleRegistry();
|
||||
var RD1 = require('ReactDOM');
|
||||
jest.resetModuleRegistry();
|
||||
var RD2 = require('ReactDOM');
|
||||
|
||||
var X = React.createClass({
|
||||
render: function() {
|
||||
return <div />;
|
||||
},
|
||||
});
|
||||
|
||||
var container = document.createElement('div');
|
||||
spyOn(console, 'error');
|
||||
var component = RD1.render(<X />, container);
|
||||
expect(console.error.argsForCall.length).toBe(0);
|
||||
|
||||
// This fails but logs a warning first
|
||||
expect(function() {
|
||||
RD2.findDOMNode(component);
|
||||
}).toThrow();
|
||||
expect(console.error.argsForCall.length).toBe(1);
|
||||
expect(console.error.argsForCall[0][0]).toContain('two copies of React');
|
||||
});
|
||||
|
||||
it('should warn if render removes React-rendered children', function() {
|
||||
var container = document.createElement('container');
|
||||
var Component = React.createClass({
|
||||
|
||||
@@ -149,62 +149,4 @@ describe('ReactMultiChild', function() {
|
||||
expect(mockUnmount.mock.calls.length).toBe(1);
|
||||
});
|
||||
});
|
||||
|
||||
describe('innerHTML', function() {
|
||||
var setInnerHTML;
|
||||
|
||||
// Only run this suite if `Element.prototype.innerHTML` can be spied on.
|
||||
var innerHTMLDescriptor = Object.getOwnPropertyDescriptor(
|
||||
Element.prototype,
|
||||
'innerHTML'
|
||||
);
|
||||
if (!innerHTMLDescriptor) {
|
||||
return;
|
||||
}
|
||||
|
||||
beforeEach(function() {
|
||||
var ReactDOMFeatureFlags = require('ReactDOMFeatureFlags');
|
||||
ReactDOMFeatureFlags.useCreateElement = false;
|
||||
|
||||
Object.defineProperty(Element.prototype, 'innerHTML', {
|
||||
set: setInnerHTML = jasmine.createSpy().andCallFake(
|
||||
innerHTMLDescriptor.set
|
||||
),
|
||||
});
|
||||
});
|
||||
|
||||
it('should only set `innerHTML` once on update', function() {
|
||||
var container = document.createElement('div');
|
||||
|
||||
ReactDOM.render(
|
||||
<div>
|
||||
<p><span /></p>
|
||||
<p><span /></p>
|
||||
<p><span /></p>
|
||||
</div>,
|
||||
container
|
||||
);
|
||||
// Warm the cache used by `getMarkupWrap`.
|
||||
ReactDOM.render(
|
||||
<div>
|
||||
<p><span /><span /></p>
|
||||
<p><span /><span /></p>
|
||||
<p><span /><span /></p>
|
||||
</div>,
|
||||
container
|
||||
);
|
||||
expect(setInnerHTML).toHaveBeenCalled();
|
||||
var callCountOnMount = setInnerHTML.calls.length;
|
||||
|
||||
ReactDOM.render(
|
||||
<div>
|
||||
<p><span /><span /><span /></p>
|
||||
<p><span /><span /><span /></p>
|
||||
<p><span /><span /><span /></p>
|
||||
</div>,
|
||||
container
|
||||
);
|
||||
expect(setInnerHTML.calls.length).toBe(callCountOnMount + 1);
|
||||
});
|
||||
});
|
||||
});
|
||||
Reference in new issue
Block a user