Skip to content

toWarnInDev matcher; throw on unexpected console.error #11786

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
bvaughn merged 19 commits into from
Jan 2, 2018
Merged

toWarnInDev matcher; throw on unexpected console.error #11786

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
f254d9a
Added toWarnInDev matcher and connected to 1 test
bvaughn Dec 5, 2017
2cf33eb
Added .toLowPriorityWarnDev() matcher
bvaughn Dec 6, 2017
178007e
Merged master
bvaughn Dec 18, 2017
472f5e3
Nest toThrow and toWarn matchers in ReactElementValidator test
bvaughn Dec 18, 2017
d8d0f66
Reply Jest spy with custom spy. Unregister spy after toWarnDev() so u…
bvaughn Dec 18, 2017
1af3d1e
console warn/error throws immediately in tests by default (if not spi…
bvaughn Dec 18, 2017
e6ff0da
Removed unused oldMethod
bvaughn Dec 18, 2017
0662fd0
Refactored toWarn matchers to fail faster
bvaughn Dec 19, 2017
36c6550
Added TODO about Jest overriding error stacks
bvaughn Dec 19, 2017
feac559
Moved TODO
bvaughn Dec 19, 2017
47fa899
Pass-thru console message before erroring to make it easier to identify
bvaughn Dec 19, 2017
cdb1b50
Updated a few more tests to use toWarnDev matcher
bvaughn Dec 19, 2017
2541749
Added TypeScript test and types, fixed equivalence test
bvaughn Dec 19, 2017
cdba8b4
Upgraded a few additional tests
bvaughn Dec 19, 2017
68fc971
More robustly handle unexpected warnings within try/catch
bvaughn Dec 19, 2017
1659c93
Merge branch 'master' into expect-toWarnInDev
bvaughn Jan 2, 2018
62c4b4f
Replaced .splice(0) with .length=0
bvaughn Jan 2, 2018
b766964
Error message includes remaining expected warnings in addition to une…
bvaughn Jan 2, 2018
eeeb0e3
Removed unnecessary "" around unexpected error message
bvaughn Jan 2, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Removed unused oldMethod
  • Loading branch information
@bvaughn
bvaughn committed Dec 18, 2017
commit e6ff0da1f14c6c608fbace1856926c2913a0c48b
1 change: 0 additions & 1 deletion scripts/jest/setupTests.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,6 @@ if (process.env.REACT_CLASS_EQUIVALENCE_TEST) {
});

['error', 'warn'].forEach(methodName => {
const oldMethod = console[methodName];
const newMethod = function() {
throw new Error(
`Expected test not to call console.${methodName}(). ` +
Expand Down