@samitouri / QOS-React-2 / commits / 4dde417214

Fix tests after merge

Test Plan: jest

Ben Alpert committed Mar 24, 2015 at 13:37 UTC 4dde417214fa48185246cddade518c57f6de32af
3 files changed +14 -14
src/browser/ui/__tests__/ReactDOMComponent-test.js
+9 -9
@@ -663,28 +663,28 @@ describe('ReactDOMComponent', function() {
663 });
664
665 it('warns on invalid nesting', () => {
666 - spyOn(console, 'warn');
666 + spyOn(console, 'error');
667 ReactTestUtils.renderIntoDocument(<div><tr /></div>);
668
669 - expect(console.warn.calls.length).toBe(1);
670 - expect(console.warn.calls[0].args[0]).toBe(
669 + expect(console.error.calls.length).toBe(1);
670 + expect(console.error.calls[0].args[0]).toBe(
671 'Warning: validateDOMNesting(...): <div> cannot contain a <tr> node.'
672 );
673 });
674
675 it('warns on invalid nesting at root', () => {
676 - spyOn(console, 'warn');
676 + spyOn(console, 'error');
677 var p = document.createElement('p');
678 React.render(<tr />, p);
679
680 - expect(console.warn.calls.length).toBe(1);
681 - expect(console.warn.calls[0].args[0]).toBe(
680 + expect(console.error.calls.length).toBe(1);
681 + expect(console.error.calls[0].args[0]).toBe(
682 'Warning: validateDOMNesting(...): <p> cannot contain a <tr> node.'
683 );
684 });
685
686 it('warns nicely for table rows', () => {
687 - spyOn(console, 'warn');
687 + spyOn(console, 'error');
688 var Foo = React.createClass({
689 render: function() {
690 return <table><tr /></table>;
@@ -692,8 +692,8 @@ describe('ReactDOMComponent', function() {
692 });
693 ReactTestUtils.renderIntoDocument(<Foo />);
694
695 - expect(console.warn.calls.length).toBe(1);
696 - expect(console.warn.calls[0].args[0]).toBe(
695 + expect(console.error.calls.length).toBe(1);
696 + expect(console.error.calls[0].args[0]).toBe(
697 'Warning: validateDOMNesting(...): <table> cannot contain a <tr> ' +
698 'node. Add a <tbody> to your code to match the DOM tree generated by ' +
699 'the browser. Check the render method of `Foo`.'
src/classic/class/__tests__/ReactClass-test.js
+2 -2
@@ -390,8 +390,8 @@ describe('ReactClass-spec', function() {
390
391 instance.getDOMNode();
392
393 - expect(console.warn.calls.length).toBe(1);
394 - expect(console.warn.calls[0].args[0]).toContain(
393 + expect(console.error.calls.length).toBe(1);
394 + expect(console.error.calls[0].args[0]).toContain(
395 'MyComponent.getDOMNode(...) is deprecated. Please use ' +
396 'React.findDOMNode(instance) instead.'
397 );
src/core/__tests__/ReactComponent-test.js
+3 -3
@@ -266,15 +266,15 @@ describe('ReactComponent', function() {
266 });
267
268 it('warns when calling getDOMNode', function() {
269 - spyOn(console, 'warn');
269 + spyOn(console, 'error');
270
271 var container = document.createElement('div');
272 var instance = React.render(<div />, container);
273
274 instance.getDOMNode();
275
276 - expect(console.warn.calls.length).toBe(1);
277 - expect(console.warn.calls[0].args[0]).toContain(
276 + expect(console.error.calls.length).toBe(1);
277 + expect(console.error.calls[0].args[0]).toContain(
278 'DIV.getDOMNode(...) is deprecated. Please use ' +
279 'React.findDOMNode(instance) instead.'
280 );