diff --git a/src/addons/__tests__/ReactFragment-test.js b/src/addons/__tests__/ReactFragment-test.js
index 0f278686a3..4788c06e2f 100644
--- a/src/addons/__tests__/ReactFragment-test.js
+++ b/src/addons/__tests__/ReactFragment-test.js
@@ -75,8 +75,8 @@ describe('ReactFragment', () => {
ReactFragment.create({1: , 2: });
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Child objects should have non-numeric keys so ordering is preserved.'
);
});
@@ -84,8 +84,8 @@ describe('ReactFragment', () => {
it('should warn if passing null to createFragment', () => {
spyOn(console, 'error');
ReactFragment.create(null);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'React.addons.createFragment only accepts a single object.'
);
});
@@ -93,8 +93,8 @@ describe('ReactFragment', () => {
it('should warn if passing an array to createFragment', () => {
spyOn(console, 'error');
ReactFragment.create([]);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'React.addons.createFragment only accepts a single object.'
);
});
@@ -102,8 +102,8 @@ describe('ReactFragment', () => {
it('should warn if passing a ReactElement to createFragment', () => {
spyOn(console, 'error');
ReactFragment.create(
);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'React.addons.createFragment does not accept a ReactElement without a ' +
'wrapper object.'
);
diff --git a/src/addons/transitions/__tests__/ReactCSSTransitionGroup-test.js b/src/addons/transitions/__tests__/ReactCSSTransitionGroup-test.js
index 492b41e2e7..a3ce179fab 100644
--- a/src/addons/transitions/__tests__/ReactCSSTransitionGroup-test.js
+++ b/src/addons/transitions/__tests__/ReactCSSTransitionGroup-test.js
@@ -45,7 +45,7 @@ describe('ReactCSSTransitionGroup', () => {
);
// Warning about the missing transitionLeaveTimeout prop
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
});
it('should not warn if timeouts is zero', () => {
@@ -61,7 +61,7 @@ describe('ReactCSSTransitionGroup', () => {
container
);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('should clean-up silently after the timeout elapses', () => {
@@ -103,7 +103,7 @@ describe('ReactCSSTransitionGroup', () => {
}
// No warnings
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
// The leaving child has been removed
expect(ReactDOM.findDOMNode(a).childNodes.length).toBe(1);
diff --git a/src/addons/transitions/__tests__/ReactTransitionGroup-test.js b/src/addons/transitions/__tests__/ReactTransitionGroup-test.js
index 0729be98f4..12555a98fd 100644
--- a/src/addons/transitions/__tests__/ReactTransitionGroup-test.js
+++ b/src/addons/transitions/__tests__/ReactTransitionGroup-test.js
@@ -21,7 +21,7 @@ describe('ReactTransitionGroup', () => {
var container;
function normalizeCodeLocInfo(str) {
- return str.replace(/\(at .+?:\d+\)/g, '(at **)');
+ return str && str.replace(/\(at .+?:\d+\)/g, '(at **)');
}
beforeEach(() => {
@@ -308,14 +308,14 @@ describe('ReactTransitionGroup', () => {
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(2);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(2);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: flattenChildren(...): ' +
'Encountered two children with the same key, `1`. ' +
'Child keys must be unique; when two children share a key, ' +
'only the first child will be used.'
);
- expect(normalizeCodeLocInfo(console.error.calls.argsFor(1)[0])).toBe(
+ expectDev(normalizeCodeLocInfo(console.error.calls.argsFor(1)[0])).toBe(
'Warning: flattenChildren(...): ' +
'Encountered two children with the same key, `1`. ' +
'Child keys must be unique; when two children share a key, ' +
diff --git a/src/isomorphic/classic/__tests__/ReactContextValidator-test.js b/src/isomorphic/classic/__tests__/ReactContextValidator-test.js
index c0ba47b624..6e237e6537 100644
--- a/src/isomorphic/classic/__tests__/ReactContextValidator-test.js
+++ b/src/isomorphic/classic/__tests__/ReactContextValidator-test.js
@@ -23,7 +23,7 @@ var ReactTestUtils;
describe('ReactContextValidator', () => {
function normalizeCodeLocInfo(str) {
- return str.replace(/\(at .+?:\d+\)/g, '(at **)');
+ return str && str.replace(/\(at .+?:\d+\)/g, '(at **)');
}
beforeEach(() => {
@@ -146,8 +146,8 @@ describe('ReactContextValidator', () => {
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
- expect(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
'Warning: Failed context type: ' +
'The context `foo` is marked as required in `Component`, but its value ' +
'is `undefined`.\n' +
@@ -175,7 +175,7 @@ describe('ReactContextValidator', () => {
);
// Previous call should not error
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
var ComponentInFooNumberContext = React.createClass({
childContextTypes: {
@@ -195,8 +195,8 @@ describe('ReactContextValidator', () => {
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(2);
- expect(normalizeCodeLocInfo(console.error.calls.argsFor(1)[0])).toBe(
+ expectDev(console.error.calls.count()).toBe(2);
+ expectDev(normalizeCodeLocInfo(console.error.calls.argsFor(1)[0])).toBe(
'Warning: Failed context type: ' +
'Invalid context `foo` of type `number` supplied ' +
'to `Component`, expected `string`.\n' +
@@ -224,8 +224,8 @@ describe('ReactContextValidator', () => {
});
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
- expect(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
'Warning: Failed childContext type: ' +
'The child context `foo` is marked as required in `Component`, but its ' +
'value is `undefined`.\n' +
@@ -234,8 +234,8 @@ describe('ReactContextValidator', () => {
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(2);
- expect(normalizeCodeLocInfo(console.error.calls.argsFor(1)[0])).toBe(
+ expectDev(console.error.calls.count()).toBe(2);
+ expectDev(normalizeCodeLocInfo(console.error.calls.argsFor(1)[0])).toBe(
'Warning: Failed childContext type: ' +
'Invalid child context `foo` of type `number` ' +
'supplied to `Component`, expected `string`.\n' +
@@ -251,7 +251,7 @@ describe('ReactContextValidator', () => {
);
// Previous calls should not log errors
- expect(console.error.calls.count()).toBe(2);
+ expectDev(console.error.calls.count()).toBe(2);
});
});
diff --git a/src/isomorphic/classic/class/__tests__/ReactBind-test.js b/src/isomorphic/classic/class/__tests__/ReactBind-test.js
index 5d33f8ed15..4056a39ae4 100644
--- a/src/isomorphic/classic/class/__tests__/ReactBind-test.js
+++ b/src/isomorphic/classic/class/__tests__/ReactBind-test.js
@@ -123,8 +123,8 @@ describe('autobinding', () => {
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: bind(): You are binding a component method to the component. ' +
'React does this for you automatically in a high-performance ' +
'way, so you can safely remove this call. See TestBindComponent'
@@ -151,7 +151,7 @@ describe('autobinding', () => {
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
});
diff --git a/src/isomorphic/classic/class/__tests__/ReactBindOptout-test.js b/src/isomorphic/classic/class/__tests__/ReactBindOptout-test.js
index bfaa8f8e0f..72852f08a3 100644
--- a/src/isomorphic/classic/class/__tests__/ReactBindOptout-test.js
+++ b/src/isomorphic/classic/class/__tests__/ReactBindOptout-test.js
@@ -186,7 +186,7 @@ describe('autobind optout', () => {
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('does not warn if you pass an manually bound method to setState', () => {
@@ -210,7 +210,7 @@ describe('autobind optout', () => {
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
});
diff --git a/src/isomorphic/classic/class/__tests__/ReactClass-test.js b/src/isomorphic/classic/class/__tests__/ReactClass-test.js
index e56d3913ba..eabfd211ee 100644
--- a/src/isomorphic/classic/class/__tests__/ReactClass-test.js
+++ b/src/isomorphic/classic/class/__tests__/ReactClass-test.js
@@ -69,8 +69,8 @@ describe('ReactClass-spec', () => {
return {this.props.prop};
},
});
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: Component: prop type `prop` is invalid; ' +
'it must be a function, usually from React.PropTypes.'
);
@@ -87,8 +87,8 @@ describe('ReactClass-spec', () => {
return {this.props.prop};
},
});
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: Component: context type `prop` is invalid; ' +
'it must be a function, usually from React.PropTypes.'
);
@@ -105,8 +105,8 @@ describe('ReactClass-spec', () => {
return {this.props.prop};
},
});
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: Component: child context type `prop` is invalid; ' +
'it must be a function, usually from React.PropTypes.'
);
@@ -123,8 +123,8 @@ describe('ReactClass-spec', () => {
return ;
},
});
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: A component has a method called componentShouldUpdate(). Did you ' +
'mean shouldComponentUpdate()? The name is phrased as a question ' +
'because the function is expected to return a value.'
@@ -139,8 +139,8 @@ describe('ReactClass-spec', () => {
return ;
},
});
- expect(console.error.calls.count()).toBe(2);
- expect(console.error.calls.argsFor(1)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(2);
+ expectDev(console.error.calls.argsFor(1)[0]).toBe(
'Warning: NamedComponent has a method called componentShouldUpdate(). Did you ' +
'mean shouldComponentUpdate()? The name is phrased as a question ' +
'because the function is expected to return a value.'
@@ -157,8 +157,8 @@ describe('ReactClass-spec', () => {
return ;
},
});
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: A component has a method called componentWillRecieveProps(). Did you ' +
'mean componentWillReceiveProps()?'
);
@@ -206,20 +206,20 @@ describe('ReactClass-spec', () => {
return ;
},
});
- expect(console.error.calls.count()).toBe(4);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(4);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'createClass(...): `mixins` is now a static property and should ' +
'be defined inside "statics".'
);
- expect(console.error.calls.argsFor(1)[0]).toBe(
+ expectDev(console.error.calls.argsFor(1)[0]).toBe(
'createClass(...): `propTypes` is now a static property and should ' +
'be defined inside "statics".'
);
- expect(console.error.calls.argsFor(2)[0]).toBe(
+ expectDev(console.error.calls.argsFor(2)[0]).toBe(
'createClass(...): `contextTypes` is now a static property and ' +
'should be defined inside "statics".'
);
- expect(console.error.calls.argsFor(3)[0]).toBe(
+ expectDev(console.error.calls.argsFor(3)[0]).toBe(
'createClass(...): `childContextTypes` is now a static property and ' +
'should be defined inside "statics".'
);
@@ -343,8 +343,8 @@ describe('ReactClass-spec', () => {
});
expect(() => Component()).toThrow();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: Something is calling a React component directly. Use a ' +
'factory or JSX instead. See: https://fb.me/react-legacyfactory'
);
diff --git a/src/isomorphic/classic/class/__tests__/ReactClassMixin-test.js b/src/isomorphic/classic/class/__tests__/ReactClassMixin-test.js
index 7e80c37f23..df17f8c370 100644
--- a/src/isomorphic/classic/class/__tests__/ReactClassMixin-test.js
+++ b/src/isomorphic/classic/class/__tests__/ReactClassMixin-test.js
@@ -321,8 +321,8 @@ describe('ReactClass-mixin', () => {
},
});
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: ReactClass: You\'re attempting to include a mixin that is ' +
'either null or not an object. Check the mixins included by the ' +
'component, as well as any mixins they include themselves. ' +
@@ -341,8 +341,8 @@ describe('ReactClass-mixin', () => {
},
});
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: ReactClass: You\'re attempting to include a mixin that is ' +
'either null or not an object. Check the mixins included by the ' +
'component, as well as any mixins they include themselves. ' +
@@ -365,8 +365,8 @@ describe('ReactClass-mixin', () => {
},
});
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: ReactClass: You\'re attempting to include a mixin that is ' +
'either null or not an object. Check the mixins included by the ' +
'component, as well as any mixins they include themselves. ' +
@@ -389,8 +389,8 @@ describe('ReactClass-mixin', () => {
},
});
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: ReactClass: You\'re attempting to include a mixin that is ' +
'either null or not an object. Check the mixins included by the ' +
'component, as well as any mixins they include themselves. ' +
diff --git a/src/isomorphic/classic/element/__tests__/ReactElement-test.js b/src/isomorphic/classic/element/__tests__/ReactElement-test.js
index 091e3e502d..80f09d69a6 100644
--- a/src/isomorphic/classic/element/__tests__/ReactElement-test.js
+++ b/src/isomorphic/classic/element/__tests__/ReactElement-test.js
@@ -76,10 +76,10 @@ describe('ReactElement', () => {
);
},
});
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Child: `key` is not a prop. Trying to access it will result ' +
'in `undefined` being returned. If you need to access the same ' +
'value within the child component, you should pass it as a different ' +
@@ -106,10 +106,10 @@ describe('ReactElement', () => {
);
},
});
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Child: `key` is not a prop. Trying to access it will result ' +
'in `undefined` being returned. If you need to access the same ' +
'value within the child component, you should pass it as a different ' +
@@ -120,10 +120,10 @@ describe('ReactElement', () => {
it('should warn when `key` is being accessed on a host element', () => {
spyOn(console, 'error');
var element = ;
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
void element.props.key;
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'div: `key` is not a prop. Trying to access it will result ' +
'in `undefined` being returned. If you need to access the same ' +
'value within the child component, you should pass it as a different ' +
@@ -148,10 +148,10 @@ describe('ReactElement', () => {
);
},
});
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Child: `ref` is not a prop. Trying to access it will result ' +
'in `undefined` being returned. If you need to access the same ' +
'value within the child component, you should pass it as a different ' +
@@ -280,7 +280,7 @@ describe('ReactElement', () => {
children: 'text',
}, a);
expect(element.props.children).toBe(a);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('does not override children if no rest args are provided', () => {
@@ -289,7 +289,7 @@ describe('ReactElement', () => {
children: 'text',
});
expect(element.props.children).toBe('text');
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('overrides children if null is provided as an argument', () => {
@@ -298,7 +298,7 @@ describe('ReactElement', () => {
children: 'text',
}, null);
expect(element.props.children).toBe(null);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('merges rest arguments onto the children prop in an array', () => {
@@ -308,7 +308,7 @@ describe('ReactElement', () => {
var c = 3;
var element = React.createFactory(ComponentClass)(null, a, b, c);
expect(element.props.children).toEqual([1, 2, 3]);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
// NOTE: We're explicitly not using JSX here. This is intended to test
@@ -332,7 +332,7 @@ describe('ReactElement', () => {
var element = React.createElement(StaticMethodComponentClass);
expect(element.type.someStaticMethod()).toBe('someReturnValue');
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
// NOTE: We're explicitly not using JSX here. This is intended to test
@@ -477,7 +477,7 @@ describe('ReactElement', () => {
});
var test = ReactTestUtils.renderIntoDocument();
expect(test.props.value).toBeNaN();
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
// NOTE: We're explicitly not using JSX here. This is intended to test
diff --git a/src/isomorphic/classic/element/__tests__/ReactElementClone-test.js b/src/isomorphic/classic/element/__tests__/ReactElementClone-test.js
index 50406d0345..49af3ea80e 100644
--- a/src/isomorphic/classic/element/__tests__/ReactElementClone-test.js
+++ b/src/isomorphic/classic/element/__tests__/ReactElementClone-test.js
@@ -250,8 +250,8 @@ describe('ReactElementClone', () => {
React.cloneElement(, null, [, ]);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Each child in an array or iterator should have a unique "key" prop.'
);
});
@@ -261,7 +261,7 @@ describe('ReactElementClone', () => {
React.cloneElement(, null, [, ]);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('does not warn when the element is directly in rest args', () => {
@@ -269,7 +269,7 @@ describe('ReactElementClone', () => {
React.cloneElement(, null, , );
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('does not warn when the array contains a non-element', () => {
@@ -277,7 +277,7 @@ describe('ReactElementClone', () => {
React.cloneElement(, null, [{}, {}]);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('should check declared prop types after clone', () => {
@@ -304,8 +304,8 @@ describe('ReactElementClone', () => {
},
});
ReactTestUtils.renderIntoDocument(React.createElement(GrandParent));
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: Failed prop type: ' +
'Invalid prop `color` of type `number` supplied to `Component`, ' +
'expected `string`.\n' +
diff --git a/src/isomorphic/classic/element/__tests__/ReactElementValidator-test.js b/src/isomorphic/classic/element/__tests__/ReactElementValidator-test.js
index 5690e2125b..11dfb266b9 100644
--- a/src/isomorphic/classic/element/__tests__/ReactElementValidator-test.js
+++ b/src/isomorphic/classic/element/__tests__/ReactElementValidator-test.js
@@ -20,7 +20,7 @@ var ReactTestUtils;
describe('ReactElementValidator', () => {
function normalizeCodeLocInfo(str) {
- return str.replace(/\(at .+?:\d+\)/g, '(at **)');
+ return str && str.replace(/\(at .+?:\d+\)/g, '(at **)');
}
var ComponentClass;
@@ -44,8 +44,8 @@ describe('ReactElementValidator', () => {
Component(null, [Component(), Component()]);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Each child in an array or iterator should have a unique "key" prop.'
);
});
@@ -74,8 +74,8 @@ describe('ReactElementValidator', () => {
React.createElement(ComponentWrapper)
);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Each child in an array or iterator should have a unique "key" prop. ' +
'Check the render method of `InnerClass`. ' +
'It was passed a child from ComponentWrapper. '
@@ -98,8 +98,8 @@ describe('ReactElementValidator', () => {
];
ReactTestUtils.renderIntoDocument({divs});
- expect(console.error.calls.count()).toBe(1);
- expect(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
'Warning: Each child in an array or iterator should have a unique ' +
'"key" prop. See https://fb.me/react-warning-keys for more information.\n' +
' in div (at **)'
@@ -115,8 +115,8 @@ describe('ReactElementValidator', () => {
];
ReactTestUtils.renderIntoDocument(
{divs}
);
- expect(console.error.calls.count()).toBe(1);
- expect(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
'Warning: Each child in an array or iterator should have a unique ' +
'"key" prop. Check the top-level render call using
. See ' +
'https://fb.me/react-warning-keys for more information.\n' +
@@ -147,8 +147,8 @@ describe('ReactElementValidator', () => {
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
- expect(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
'Warning: Each child in an array or iterator should have a unique ' +
'"key" prop. Check the render method of `Component`. See ' +
'https://fb.me/react-warning-keys for more information.\n' +
@@ -180,7 +180,7 @@ describe('ReactElementValidator', () => {
);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('warns for keys for iterables of elements in rest args', () => {
@@ -201,8 +201,8 @@ describe('ReactElementValidator', () => {
Component(null, iterable);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Each child in an array or iterator should have a unique "key" prop.'
);
});
@@ -213,7 +213,7 @@ describe('ReactElementValidator', () => {
Component(null, [Component({key: '#1'}), Component({key: '#2'})]);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('does not warns for iterable elements with keys', () => {
@@ -237,7 +237,7 @@ describe('ReactElementValidator', () => {
Component(null, iterable);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('does not warn when the element is directly in rest args', () => {
@@ -246,7 +246,7 @@ describe('ReactElementValidator', () => {
Component(null, Component(), Component());
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('does not warn when the array contains a non-element', () => {
@@ -255,7 +255,7 @@ describe('ReactElementValidator', () => {
Component(null, [{}, {}]);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
// TODO: These warnings currently come from the composite component, but
@@ -280,7 +280,7 @@ describe('ReactElementValidator', () => {
},
});
ReactTestUtils.renderIntoDocument(React.createElement(ParentComp));
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: Failed prop type: ' +
'Invalid prop `color` of type `number` supplied to `MyComp`, ' +
'expected `string`.\n' +
@@ -295,29 +295,29 @@ describe('ReactElementValidator', () => {
React.createElement(null);
React.createElement(true);
React.createElement(123);
- expect(console.error.calls.count()).toBe(4);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(4);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: React.createElement: type should not be null, undefined, ' +
'boolean, or number. It should be a string (for DOM elements) or a ' +
'ReactClass (for composite components).'
);
- expect(console.error.calls.argsFor(1)[0]).toBe(
+ expectDev(console.error.calls.argsFor(1)[0]).toBe(
'Warning: React.createElement: type should not be null, undefined, ' +
'boolean, or number. It should be a string (for DOM elements) or a ' +
'ReactClass (for composite components).'
);
- expect(console.error.calls.argsFor(2)[0]).toBe(
+ expectDev(console.error.calls.argsFor(2)[0]).toBe(
'Warning: React.createElement: type should not be null, undefined, ' +
'boolean, or number. It should be a string (for DOM elements) or a ' +
'ReactClass (for composite components).'
);
- expect(console.error.calls.argsFor(3)[0]).toBe(
+ expectDev(console.error.calls.argsFor(3)[0]).toBe(
'Warning: React.createElement: type should not be null, undefined, ' +
'boolean, or number. It should be a string (for DOM elements) or a ' +
'ReactClass (for composite components).'
);
React.createElement('div');
- expect(console.error.calls.count()).toBe(4);
+ expectDev(console.error.calls.count()).toBe(4);
});
it('includes the owner name when passing null, undefined, boolean, or number', () => {
@@ -334,8 +334,8 @@ describe('ReactElementValidator', () => {
'or a class/function (for composite components) but got: null. Check ' +
'the render method of `ParentComp`.'
);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: React.createElement: type should not be null, undefined, ' +
'boolean, or number. It should be a string (for DOM elements) or a ' +
'ReactClass (for composite components). Check the render method of ' +
@@ -358,8 +358,8 @@ describe('ReactElementValidator', () => {
ReactTestUtils.renderIntoDocument(React.createElement(Component));
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: Failed prop type: The prop `prop` is marked as required in ' +
'`Component`, but its value is `null`.\n' +
' in Component'
@@ -383,8 +383,8 @@ describe('ReactElementValidator', () => {
React.createElement(Component, {prop:null})
);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: Failed prop type: The prop `prop` is marked as required in ' +
'`Component`, but its value is `null`.\n' +
' in Component'
@@ -410,15 +410,15 @@ describe('ReactElementValidator', () => {
React.createElement(Component, {prop: 42})
);
- expect(console.error.calls.count()).toBe(2);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(2);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: Failed prop type: ' +
'The prop `prop` is marked as required in `Component`, but its value ' +
'is `undefined`.\n' +
' in Component'
);
- expect(console.error.calls.argsFor(1)[0]).toBe(
+ expectDev(console.error.calls.argsFor(1)[0]).toBe(
'Warning: Failed prop type: ' +
'Invalid prop `prop` of type `number` supplied to ' +
'`Component`, expected `string`.\n' +
@@ -430,7 +430,7 @@ describe('ReactElementValidator', () => {
);
// Should not error for strings
- expect(console.error.calls.count()).toBe(2);
+ expectDev(console.error.calls.count()).toBe(2);
});
it('should warn if a PropType creator is used as a PropType', () => {
@@ -449,8 +449,8 @@ describe('ReactElementValidator', () => {
React.createElement(Component, {myProp: {value: 'hi'}})
);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: Component: type specification of prop `myProp` is invalid; ' +
'the type checker function must return `null` or an `Error` but ' +
'returned a function. You may have forgotten to pass an argument to ' +
@@ -468,14 +468,14 @@ describe('ReactElementValidator', () => {
});
var TestFactory = React.createFactory(TestComponent);
expect(TestFactory.type).toBe(TestComponent);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: Factory.type is deprecated. Access the class directly before ' +
'passing it to createFactory.'
);
// Warn once, not again
expect(TestFactory.type).toBe(TestComponent);
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
});
it('does not warn when using DOM node as children', () => {
@@ -492,7 +492,7 @@ describe('ReactElementValidator', () => {
var node = document.createElement('div');
// This shouldn't cause a stack overflow or any other problems (#3883)
ReactTestUtils.renderIntoDocument({node});
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('should not enumerate enumerable numbers (#4776)', () => {
@@ -535,8 +535,8 @@ describe('ReactElementValidator', () => {
spyOn(console, 'error');
var Foo = undefined;
void {[]};
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: React.createElement: type should not be null, undefined, ' +
'boolean, or number. It should be a string (for DOM elements) or a ' +
'ReactClass (for composite components).'
diff --git a/src/isomorphic/classic/types/__tests__/ReactPropTypes-test.js b/src/isomorphic/classic/types/__tests__/ReactPropTypes-test.js
index fa51c97ccb..18a57c7be1 100644
--- a/src/isomorphic/classic/types/__tests__/ReactPropTypes-test.js
+++ b/src/isomorphic/classic/types/__tests__/ReactPropTypes-test.js
@@ -310,7 +310,7 @@ describe('ReactPropTypes', () => {
var instance = } />;
instance = ReactTestUtils.renderIntoDocument(instance);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('should warn when passing no label and isRequired is set', () => {
@@ -319,7 +319,7 @@ describe('ReactPropTypes', () => {
var instance = ;
instance = ReactTestUtils.renderIntoDocument(instance);
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
});
it('should be implicitly optional and not warn without values', () => {
@@ -454,7 +454,7 @@ describe('ReactPropTypes', () => {
k4: null,
k5: undefined,
}));
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('should not warn for iterables', () => {
@@ -612,8 +612,8 @@ describe('ReactPropTypes', () => {
PropTypes.oneOf('red', 'blue');
- expect(console.error).toHaveBeenCalled();
- expect(console.error.calls.argsFor(0)[0])
+ expectDev(console.error).toHaveBeenCalled();
+ expectDev(console.error.calls.argsFor(0)[0])
.toContain('Invalid argument supplied to oneOf, expected an instance of array.');
typeCheckPass(PropTypes.oneOf('red', 'blue'), 'red');
@@ -668,8 +668,8 @@ describe('ReactPropTypes', () => {
PropTypes.oneOfType(PropTypes.string, PropTypes.number);
- expect(console.error).toHaveBeenCalled();
- expect(console.error.calls.argsFor(0)[0])
+ expectDev(console.error).toHaveBeenCalled();
+ expectDev(console.error.calls.argsFor(0)[0])
.toContain('Invalid argument supplied to oneOfType, expected an instance of array.');
typeCheckPass(PropTypes.oneOf(PropTypes.string, PropTypes.number), []);
@@ -893,7 +893,7 @@ describe('ReactPropTypes', () => {
var instance = ;
instance = ReactTestUtils.renderIntoDocument(instance);
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
expect(
console.error.calls.argsFor(0)[0].replace(/\(at .+?:\d+\)/g, '(at **)')
).toBe(
@@ -920,7 +920,7 @@ describe('ReactPropTypes', () => {
var instance = ;
instance = ReactTestUtils.renderIntoDocument(instance);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
}
);
});
diff --git a/src/isomorphic/modern/element/__tests__/ReactJSXElement-test.js b/src/isomorphic/modern/element/__tests__/ReactJSXElement-test.js
index 1a7be9d7b0..84970acf8c 100644
--- a/src/isomorphic/modern/element/__tests__/ReactJSXElement-test.js
+++ b/src/isomorphic/modern/element/__tests__/ReactJSXElement-test.js
@@ -100,21 +100,21 @@ describe('ReactJSXElement', () => {
var a = 1;
var element = {a};
expect(element.props.children).toBe(a);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('does not override children if no JSX children are provided', () => {
spyOn(console, 'error');
var element = ;
expect(element.props.children).toBe('text');
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('overrides children if null is provided as a JSX child', () => {
spyOn(console, 'error');
var element = {null};
expect(element.props.children).toBe(null);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('overrides children if undefined is provided as an argument', () => {
@@ -136,7 +136,7 @@ describe('ReactJSXElement', () => {
var c = 3;
var element = {a}{b}{c};
expect(element.props.children).toEqual([1, 2, 3]);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('allows static methods to be called using the type property', () => {
@@ -153,7 +153,7 @@ describe('ReactJSXElement', () => {
var element = ;
expect(element.type.someStaticMethod()).toBe('someReturnValue');
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('identifies valid elements', () => {
diff --git a/src/isomorphic/modern/element/__tests__/ReactJSXElementValidator-test.js b/src/isomorphic/modern/element/__tests__/ReactJSXElementValidator-test.js
index 71fbed5bd3..47563ff67b 100644
--- a/src/isomorphic/modern/element/__tests__/ReactJSXElementValidator-test.js
+++ b/src/isomorphic/modern/element/__tests__/ReactJSXElementValidator-test.js
@@ -47,8 +47,8 @@ describe('ReactJSXElementValidator', () => {
void {[, ]};
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Each child in an array or iterator should have a unique "key" prop.'
);
});
@@ -74,8 +74,8 @@ describe('ReactJSXElementValidator', () => {
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Each child in an array or iterator should have a unique "key" prop. ' +
'Check the render method of `InnerComponent`. ' +
'It was passed a child from ComponentWrapper. '
@@ -99,8 +99,8 @@ describe('ReactJSXElementValidator', () => {
void {iterable};
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Each child in an array or iterator should have a unique "key" prop.'
);
});
@@ -110,7 +110,7 @@ describe('ReactJSXElementValidator', () => {
void {[, ]};
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('does not warns for iterable elements with keys', () => {
@@ -133,7 +133,7 @@ describe('ReactJSXElementValidator', () => {
void {iterable};
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('does not warn for numeric keys in entry iterable as a child', () => {
@@ -154,7 +154,7 @@ describe('ReactJSXElementValidator', () => {
void {iterable};
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('does not warn when the element is directly as children', () => {
@@ -162,7 +162,7 @@ describe('ReactJSXElementValidator', () => {
void ;
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('does not warn when the child array contains non-elements', () => {
@@ -170,7 +170,7 @@ describe('ReactJSXElementValidator', () => {
void {[{}, {}]};
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
// TODO: These warnings currently come from the composite component, but
@@ -217,25 +217,25 @@ describe('ReactJSXElementValidator', () => {
void ;
void ;
void ;
- expect(console.error.calls.count()).toBe(4);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(4);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'type should not be null, undefined, boolean, or number. It should be ' +
'a string (for DOM elements) or a ReactClass (for composite components).'
);
- expect(console.error.calls.argsFor(1)[0]).toContain(
+ expectDev(console.error.calls.argsFor(1)[0]).toContain(
'type should not be null, undefined, boolean, or number. It should be ' +
'a string (for DOM elements) or a ReactClass (for composite components).'
);
- expect(console.error.calls.argsFor(2)[0]).toContain(
+ expectDev(console.error.calls.argsFor(2)[0]).toContain(
'type should not be null, undefined, boolean, or number. It should be ' +
'a string (for DOM elements) or a ReactClass (for composite components).'
);
- expect(console.error.calls.argsFor(3)[0]).toContain(
+ expectDev(console.error.calls.argsFor(3)[0]).toContain(
'type should not be null, undefined, boolean, or number. It should be ' +
'a string (for DOM elements) or a ReactClass (for composite components).'
);
void ;
- expect(console.error.calls.count()).toBe(4);
+ expectDev(console.error.calls.count()).toBe(4);
});
it('should check default prop values', () => {
@@ -245,7 +245,7 @@ describe('ReactJSXElementValidator', () => {
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
expect(
console.error.calls.argsFor(0)[0].replace(/\(at .+?:\d+\)/g, '(at **)')
).toBe(
@@ -260,7 +260,7 @@ describe('ReactJSXElementValidator', () => {
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
expect(
console.error.calls.argsFor(0)[0].replace(/\(at .+?:\d+\)/g, '(at **)')
).toBe(
@@ -276,7 +276,7 @@ describe('ReactJSXElementValidator', () => {
ReactTestUtils.renderIntoDocument();
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(2);
+ expectDev(console.error.calls.count()).toBe(2);
expect(
console.error.calls.argsFor(0)[0].replace(/\(at .+?:\d+\)/g, '(at **)')
).toBe(
@@ -298,7 +298,7 @@ describe('ReactJSXElementValidator', () => {
ReactTestUtils.renderIntoDocument();
// Should not error for strings
- expect(console.error.calls.count()).toBe(2);
+ expectDev(console.error.calls.count()).toBe(2);
});
it('should warn on invalid prop types', () => {
@@ -316,8 +316,8 @@ describe('ReactJSXElementValidator', () => {
prop: null,
};
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'NullPropTypeComponent: prop type `prop` is invalid; it must be a ' +
'function, usually from React.PropTypes.'
);
@@ -334,8 +334,8 @@ describe('ReactJSXElementValidator', () => {
prop: null,
};
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'NullContextTypeComponent: context type `prop` is invalid; it must ' +
'be a function, usually from React.PropTypes.'
);
@@ -352,8 +352,8 @@ describe('ReactJSXElementValidator', () => {
prop: 'foo',
});
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'getDefaultProps is only used on classic React.createClass definitions.' +
' Use a static property named `defaultProps` instead.'
);
diff --git a/src/renderers/dom/__tests__/ReactDOMProduction-test.js b/src/renderers/dom/__tests__/ReactDOMProduction-test.js
index 2033abd1ad..4699975e36 100644
--- a/src/renderers/dom/__tests__/ReactDOMProduction-test.js
+++ b/src/renderers/dom/__tests__/ReactDOMProduction-test.js
@@ -36,7 +36,7 @@ describe('ReactDOMProduction', () => {
spyOn(console, 'error');
warning(false, 'Do cows go moo?');
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('should use prod React', () => {
@@ -45,7 +45,7 @@ describe('ReactDOMProduction', () => {
// no key warning
void
{[]}
;
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('should handle a simple flow', () => {
diff --git a/src/renderers/dom/shared/__tests__/CSSPropertyOperations-test.js b/src/renderers/dom/shared/__tests__/CSSPropertyOperations-test.js
index e0133bcbf7..4acf6b3363 100644
--- a/src/renderers/dom/shared/__tests__/CSSPropertyOperations-test.js
+++ b/src/renderers/dom/shared/__tests__/CSSPropertyOperations-test.js
@@ -119,8 +119,8 @@ describe('CSSPropertyOperations', () => {
spyOn(console, 'error');
var root = document.createElement('div');
ReactDOM.render(, root);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toEqual(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toEqual(
'Warning: Unsupported style property background-color. Did you mean backgroundColor? ' +
'Check the render method of `Comp`.'
);
@@ -144,12 +144,12 @@ describe('CSSPropertyOperations', () => {
ReactDOM.render(, root);
ReactDOM.render(, root);
- expect(console.error.calls.count()).toBe(2);
- expect(console.error.calls.argsFor(0)[0]).toEqual(
+ expectDev(console.error.calls.count()).toBe(2);
+ expectDev(console.error.calls.argsFor(0)[0]).toEqual(
'Warning: Unsupported style property -ms-transform. Did you mean msTransform? ' +
'Check the render method of `Comp`.'
);
- expect(console.error.calls.argsFor(1)[0]).toEqual(
+ expectDev(console.error.calls.argsFor(1)[0]).toEqual(
'Warning: Unsupported style property -webkit-transform. Did you mean WebkitTransform? ' +
'Check the render method of `Comp`.'
);
@@ -172,12 +172,12 @@ describe('CSSPropertyOperations', () => {
var root = document.createElement('div');
ReactDOM.render(, root);
// msTransform is correct already and shouldn't warn
- expect(console.error.calls.count()).toBe(2);
- expect(console.error.calls.argsFor(0)[0]).toEqual(
+ expectDev(console.error.calls.count()).toBe(2);
+ expectDev(console.error.calls.argsFor(0)[0]).toEqual(
'Warning: Unsupported vendor-prefixed style property oTransform. ' +
'Did you mean OTransform? Check the render method of `Comp`.'
);
- expect(console.error.calls.argsFor(1)[0]).toEqual(
+ expectDev(console.error.calls.argsFor(1)[0]).toEqual(
'Warning: Unsupported vendor-prefixed style property webkitTransform. ' +
'Did you mean WebkitTransform? Check the render method of `Comp`.'
);
@@ -200,12 +200,12 @@ describe('CSSPropertyOperations', () => {
spyOn(console, 'error');
var root = document.createElement('div');
ReactDOM.render(, root);
- expect(console.error.calls.count()).toBe(2);
- expect(console.error.calls.argsFor(0)[0]).toEqual(
+ expectDev(console.error.calls.count()).toBe(2);
+ expectDev(console.error.calls.argsFor(0)[0]).toEqual(
'Warning: Style property values shouldn\'t contain a semicolon. ' +
'Check the render method of `Comp`. Try "backgroundColor: blue" instead.',
);
- expect(console.error.calls.argsFor(1)[0]).toEqual(
+ expectDev(console.error.calls.argsFor(1)[0]).toEqual(
'Warning: Style property values shouldn\'t contain a semicolon. ' +
'Check the render method of `Comp`. Try "color: red" instead.',
);
@@ -224,8 +224,8 @@ describe('CSSPropertyOperations', () => {
var root = document.createElement('div');
ReactDOM.render(, root);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toEqual(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toEqual(
'Warning: `NaN` is an invalid value for the `fontSize` css style property. ' +
'Check the render method of `Comp`.'
);
diff --git a/src/renderers/dom/shared/__tests__/ReactBrowserEventEmitter-test.js b/src/renderers/dom/shared/__tests__/ReactBrowserEventEmitter-test.js
index d76d4868a3..e4171e3d97 100644
--- a/src/renderers/dom/shared/__tests__/ReactBrowserEventEmitter-test.js
+++ b/src/renderers/dom/shared/__tests__/ReactBrowserEventEmitter-test.js
@@ -269,7 +269,7 @@ describe('ReactBrowserEventEmitter', () => {
expect(idCallOrder[0]).toBe(getInternal(CHILD));
expect(idCallOrder[1]).toBe(getInternal(PARENT));
expect(idCallOrder[2]).toBe(getInternal(GRANDPARENT));
- expect(console.error.calls.count()).toEqual(0);
+ expectDev(console.error.calls.count()).toEqual(0);
});
/**
diff --git a/src/renderers/dom/shared/__tests__/ReactDOMComponent-test.js b/src/renderers/dom/shared/__tests__/ReactDOMComponent-test.js
index 00d0bd445d..657472cc17 100644
--- a/src/renderers/dom/shared/__tests__/ReactDOMComponent-test.js
+++ b/src/renderers/dom/shared/__tests__/ReactDOMComponent-test.js
@@ -20,7 +20,7 @@ describe('ReactDOMComponent', () => {
var inputValueTracking;
function normalizeCodeLocInfo(str) {
- return str.replace(/\(at .+?:\d+\)/g, '(at **)');
+ return str && str.replace(/\(at .+?:\d+\)/g, '(at **)');
}
beforeEach(() => {
@@ -128,8 +128,8 @@ describe('ReactDOMComponent', () => {
var stub = ReactTestUtils.renderIntoDocument();
style.position = 'absolute';
stub.setState({style: style});
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toEqual(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toEqual(
'Warning: `div` was passed a style object that has previously been ' +
'mutated. Mutating `style` is deprecated. Consider cloning it ' +
'beforehand. Check the `render` of `App`. Previous style: ' +
@@ -142,22 +142,22 @@ describe('ReactDOMComponent', () => {
style.background = 'green';
stub.setState({style: {background: 'green'}});
// already warned once for the same component and owner
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
style = {background: 'red'};
var div = document.createElement('div');
ReactDOM.render(, div);
style.background = 'blue';
ReactDOM.render(, div);
- expect(console.error.calls.count()).toBe(2);
+ expectDev(console.error.calls.count()).toBe(2);
});
it('should warn for unknown prop', () => {
spyOn(console, 'error');
var container = document.createElement('div');
ReactDOM.render(, container);
- expect(console.error.calls.count(0)).toBe(1);
- expect(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
+ expectDev(console.error.calls.count(0)).toBe(1);
+ expectDev(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
'Warning: Unknown prop `foo` on
tag. Remove this prop from the element. ' +
'For details, see https://fb.me/react-unknown-prop\n in div (at **)'
);
@@ -167,8 +167,8 @@ describe('ReactDOMComponent', () => {
spyOn(console, 'error');
var container = document.createElement('div');
ReactDOM.render(, container);
- expect(console.error.calls.count(0)).toBe(1);
- expect(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
+ expectDev(console.error.calls.count(0)).toBe(1);
+ expectDev(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
'Warning: Unknown props `foo`, `baz` on
tag. Remove these props from the element. ' +
'For details, see https://fb.me/react-unknown-prop\n in div (at **)'
);
@@ -178,8 +178,8 @@ describe('ReactDOMComponent', () => {
spyOn(console, 'error');
var container = document.createElement('div');
ReactDOM.render(
. See Foo > table > Row > tr. Add a to your code to ' +
'match the DOM tree generated by the browser.'
);
- expect(console.error.calls.argsFor(1)[0]).toBe(
+ expectDev(console.error.calls.argsFor(1)[0]).toBe(
'Warning: validateDOMNesting(...): Text nodes cannot appear as a ' +
'child of
. See Row > tr > #text.'
);
- expect(console.error.calls.argsFor(2)[0]).toBe(
+ expectDev(console.error.calls.argsFor(2)[0]).toBe(
'Warning: validateDOMNesting(...): Whitespace text nodes cannot ' +
'appear as a child of
. Make sure you don\'t have any extra ' +
'whitespace between tags on each line of your source code. See Foo > ' +
@@ -1365,8 +1365,8 @@ describe('ReactDOMComponent', () => {
render: () => ,
});
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'See Viz1 > table > FancyRow > Row > tr.'
);
@@ -1377,26 +1377,26 @@ describe('ReactDOMComponent', () => {
render: () => ,
});
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(2);
- expect(console.error.calls.argsFor(1)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(2);
+ expectDev(console.error.calls.argsFor(1)[0]).toContain(
'See Viz2 > FancyTable > Table > table > FancyRow > Row > tr.'
);
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(3);
- expect(console.error.calls.argsFor(2)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(3);
+ expectDev(console.error.calls.argsFor(2)[0]).toContain(
'See FancyTable > Table > table > FancyRow > Row > tr.'
);
ReactTestUtils.renderIntoDocument(
' +
'tag. For details, see https://fb.me/invalid-aria-prop'
);
@@ -59,8 +59,8 @@ describe('ReactDOMInvalidARIAHook', () => {
spyOn(console, 'error');
// The valid attribute name is aria-haspopup.
mountComponent({'aria-hasPopup': 'true'});
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Warning: Unknown ARIA attribute aria-hasPopup. ' +
'Did you mean aria-haspopup?'
);
diff --git a/src/renderers/dom/shared/syntheticEvents/__tests__/SyntheticEvent-test.js b/src/renderers/dom/shared/syntheticEvents/__tests__/SyntheticEvent-test.js
index f54d45411e..b1ca2d61e1 100644
--- a/src/renderers/dom/shared/syntheticEvents/__tests__/SyntheticEvent-test.js
+++ b/src/renderers/dom/shared/syntheticEvents/__tests__/SyntheticEvent-test.js
@@ -87,9 +87,9 @@ describe('SyntheticEvent', () => {
expect(syntheticEvent.nativeEvent).toBe(null);
expect(syntheticEvent.target).toBe(null);
// once for each property accessed
- expect(console.error.calls.count()).toBe(3);
+ expectDev(console.error.calls.count()).toBe(3);
// assert the first warning for accessing `type`
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: This synthetic event is reused for performance reasons. If ' +
'you\'re seeing this, you\'re accessing the property `type` on a ' +
'released/nullified synthetic event. This is set to null. If you must ' +
@@ -104,8 +104,8 @@ describe('SyntheticEvent', () => {
var syntheticEvent = createEvent({srcElement: target});
syntheticEvent.destructor();
expect(syntheticEvent.type = 'MouseEvent').toBe('MouseEvent');
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: This synthetic event is reused for performance reasons. If ' +
'you\'re seeing this, you\'re setting the property `type` on a ' +
'released/nullified synthetic event. This is effectively a no-op. If you must ' +
@@ -119,8 +119,8 @@ describe('SyntheticEvent', () => {
var syntheticEvent = createEvent({});
SyntheticEvent.release(syntheticEvent);
syntheticEvent.preventDefault();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: This synthetic event is reused for performance reasons. If ' +
'you\'re seeing this, you\'re accessing the method `preventDefault` on a ' +
'released/nullified synthetic event. This is a no-op function. If you must ' +
@@ -134,8 +134,8 @@ describe('SyntheticEvent', () => {
var syntheticEvent = createEvent({});
SyntheticEvent.release(syntheticEvent);
syntheticEvent.stopPropagation();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: This synthetic event is reused for performance reasons. If ' +
'you\'re seeing this, you\'re accessing the method `stopPropagation` on a ' +
'released/nullified synthetic event. This is a no-op function. If you must ' +
@@ -156,13 +156,13 @@ describe('SyntheticEvent', () => {
}
var instance = ReactDOM.render(, element);
ReactTestUtils.Simulate.click(ReactDOM.findDOMNode(instance));
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
// access a property to cause the warning
event.nativeEvent; // eslint-disable-line no-unused-expressions
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: This synthetic event is reused for performance reasons. If ' +
'you\'re seeing this, you\'re accessing the property `nativeEvent` on a ' +
'released/nullified synthetic event. This is set to null. If you must ' +
@@ -178,15 +178,15 @@ describe('SyntheticEvent', () => {
SyntheticEvent.release(syntheticEvent);
expect(syntheticEvent.foo).toBe('bar');
if (typeof Proxy === 'function') {
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: This synthetic event is reused for performance reasons. If ' +
'you\'re seeing this, you\'re adding a new property in the synthetic ' +
'event object. The property is never released. ' +
'See https://fb.me/react-event-pooling for more information.'
);
} else {
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
}
});
});
diff --git a/src/renderers/dom/shared/wrappers/__tests__/ReactDOMInput-test.js b/src/renderers/dom/shared/wrappers/__tests__/ReactDOMInput-test.js
index efc315b2f5..e17f8e1142 100644
--- a/src/renderers/dom/shared/wrappers/__tests__/ReactDOMInput-test.js
+++ b/src/renderers/dom/shared/wrappers/__tests__/ReactDOMInput-test.js
@@ -50,7 +50,7 @@ describe('ReactDOMInput', () => {
document.body.appendChild(container);
var node = ReactDOM.findDOMNode(stub);
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
// Simulate a native change event
setUntrackedValue(node, 'giraffe');
@@ -593,12 +593,12 @@ describe('ReactDOMInput', () => {
ReactTestUtils.renderIntoDocument(
);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
ReactTestUtils.renderIntoDocument(
);
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
});
@@ -617,12 +617,12 @@ describe('ReactDOMInput', () => {
ReactTestUtils.renderIntoDocument(
);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
ReactTestUtils.renderIntoDocument(
);
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
});
it('should update defaultValue to empty string', () => {
@@ -634,21 +634,21 @@ describe('ReactDOMInput', () => {
it('should warn if value is null', () => {
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'`value` prop on `input` should not be null. ' +
'Consider using the empty string to clear the component or `undefined` ' +
'for uncontrolled components.'
);
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
});
it('should warn if checked and defaultChecked props are specified', () => {
ReactTestUtils.renderIntoDocument(
);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'A component contains an input of type radio with both checked and defaultChecked props. ' +
'Input elements must be either controlled or uncontrolled ' +
'(specify either the checked prop, or the defaultChecked prop, but not ' +
@@ -660,14 +660,14 @@ describe('ReactDOMInput', () => {
ReactTestUtils.renderIntoDocument(
);
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
});
it('should warn if value and defaultValue props are specified', () => {
ReactTestUtils.renderIntoDocument(
);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'A component contains an input of type text with both value and defaultValue props. ' +
'Input elements must be either controlled or uncontrolled ' +
'(specify either the value prop, or the defaultValue prop, but not ' +
@@ -679,7 +679,7 @@ describe('ReactDOMInput', () => {
ReactTestUtils.renderIntoDocument(
);
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
});
it('should warn if controlled input switches to uncontrolled (value is undefined)', () => {
@@ -687,8 +687,8 @@ describe('ReactDOMInput', () => {
var container = document.createElement('div');
ReactDOM.render(stub, container);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'A component is changing a controlled input of type text to be uncontrolled. ' +
'Input elements should not switch from controlled to uncontrolled (or vice versa). ' +
'Decide between using a controlled or uncontrolled input ' +
@@ -701,8 +701,8 @@ describe('ReactDOMInput', () => {
var container = document.createElement('div');
ReactDOM.render(stub, container);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBeGreaterThan(0);
- expect(console.error.calls.argsFor(1)[0]).toContain(
+ expectDev(console.error.calls.count()).toBeGreaterThan(0);
+ expectDev(console.error.calls.argsFor(1)[0]).toContain(
'A component is changing a controlled input of type text to be uncontrolled. ' +
'Input elements should not switch from controlled to uncontrolled (or vice versa). ' +
'Decide between using a controlled or uncontrolled input ' +
@@ -715,8 +715,8 @@ describe('ReactDOMInput', () => {
var container = document.createElement('div');
ReactDOM.render(stub, container);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'A component is changing a controlled input of type text to be uncontrolled. ' +
'Input elements should not switch from controlled to uncontrolled (or vice versa). ' +
'Decide between using a controlled or uncontrolled input ' +
@@ -729,8 +729,8 @@ describe('ReactDOMInput', () => {
var container = document.createElement('div');
ReactDOM.render(stub, container);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'A component is changing an uncontrolled input of type text to be controlled. ' +
'Input elements should not switch from uncontrolled to controlled (or vice versa). ' +
'Decide between using a controlled or uncontrolled input ' +
@@ -743,8 +743,8 @@ describe('ReactDOMInput', () => {
var container = document.createElement('div');
ReactDOM.render(stub, container);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBeGreaterThan(0);
- expect(console.error.calls.argsFor(1)[0]).toContain(
+ expectDev(console.error.calls.count()).toBeGreaterThan(0);
+ expectDev(console.error.calls.argsFor(1)[0]).toContain(
'A component is changing an uncontrolled input of type text to be controlled. ' +
'Input elements should not switch from uncontrolled to controlled (or vice versa). ' +
'Decide between using a controlled or uncontrolled input ' +
@@ -757,8 +757,8 @@ describe('ReactDOMInput', () => {
var container = document.createElement('div');
ReactDOM.render(stub, container);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'A component is changing a controlled input of type checkbox to be uncontrolled. ' +
'Input elements should not switch from controlled to uncontrolled (or vice versa). ' +
'Decide between using a controlled or uncontrolled input ' +
@@ -771,8 +771,8 @@ describe('ReactDOMInput', () => {
var container = document.createElement('div');
ReactDOM.render(stub, container);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'A component is changing a controlled input of type checkbox to be uncontrolled. ' +
'Input elements should not switch from controlled to uncontrolled (or vice versa). ' +
'Decide between using a controlled or uncontrolled input ' +
@@ -785,8 +785,8 @@ describe('ReactDOMInput', () => {
var container = document.createElement('div');
ReactDOM.render(stub, container);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'A component is changing a controlled input of type checkbox to be uncontrolled. ' +
'Input elements should not switch from controlled to uncontrolled (or vice versa). ' +
'Decide between using a controlled or uncontrolled input ' +
@@ -799,8 +799,8 @@ describe('ReactDOMInput', () => {
var container = document.createElement('div');
ReactDOM.render(stub, container);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'A component is changing an uncontrolled input of type checkbox to be controlled. ' +
'Input elements should not switch from uncontrolled to controlled (or vice versa). ' +
'Decide between using a controlled or uncontrolled input ' +
@@ -813,8 +813,8 @@ describe('ReactDOMInput', () => {
var container = document.createElement('div');
ReactDOM.render(stub, container);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'A component is changing an uncontrolled input of type checkbox to be controlled. ' +
'Input elements should not switch from uncontrolled to controlled (or vice versa). ' +
'Decide between using a controlled or uncontrolled input ' +
@@ -827,8 +827,8 @@ describe('ReactDOMInput', () => {
var container = document.createElement('div');
ReactDOM.render(stub, container);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'A component is changing a controlled input of type radio to be uncontrolled. ' +
'Input elements should not switch from controlled to uncontrolled (or vice versa). ' +
'Decide between using a controlled or uncontrolled input ' +
@@ -841,8 +841,8 @@ describe('ReactDOMInput', () => {
var container = document.createElement('div');
ReactDOM.render(stub, container);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'A component is changing a controlled input of type radio to be uncontrolled. ' +
'Input elements should not switch from controlled to uncontrolled (or vice versa). ' +
'Decide between using a controlled or uncontrolled input ' +
@@ -855,8 +855,8 @@ describe('ReactDOMInput', () => {
var container = document.createElement('div');
ReactDOM.render(stub, container);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'A component is changing a controlled input of type radio to be uncontrolled. ' +
'Input elements should not switch from controlled to uncontrolled (or vice versa). ' +
'Decide between using a controlled or uncontrolled input ' +
@@ -869,8 +869,8 @@ describe('ReactDOMInput', () => {
var container = document.createElement('div');
ReactDOM.render(stub, container);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'A component is changing an uncontrolled input of type radio to be controlled. ' +
'Input elements should not switch from uncontrolled to controlled (or vice versa). ' +
'Decide between using a controlled or uncontrolled input ' +
@@ -883,8 +883,8 @@ describe('ReactDOMInput', () => {
var container = document.createElement('div');
ReactDOM.render(stub, container);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'A component is changing an uncontrolled input of type radio to be controlled. ' +
'Input elements should not switch from uncontrolled to controlled (or vice versa). ' +
'Decide between using a controlled or uncontrolled input ' +
@@ -899,7 +899,7 @@ describe('ReactDOMInput', () => {
ReactDOM.render(, container);
ReactDOM.render( null} />, container);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('should not warn if radio value changes but never becomes uncontrolled', () => {
@@ -913,14 +913,14 @@ describe('ReactDOMInput', () => {
checked={false}
onChange={() => null}
/>, container);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('should warn if radio checked false changes to become uncontrolled', () => {
var container = document.createElement('div');
ReactDOM.render( null} />, container);
ReactDOM.render(, container);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'A component is changing a controlled input of type radio to be uncontrolled. ' +
'Input elements should not switch from controlled to uncontrolled (or vice versa). ' +
'Decide between using a controlled or uncontrolled input ' +
diff --git a/src/renderers/dom/shared/wrappers/__tests__/ReactDOMOption-test.js b/src/renderers/dom/shared/wrappers/__tests__/ReactDOMOption-test.js
index 952217dfb8..881ff7d32b 100644
--- a/src/renderers/dom/shared/wrappers/__tests__/ReactDOMOption-test.js
+++ b/src/renderers/dom/shared/wrappers/__tests__/ReactDOMOption-test.js
@@ -39,8 +39,8 @@ describe('ReactDOMOption', () => {
expect(node.innerHTML).toBe('1 2');
ReactTestUtils.renderIntoDocument();
// only warn once
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain('Only strings and numbers are supported as
A gorilla!
);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Select elements must be either controlled or uncontrolled ' +
'(specify either the value prop, or the defaultValue prop, but not ' +
'both). Decide between using a controlled or uncontrolled select ' +
@@ -479,7 +479,7 @@ describe('ReactDOMSelect', () => {
A gorilla!
);
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
});
it('should be able to safely remove select onChange', () => {
diff --git a/src/renderers/dom/shared/wrappers/__tests__/ReactDOMTextarea-test.js b/src/renderers/dom/shared/wrappers/__tests__/ReactDOMTextarea-test.js
index 511c2325a0..2787a8c75d 100644
--- a/src/renderers/dom/shared/wrappers/__tests__/ReactDOMTextarea-test.js
+++ b/src/renderers/dom/shared/wrappers/__tests__/ReactDOMTextarea-test.js
@@ -255,7 +255,7 @@ describe('ReactDOMTextarea', () => {
var stub = ;
var node = renderTextarea(stub, container);
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
expect(node.value).toBe('giraffe');
// Changing children should do nothing, it functions like `defaultValue`.
@@ -294,14 +294,14 @@ describe('ReactDOMTextarea', () => {
it('should allow numbers as children', () => {
spyOn(console, 'error');
var node = renderTextarea();
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
expect(node.value).toBe('17');
});
it('should allow booleans as children', () => {
spyOn(console, 'error');
var node = renderTextarea();
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
expect(node.value).toBe('false');
});
@@ -313,7 +313,7 @@ describe('ReactDOMTextarea', () => {
},
};
var node = renderTextarea();
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
expect(node.value).toBe('sharkswithlasers');
});
@@ -326,7 +326,7 @@ describe('ReactDOMTextarea', () => {
);
}).toThrow();
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
var node;
expect(function() {
@@ -335,7 +335,7 @@ describe('ReactDOMTextarea', () => {
expect(node.value).toBe('[object Object]');
- expect(console.error.calls.count()).toBe(2);
+ expectDev(console.error.calls.count()).toBe(2);
});
it('should unmount', () => {
@@ -348,14 +348,14 @@ describe('ReactDOMTextarea', () => {
spyOn(console, 'error');
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'`value` prop on `textarea` should not be null. ' +
'Consider using the empty string to clear the component or `undefined` ' +
'for uncontrolled components.'
);
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
});
it('should warn if value and defaultValue are specified', () => {
@@ -363,7 +363,7 @@ describe('ReactDOMTextarea', () => {
ReactTestUtils.renderIntoDocument(
);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Textarea elements must be either controlled or uncontrolled ' +
'(specify either the value prop, or the defaultValue prop, but not ' +
'both). Decide between using a controlled or uncontrolled textarea ' +
@@ -374,7 +374,7 @@ describe('ReactDOMTextarea', () => {
ReactTestUtils.renderIntoDocument(
);
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
});
});
diff --git a/src/renderers/dom/stack/client/__tests__/ReactDOM-test.js b/src/renderers/dom/stack/client/__tests__/ReactDOM-test.js
index 76bc51e85e..7041673d92 100644
--- a/src/renderers/dom/stack/client/__tests__/ReactDOM-test.js
+++ b/src/renderers/dom/stack/client/__tests__/ReactDOM-test.js
@@ -115,7 +115,7 @@ describe('ReactDOM', () => {
spyOn(console, 'error');
var element = React.DOM.div();
expect(element.type).toBe('div');
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('throws in render() if the mount callback is not a function', () => {
diff --git a/src/renderers/dom/stack/client/__tests__/ReactMount-test.js b/src/renderers/dom/stack/client/__tests__/ReactMount-test.js
index 53538b4af0..1bbfeff822 100644
--- a/src/renderers/dom/stack/client/__tests__/ReactMount-test.js
+++ b/src/renderers/dom/stack/client/__tests__/ReactMount-test.js
@@ -134,12 +134,12 @@ describe('ReactMount', () => {
spyOn(console, 'error');
ReactMount.render(, container);
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
container.innerHTML = ' ' + ReactDOMServer.renderToString();
ReactMount.render(, container);
- expect(console.error.calls.count()).toBe(2);
+ expectDev(console.error.calls.count()).toBe(2);
});
it('should not warn if mounting into non-empty node', () => {
@@ -148,7 +148,7 @@ describe('ReactMount', () => {
spyOn(console, 'error');
ReactMount.render(, container);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('should warn when mounting into document.body', () => {
@@ -158,8 +158,8 @@ describe('ReactMount', () => {
ReactMount.render(, iFrame.contentDocument.body);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Rendering components directly into document.body is discouraged'
);
});
@@ -175,8 +175,8 @@ describe('ReactMount', () => {
This markup contains an nbsp entity: client text
,
div
);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
' (client) nbsp entity: client text
\n' +
' (server) nbsp entity: server text
'
);
@@ -220,8 +220,8 @@ describe('ReactMount', () => {
spyOn(console, 'error');
var rootNode = container.firstChild;
ReactDOM.render(, rootNode);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: render(...): Replacing React-rendered children with a new ' +
'root component. If you intended to update the children of this node, ' +
'you should instead have the existing children update their state and ' +
@@ -246,15 +246,15 @@ describe('ReactMount', () => {
spyOn(console, 'error');
ReactDOMOther.unmountComponentAtNode(container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: unmountComponentAtNode(): The node you\'re attempting to unmount ' +
'was rendered by another copy of React.'
);
// Don't throw a warning if the correct React copy unmounts the node
ReactDOM.unmountComponentAtNode(container);
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
});
it('passes the correct callback context', () => {
diff --git a/src/renderers/dom/stack/client/__tests__/ReactMountDestruction-test.js b/src/renderers/dom/stack/client/__tests__/ReactMountDestruction-test.js
index 6068d99339..f41c7def8a 100644
--- a/src/renderers/dom/stack/client/__tests__/ReactMountDestruction-test.js
+++ b/src/renderers/dom/stack/client/__tests__/ReactMountDestruction-test.js
@@ -53,8 +53,8 @@ describe('ReactMount', () => {
var rootDiv = mainContainerDiv.firstChild;
spyOn(console, 'error');
ReactDOM.unmountComponentAtNode(rootDiv);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: unmountComponentAtNode(): The node you\'re attempting to ' +
'unmount was rendered by React and is not a top-level container. You ' +
'may have accidentally passed in a React root node instead of its ' +
@@ -77,8 +77,8 @@ describe('ReactMount', () => {
var nonRootDiv = mainContainerDiv.firstChild.firstChild;
spyOn(console, 'error');
ReactDOM.unmountComponentAtNode(nonRootDiv);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: unmountComponentAtNode(): The node you\'re attempting to ' +
'unmount was rendered by React and is not a top-level container. ' +
'Instead, have the parent component update its state and rerender in ' +
diff --git a/src/renderers/dom/stack/server/__tests__/ReactServerRendering-test.js b/src/renderers/dom/stack/server/__tests__/ReactServerRendering-test.js
index 8572dbe15d..e89fd92bc9 100644
--- a/src/renderers/dom/stack/server/__tests__/ReactServerRendering-test.js
+++ b/src/renderers/dom/stack/server/__tests__/ReactServerRendering-test.js
@@ -265,7 +265,7 @@ describe('ReactServerRendering', () => {
spyOn(console, 'error');
instance = ReactDOM.render(, element);
expect(mountCount).toEqual(4);
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
expect(element.innerHTML.length > 0).toBe(true);
expect(element.innerHTML).not.toEqual(lastMarkup);
@@ -476,8 +476,8 @@ describe('ReactServerRendering', () => {
spyOn(console, 'error');
ReactServerRendering.renderToString();
jest.runOnlyPendingTimers();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.mostRecent().args[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.mostRecent().args[0]).toBe(
'Warning: setState(...): Can only update a mounting component.' +
' This usually means you called setState() outside componentWillMount() on the server.' +
' This is a no-op. Please check the code for the Foo component.'
@@ -502,8 +502,8 @@ describe('ReactServerRendering', () => {
spyOn(console, 'error');
ReactServerRendering.renderToString();
jest.runOnlyPendingTimers();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.mostRecent().args[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.mostRecent().args[0]).toBe(
'Warning: replaceState(...): Can only update a mounting component. ' +
'This usually means you called replaceState() outside componentWillMount() on the server. ' +
'This is a no-op. Please check the code for the Bar component.'
@@ -529,8 +529,8 @@ describe('ReactServerRendering', () => {
spyOn(console, 'error');
ReactServerRendering.renderToString();
jest.runOnlyPendingTimers();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.mostRecent().args[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.mostRecent().args[0]).toBe(
'Warning: forceUpdate(...): Can only update a mounting component. ' +
'This usually means you called forceUpdate() outside componentWillMount() on the server. ' +
'This is a no-op. Please check the code for the Baz component.'
diff --git a/src/renderers/shared/__tests__/ReactDebugTool-test.js b/src/renderers/shared/__tests__/ReactDebugTool-test.js
index 6f849c62f6..76d582f6f1 100644
--- a/src/renderers/shared/__tests__/ReactDebugTool-test.js
+++ b/src/renderers/shared/__tests__/ReactDebugTool-test.js
@@ -63,14 +63,14 @@ describe('ReactDebugTool', () => {
});
ReactDebugTool.onTestEvent();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Exception thrown by hook while handling ' +
'onTestEvent: Error: Hi.'
);
ReactDebugTool.onTestEvent();
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
});
it('returns isProfiling state', () => {
diff --git a/src/renderers/shared/__tests__/ReactPerf-test.js b/src/renderers/shared/__tests__/ReactPerf-test.js
index dfd1e517fc..b8f895fc88 100644
--- a/src/renderers/shared/__tests__/ReactPerf-test.js
+++ b/src/renderers/shared/__tests__/ReactPerf-test.js
@@ -389,28 +389,28 @@ describe('ReactPerf', () => {
var measurements = measure(() => {});
spyOn(console, 'error');
ReactPerf.getMeasurementsSummaryMap(measurements);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'`ReactPerf.getMeasurementsSummaryMap(...)` is deprecated. Use ' +
'`ReactPerf.getWasted(...)` instead.'
);
ReactPerf.getMeasurementsSummaryMap(measurements);
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
});
it('warns once when using printDOM', () => {
var measurements = measure(() => {});
spyOn(console, 'error');
ReactPerf.printDOM(measurements);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'`ReactPerf.printDOM(...)` is deprecated. Use ' +
'`ReactPerf.printOperations(...)` instead.'
);
ReactPerf.printDOM(measurements);
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
});
it('returns isRunning state', () => {
@@ -464,7 +464,7 @@ describe('ReactPerf', () => {
expect(ReactPerf.stop()).toBe(undefined);
expect(ReactPerf.isRunning()).toBe(false);
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
__DEV__ = true;
});
diff --git a/src/renderers/shared/hooks/__tests__/ReactComponentTreeHook-test.js b/src/renderers/shared/hooks/__tests__/ReactComponentTreeHook-test.js
index 61fd34902d..8bfa7be970 100644
--- a/src/renderers/shared/hooks/__tests__/ReactComponentTreeHook-test.js
+++ b/src/renderers/shared/hooks/__tests__/ReactComponentTreeHook-test.js
@@ -1830,8 +1830,8 @@ describe('ReactComponentTreeHook', () => {
spyOn(console, 'error');
getAddendum(-17);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: ReactComponentTreeHook: Missing React element for ' +
'debugID -17 when building stack'
);
diff --git a/src/renderers/shared/stack/reconciler/__tests__/ReactComponent-test.js b/src/renderers/shared/stack/reconciler/__tests__/ReactComponent-test.js
index 6043979b0c..0b3cbd39bc 100644
--- a/src/renderers/shared/stack/reconciler/__tests__/ReactComponent-test.js
+++ b/src/renderers/shared/stack/reconciler/__tests__/ReactComponent-test.js
@@ -348,7 +348,7 @@ describe('ReactComponent', () => {
);
// One warning for each element creation
- expect(console.error.calls.count()).toBe(2);
+ expectDev(console.error.calls.count()).toBe(2);
});
});
diff --git a/src/renderers/shared/stack/reconciler/__tests__/ReactComponentLifeCycle-test.js b/src/renderers/shared/stack/reconciler/__tests__/ReactComponentLifeCycle-test.js
index 9c2f564675..bb2e9db48c 100644
--- a/src/renderers/shared/stack/reconciler/__tests__/ReactComponentLifeCycle-test.js
+++ b/src/renderers/shared/stack/reconciler/__tests__/ReactComponentLifeCycle-test.js
@@ -234,8 +234,8 @@ describe('ReactComponentLifeCycle', () => {
}
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: setState(...): Can only update a mounted or ' +
'mounting component. This usually means you called setState() on an ' +
'unmounted component. This is a no-op. Please check the code for the ' +
@@ -263,8 +263,8 @@ describe('ReactComponentLifeCycle', () => {
var instance = ReactTestUtils.renderIntoDocument(element);
expect(instance.isMounted()).toBeTruthy();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Component is accessing isMounted inside its render()'
);
});
@@ -289,8 +289,8 @@ describe('ReactComponentLifeCycle', () => {
var instance = ReactTestUtils.renderIntoDocument(element);
expect(instance.isMounted()).toBeTruthy();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Component is accessing isMounted inside its render()'
);
});
@@ -330,8 +330,8 @@ describe('ReactComponentLifeCycle', () => {
});
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Component is accessing findDOMNode inside its render()'
);
});
diff --git a/src/renderers/shared/stack/reconciler/__tests__/ReactCompositeComponent-test.js b/src/renderers/shared/stack/reconciler/__tests__/ReactCompositeComponent-test.js
index 16c44dfbf6..c8055754ab 100644
--- a/src/renderers/shared/stack/reconciler/__tests__/ReactCompositeComponent-test.js
+++ b/src/renderers/shared/stack/reconciler/__tests__/ReactCompositeComponent-test.js
@@ -184,11 +184,11 @@ describe('ReactCompositeComponent', () => {
mountedInstance.methodAutoBound();
}).not.toThrow();
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
var explicitlyBound = mountedInstance.methodToBeExplicitlyBound.bind(
mountedInstance
);
- expect(console.error.calls.count()).toBe(2);
+ expectDev(console.error.calls.count()).toBe(2);
var autoBound = mountedInstance.methodAutoBound;
var context = {};
@@ -269,13 +269,13 @@ describe('ReactCompositeComponent', () => {
instance = ReactDOM.render(instance, container);
instance.forceUpdate();
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
ReactDOM.unmountComponentAtNode(container);
instance.forceUpdate();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: forceUpdate(...): Can only update a mounted or ' +
'mounting component. This usually means you called forceUpdate() on an ' +
'unmounted component. This is a no-op. Please check the code for the ' +
@@ -309,7 +309,7 @@ describe('ReactCompositeComponent', () => {
instance.setState({value: 1});
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
expect(renders).toBe(2);
@@ -318,8 +318,8 @@ describe('ReactCompositeComponent', () => {
expect(renders).toBe(2);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: setState(...): Can only update a mounted or ' +
'mounting component. This usually means you called setState() on an ' +
'unmounted component. This is a no-op. Please check the code for the ' +
@@ -377,12 +377,12 @@ describe('ReactCompositeComponent', () => {
}
}
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
var instance = ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: setState(...): Cannot update during an existing state ' +
'transition (such as within `render` or another component\'s ' +
'constructor). Render methods should be a pure function of props and ' +
@@ -426,12 +426,12 @@ describe('ReactCompositeComponent', () => {
}
}
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
var instance = ReactDOM.render(, container);
expect(renderPasses).toBe(2);
expect(instance.state.value).toBe(1);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: setState(...): Cannot call setState() inside getChildContext()'
);
});
@@ -502,8 +502,8 @@ describe('ReactCompositeComponent', () => {
var instance = ReactTestUtils.renderIntoDocument();
instance.setState({bogus: true});
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: Component.shouldComponentUpdate(): Returned undefined instead of a ' +
'boolean value. Make sure to return true or false.'
);
@@ -523,8 +523,8 @@ describe('ReactCompositeComponent', () => {
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: Component has a method called ' +
'componentDidUnmount(). But there is no such lifecycle method. ' +
'Did you mean componentWillUnmount()?'
@@ -1022,8 +1022,8 @@ describe('ReactCompositeComponent', () => {
}
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: _renderNewRootComponent(): Render methods should ' +
'be a pure function of props and state; triggering nested component ' +
'updates from render is not allowed. If necessary, trigger nested ' +
@@ -1267,12 +1267,12 @@ describe('ReactCompositeComponent', () => {
}
}
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Foo(...): When calling super() in `Foo`, make sure to pass ' +
'up the same props that your component\'s constructor was passed.'
);
diff --git a/src/renderers/shared/stack/reconciler/__tests__/ReactMockedComponent-test.js b/src/renderers/shared/stack/reconciler/__tests__/ReactMockedComponent-test.js
index 78ba7b4c86..213925915f 100644
--- a/src/renderers/shared/stack/reconciler/__tests__/ReactMockedComponent-test.js
+++ b/src/renderers/shared/stack/reconciler/__tests__/ReactMockedComponent-test.js
@@ -32,7 +32,7 @@ describe('ReactMockedComponent', () => {
it('should allow an implicitly mocked component to be rendered without warnings', () => {
spyOn(console, 'error');
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('should allow an implicitly mocked component to be updated', () => {
diff --git a/src/renderers/shared/stack/reconciler/__tests__/ReactMultiChild-test.js b/src/renderers/shared/stack/reconciler/__tests__/ReactMultiChild-test.js
index 4fa299e3fd..500d00aa39 100644
--- a/src/renderers/shared/stack/reconciler/__tests__/ReactMultiChild-test.js
+++ b/src/renderers/shared/stack/reconciler/__tests__/ReactMultiChild-test.js
@@ -13,7 +13,7 @@
describe('ReactMultiChild', () => {
function normalizeCodeLocInfo(str) {
- return str.replace(/\(at .+?:\d+\)/g, '(at **)');
+ return str && str.replace(/\(at .+?:\d+\)/g, '(at **)');
}
var React;
@@ -185,8 +185,8 @@ describe('ReactMultiChild', () => {
container
);
- expect(console.error.calls.count()).toBe(1);
- expect(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(normalizeCodeLocInfo(console.error.calls.argsFor(0)[0])).toBe(
'Warning: flattenChildren(...): ' +
'Encountered two children with the same key, `1`. ' +
'Child keys must be unique; when two children share a key, ' +
diff --git a/src/renderers/shared/stack/reconciler/__tests__/ReactMultiChildText-test.js b/src/renderers/shared/stack/reconciler/__tests__/ReactMultiChildText-test.js
index 57d4331d59..582e1151e7 100644
--- a/src/renderers/shared/stack/reconciler/__tests__/ReactMultiChildText-test.js
+++ b/src/renderers/shared/stack/reconciler/__tests__/ReactMultiChildText-test.js
@@ -183,8 +183,8 @@ describe('ReactMultiChildText', () => {
[true,
{1.2}{''}{}{'foo'}
, true, 1.2], [, '1.2'],
['', 'foo',
{true}{}{1.2}{''}
, 'foo'], ['', 'foo', , 'foo'],
]);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Warning: Each child in an array or iterator should have a unique "key" prop.'
);
});
@@ -244,6 +244,6 @@ describe('ReactMultiChildText', () => {
expect(childNodes[5]).toBe(alpha3);
// Using Maps as children gives a single warning
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
});
});
diff --git a/src/renderers/shared/stack/reconciler/__tests__/ReactStatelessComponent-test.js b/src/renderers/shared/stack/reconciler/__tests__/ReactStatelessComponent-test.js
index 39a131d96c..8577ca180e 100644
--- a/src/renderers/shared/stack/reconciler/__tests__/ReactStatelessComponent-test.js
+++ b/src/renderers/shared/stack/reconciler/__tests__/ReactStatelessComponent-test.js
@@ -112,8 +112,8 @@ describe('ReactStatelessComponent', () => {
ReactDOM.render(, container);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'StatelessComponentWithChildContext(...): childContextTypes cannot ' +
'be defined on a functional component.'
);
@@ -127,8 +127,8 @@ describe('ReactStatelessComponent', () => {
expect(function() {
ReactTestUtils.renderIntoDocument(
);
}).toThrow();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'NotAComponent(...): A valid React element (or null) must be returned. ' +
'You may have returned undefined, an array or some other invalid object.'
);
@@ -159,8 +159,8 @@ describe('ReactStatelessComponent', () => {
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Stateless function components cannot be given refs ' +
'(See ref "stateless" in StatelessComponent created by Parent). ' +
'Attempts to access this ref will fail.'
@@ -183,9 +183,9 @@ describe('ReactStatelessComponent', () => {
spyOn(console, 'error');
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain('a unique "key" prop');
- expect(console.error.calls.argsFor(0)[0]).toContain('Child');
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain('a unique "key" prop');
+ expectDev(console.error.calls.argsFor(0)[0]).toContain('Child');
});
it('should support default props and prop types', () => {
@@ -197,7 +197,7 @@ describe('ReactStatelessComponent', () => {
spyOn(console, 'error');
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
expect(
console.error.calls.argsFor(0)[0].replace(/\(at .+?:\d+\)/g, '(at **)')
).toBe(
@@ -265,8 +265,8 @@ describe('ReactStatelessComponent', () => {
expect(function() {
ReactTestUtils.renderIntoDocument(
);
}).toThrow(); // has no method 'render'
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'NotAComponent(...): A valid React element (or null) must be returned. You may ' +
'have returned undefined, an array or some other invalid object.'
);
diff --git a/src/renderers/shared/stack/reconciler/__tests__/ReactUpdates-test.js b/src/renderers/shared/stack/reconciler/__tests__/ReactUpdates-test.js
index 5812ce35c0..f86a015531 100644
--- a/src/renderers/shared/stack/reconciler/__tests__/ReactUpdates-test.js
+++ b/src/renderers/shared/stack/reconciler/__tests__/ReactUpdates-test.js
@@ -897,6 +897,7 @@ describe('ReactUpdates', () => {
var component = ReactTestUtils.renderIntoDocument();
+ console.log('xxx', expect().toThrowError);
expect(() => component.setState({}, 'no')).toThrowError(
'setState(...): Expected the last optional `callback` argument ' +
'to be a function. Instead received: string.'
diff --git a/src/renderers/testing/__tests__/ReactTestRenderer-test.js b/src/renderers/testing/__tests__/ReactTestRenderer-test.js
index d939fa86bc..51447852dd 100644
--- a/src/renderers/testing/__tests__/ReactTestRenderer-test.js
+++ b/src/renderers/testing/__tests__/ReactTestRenderer-test.js
@@ -223,8 +223,8 @@ describe('ReactTestRenderer', () => {
}
ReactTestRenderer.create();
ReactTestRenderer.create();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Stateless function components cannot be given refs ' +
'(See ref "foo" in Bar created by Foo). ' +
'Attempts to access this ref will fail.'
diff --git a/src/shared/utils/__tests__/traverseAllChildren-test.js b/src/shared/utils/__tests__/traverseAllChildren-test.js
index 9fe5774de2..4c160703bf 100644
--- a/src/shared/utils/__tests__/traverseAllChildren-test.js
+++ b/src/shared/utils/__tests__/traverseAllChildren-test.js
@@ -84,8 +84,8 @@ describe('traverseAllChildren', () => {
'.0'
);
expect(traverseContext.length).toEqual(1);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Warning: Each child in an array or iterator should have a unique "key" prop.'
);
});
@@ -330,8 +330,8 @@ describe('traverseAllChildren', () => {
'.2'
);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Warning: Each child in an array or iterator should have a unique "key" prop.'
);
});
@@ -434,8 +434,8 @@ describe('traverseAllChildren', () => {
'.$#3:0'
);
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toContain(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toContain(
'Warning: Using Maps as children is not yet fully supported. It is an ' +
'experimental feature that might be removed. Convert it to a sequence ' +
'/ iterable of keyed ReactElements instead.'
@@ -552,8 +552,8 @@ describe('traverseAllChildren', () => {
ReactTestUtils.renderIntoDocument();
- expect(console.error.calls.count()).toBe(1);
- expect(console.error.calls.argsFor(0)[0]).toBe(
+ expectDev(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.argsFor(0)[0]).toBe(
'Warning: Using Maps as children is not yet fully supported. It is an ' +
'experimental feature that might be removed. Convert it to a sequence ' +
'/ iterable of keyed ReactElements instead. Check the render method of `Parent`.'
diff --git a/src/test/__tests__/ReactTestUtils-test.js b/src/test/__tests__/ReactTestUtils-test.js
index 727d69d2b7..7ebdabdb4a 100644
--- a/src/test/__tests__/ReactTestUtils-test.js
+++ b/src/test/__tests__/ReactTestUtils-test.js
@@ -292,7 +292,7 @@ describe('ReactTestUtils', () => {
var shallowRenderer = ReactTestUtils.createRenderer();
shallowRenderer.render();
- expect(console.error.calls.count()).toBe(1);
+ expectDev(console.error.calls.count()).toBe(1);
expect(
console.error.calls.argsFor(0)[0].replace(/\(at .+?:\d+\)/g, '(at **)')
).toBe(
@@ -546,7 +546,7 @@ describe('ReactTestUtils', () => {
ReactDOM.findDOMNode(instance),
{clientX: CLIENT_X}
);
- expect(console.error.calls.count()).toBe(0);
+ expectDev(console.error.calls.count()).toBe(0);
});
it('can scry with stateless components involved', () => {