1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: ensure failed assertions cause build to fail

Browse Source 
This updates the test in `test/parallel/test-assert-async.js` to add an
assertion that the Promises used in the test end up fulfilled.
Previously, if an assertion failure occurred, the Promises would have
rejected and a warning would have been logged, but the test would still
have exit code 0.

PR-URL: https://github.com/nodejs/node/pull/19650
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
This commit is contained in:
Teddy Katz 2018-03-28 01:16:20 -04:00
parent 88910724c8
commit 8995125c14
No known key found for this signature in database
GPG Key ID: B79EC7ABC0AA53A0
1 changed files with 46 additions and 52 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
test/parallel/test-assert-async.js
View File

@ -9,7 +9,10 @@ const wait = promisify(setTimeout);
// Test assert.rejects() and assert.doesNotReject() by checking their
// expected output and by verifying that they do not work sync
assert.rejects(
common.crashOnUnhandledRejection();
(async () => {
await assert.rejects(
() => assert.fail(),
common.expectsError({
code: 'ERR_ASSERTION',
@ -18,26 +21,15 @@ assert.rejects(
})
);
assert.doesNotReject(() => {});
{
const promise = assert.rejects(async () => {
await wait(1);
assert.fail();
}, common.expectsError({
code: 'ERR_ASSERTION',
type: assert.AssertionError,
message: 'Failed'
}));
assert.doesNotReject(() => promise);
}
await assert.doesNotReject(() => {});
{
const promise = assert.doesNotReject(async () => {
await wait(1);
throw new Error();
});
assert.rejects(() => promise,
await assert.rejects(
() => promise,
(err) => {
assert(err instanceof assert.AssertionError,
`${err.name} is not instance of AssertionError`);
@ -52,7 +44,8 @@ assert.doesNotReject(() => {});
{
const promise = assert.rejects(() => {});
assert.rejects(() => promise,
await assert.rejects(
() => promise,
(err) => {
assert(err instanceof assert.AssertionError,
`${err.name} is not instance of AssertionError`);
@ -64,3 +57,4 @@ assert.doesNotReject(() => {});
}
);
}
})().then(common.mustCall());