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 strict regression 
Using `assert()` or `assert.ok()` resulted in a error since a
refactoring.

Refs: #17582

PR-URL: #17903
Refs: #17582
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
@BridgeAR
BridgeAR committed Mar 8, 2018
commit fe9f098df2b877a06dcf587dca5092997e48da25
10 changes: 9 additions & 1 deletion lib/assert.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -319,7 +319,15 @@ assert.ifError = function ifError(err) { if (err) throw err; };

// Expose a strict only variant of assert
function strict(value, message) {
if (!value) innerFail(value, true, message, '==', strict);
if (!value) {
innerFail({
actual: value,
expected: true,
message,
operator: '==',
stackStartFn: strict
});
}
}
assert.strict = Object.assign(strict, assert, {
equal: assert.strictEqual,
Expand Down
8 changes: 8 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 @@ -753,6 +753,14 @@ common.expectsError(
assert.equal(Object.keys(assert).length, Object.keys(a).length);
/* eslint-enable no-restricted-properties */
assert(7);
common.expectsError(
() => assert(),
{
code: 'ERR_ASSERTION',
type: assert.AssertionError,
message: 'undefined == true'
}
);
}

common.expectsError(
Expand Down