aboutsummaryrefslogtreecommitdiff
path: root/test/message
diff options
context:
space:
mode:
authorJames M Snell <jasnell@gmail.com>2017-04-25 13:18:25 -0700
committerJames M Snell <jasnell@gmail.com>2017-05-04 07:18:17 -0700
commite48d58b8b26bd6f1a59cfb57307906b9b0350272 (patch)
tree20f52e83c2b12abf6f40b7b9b988a039fce467c8 /test/message
parentc1b3b95939884b20a6312822091b9fec2ef9f31f (diff)
downloadandroid-node-v8-e48d58b8b26bd6f1a59cfb57307906b9b0350272.tar.gz
android-node-v8-e48d58b8b26bd6f1a59cfb57307906b9b0350272.tar.bz2
android-node-v8-e48d58b8b26bd6f1a59cfb57307906b9b0350272.zip
assert: fix AssertionError, assign error code
Using `assert.AssertionError()` without the `new` keyword results in a non-intuitive error: ```js > assert.AssertionError({}) TypeError: Cannot assign to read only property 'name' of function 'function ok(value, message) { if (!value) fail(value, true, message, '==', assert.ok); }' at Function.AssertionError (assert.js:45:13) at repl:1:8 at realRunInThisContextScript (vm.js:22:35) at sigintHandlersWrap (vm.js:98:12) at ContextifyScript.Script.runInThisContext (vm.js:24:12) at REPLServer.defaultEval (repl.js:346:29) at bound (domain.js:280:14) at REPLServer.runBound [as eval] (domain.js:293:12) at REPLServer.onLine (repl.js:545:10) at emitOne (events.js:101:20) > ``` The `assert.AssertionError()` can only be used correctly with `new`, so this converts it into a proper ES6 class that will give an appropriate error message. This also associates the appropriate internal/errors code with all `assert.AssertionError` instances and updates the appropriate test cases. PR-URL: https://github.com/nodejs/node/pull/12651 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Diffstat (limited to 'test/message')
-rw-r--r--test/message/error_exit.out6
1 files changed, 3 insertions, 3 deletions
diff --git a/test/message/error_exit.out b/test/message/error_exit.out
index c1b778d6e9..f6ac80794b 100644
--- a/test/message/error_exit.out
+++ b/test/message/error_exit.out
@@ -1,9 +1,9 @@
Exiting with code=1
-
assert.js:*
- throw new assert.AssertionError({
+ throw new AssertionError({
^
-AssertionError: 1 === 2
+
+AssertionError [ERR_ASSERTION]: 1 === 2
at Object.<anonymous> (*test*message*error_exit.js:*:*)
at Module._compile (module.js:*:*)
at Object.Module._extensions..js (module.js:*:*)