From 015c1469e6b4bb89c2ac290ee5db2fa381810e44 Mon Sep 17 00:00:00 2001 From: Refael Ackermann Date: Mon, 19 Jun 2017 15:24:04 -0400 Subject: [PATCH] test: add more asserts to `test-internal-errors` MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PR-URL: https://github.com/nodejs/node/pull/13686 Fixes: https://github.com/nodejs/node/issues/13682 Reviewed-By: James M Snell Reviewed-By: Michaël Zasso --- test/parallel/test-internal-errors.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/test/parallel/test-internal-errors.js b/test/parallel/test-internal-errors.js index 78582a53504748..12551d89c22ce9 100644 --- a/test/parallel/test-internal-errors.js +++ b/test/parallel/test-internal-errors.js @@ -183,6 +183,7 @@ common.expectsError(() => { message: /^Error for testing 2/ }); }, { code: 'ERR_ASSERTION', + type: assert.AssertionError, message: /.+ does not match \S/ }); @@ -233,6 +234,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$/ }); @@ -247,6 +249,7 @@ common.expectsError( () => errors.message('ERR_MISSING_ARGS'), { code: 'ERR_ASSERTION', + type: assert.AssertionError, message: /^At least one arg needs to be specified$/ });