Skip to content

Commit

Permalink
test: add more asserts to test-internal-errors
Browse files Browse the repository at this point in the history
Backport-PR-URL: #19579
PR-URL: #13686
Fixes: #13682
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
  • Loading branch information
refack authored and targos committed Mar 30, 2018
1 parent 6bc49f0 commit cc68bc2
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions test/parallel/test-internal-errors.js
Original file line number Diff line number Diff line change
Expand Up @@ -175,6 +175,7 @@ common.expectsError(() => {
message: /^Error for testing 2/ });
}, {
code: 'ERR_ASSERTION',
type: assert.AssertionError,
message: /.+ does not match \S/
});

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

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

Expand Down

0 comments on commit cc68bc2

Please sign in to comment.