Skip to content

Commit b59f421

Browse files
refacktargos
authored andcommitted
test: add more asserts to test-internal-errors
PR-URL: nodejs#13686 Fixes: nodejs#13682 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
1 parent b4a99a7 commit b59f421

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

test/parallel/test-internal-errors.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -175,6 +175,7 @@ common.expectsError(() => {
175175
message: /^Error for testing 2/ });
176176
}, {
177177
code: 'ERR_ASSERTION',
178+
type: assert.AssertionError,
178179
message: /.+ does not match \S/
179180
});
180181

@@ -225,6 +226,7 @@ common.expectsError(
225226
() => errors.message('ERR_INVALID_URL_SCHEME', [[]]),
226227
{
227228
code: 'ERR_ASSERTION',
229+
type: assert.AssertionError,
228230
message: /^At least one expected value needs to be specified$/
229231
});
230232

@@ -239,6 +241,7 @@ common.expectsError(
239241
() => errors.message('ERR_MISSING_ARGS'),
240242
{
241243
code: 'ERR_ASSERTION',
244+
type: assert.AssertionError,
242245
message: /^At least one arg needs to be specified$/
243246
});
244247

0 commit comments

Comments
 (0)