Skip to content

[9.x backport] assert & util PRs #19230

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

Closed
BridgeAR wants to merge 21 commits into from
Closed

[9.x backport] assert & util PRs #19230

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
c6aa5ff
doc: improve assert documentation
BridgeAR Nov 13, 2017
b3e4476
assert: add strict functionality export
BridgeAR Nov 13, 2017
da78022
assert: use object argument in innerFail
BridgeAR Dec 9, 2017
9664cc8
assert: fix throws and doesNotThrow stack frames
BridgeAR Dec 15, 2017
0ed963b
test: refactor common.expectsError
BridgeAR Dec 15, 2017
cc52dec
assert: improve assert.throws
BridgeAR Dec 9, 2017
f38be41
doc: improve .throws RegExp info
BridgeAR Dec 11, 2017
7f558d5
assert: .throws accept objects
BridgeAR Dec 10, 2017
fe9f098
assert: fix strict regression
BridgeAR Dec 28, 2017
49dc778
lib: handle `throw undefined` in assert.throws()
bnoordhuis Jan 7, 2018
5d88557
util: fix custom inspect description
BridgeAR Dec 9, 2017
f77f91d
util: rename util.inspect argument
BridgeAR Dec 9, 2017
fb85597
util: add util.inspect compact option
BridgeAR Dec 9, 2017
70ac318
assert: improve error messages
BridgeAR Dec 9, 2017
fae00e3
tty: add getColorDepth function
BridgeAR Jan 16, 2018
073cf69
tty: refactor to es6
BridgeAR Jan 17, 2018
4fbed71
assert: use destructuring for errors
BridgeAR Jan 22, 2018
6694f48
assert: fix throws trace
BridgeAR Feb 6, 2018
1ce2898
doc: document asserts Weak(Map|Set) behavior
BridgeAR Jan 19, 2018
ca8a591
assert: fix infinite loop
BridgeAR Feb 7, 2018
254f888
assert: show proper differences
BridgeAR Feb 7, 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
assert: fix throws and doesNotThrow stack frames 
The stack frames from .throws and .doesNotThrow got included
even though that was not intended.

PR-URL: #17703
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
@BridgeAR
BridgeAR committed Mar 8, 2018
commit 9664cc825a923786db19aff9effbfad59cbc5ec6
4 changes: 2 additions & 2 deletions lib/assert.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -245,7 +245,7 @@ function innerThrows(shouldThrow, block, expected, message) {
expected,
operator: 'throws',
message: `Missing expected exception${details}`,
stackStartFn: innerThrows
stackStartFn: assert.throws
});
}
if (expected && expectedException(actual, expected) === false) {
Expand All @@ -259,7 +259,7 @@ function innerThrows(shouldThrow, block, expected, message) {
expected,
operator: 'doesNotThrow',
message: `Got unwanted exception${details}\n${actual.message}`,
stackStartFn: innerThrows
stackStartFn: assert.doesNotThrow
});
}
throw actual;
Expand Down
2 changes: 2 additions & 0 deletions test/parallel/test-assert.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -634,6 +634,7 @@ testAssertionMessage({ a: NaN, b: Infinity, c: -Infinity },
} catch (e) {
threw = true;
assert.strictEqual(e.message, 'Missing expected exception.');
assert.ok(!e.stack.includes('throws'), e.stack);
}
assert.ok(threw);
}
Expand Down Expand Up @@ -678,6 +679,7 @@ try {
threw = true;
assert.ok(e.message.includes(rangeError.message));
assert.ok(e instanceof assert.AssertionError);
assert.ok(!e.stack.includes('doesNotThrow'), e.stack);
}
assert.ok(threw);
}
Expand Down