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

test: use assert.throws() in test-require-json.js

Browse Source 
Use assert.throws() instead of try/catch.

PR-URL: https://github.com/nodejs/node/pull/28358
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
This commit is contained in:
Alejandro Nanez 2019-06-21 16:58:01 -05:00 committed by Rich Trott
parent 698d479aff
commit c3b2111423
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
test/parallel/test-require-json.js
View File

@ -24,10 +24,9 @@ require('../common');
const assert = require('assert');
const fixtures = require('../common/fixtures');
try {
assert.throws(function() {
require(fixtures.path('invalid.json'));
} catch (err) {
assert.ok(
/test[/\\]fixtures[/\\]invalid\.json: /.test(err.message),
`require() json error should include path: ${err.message}`);
}
}, {
name: 'SyntaxError',
message: /test[/\\]fixtures[/\\]invalid\.json: /,
});