diff options
author | Ruben Bridgewater <ruben@bridgewater.de> | 2019-04-02 03:46:17 +0200 |
---|---|---|
committer | Ruben Bridgewater <ruben@bridgewater.de> | 2019-04-04 12:51:03 +0200 |
commit | 3b044962c48fe313905877a96b5d0894a5404f6f (patch) | |
tree | f88086693fd685477a88b5cbc5c7442f25a49986 /test/parallel/test-timers-refresh.js | |
parent | a9bf6652b5353f2098d4c0cd0eb77d17e02e164d (diff) | |
download | android-node-v8-3b044962c48fe313905877a96b5d0894a5404f6f.tar.gz android-node-v8-3b044962c48fe313905877a96b5d0894a5404f6f.tar.bz2 android-node-v8-3b044962c48fe313905877a96b5d0894a5404f6f.zip |
errors: add more information in case of invalid callbacks
This adds the actual callback that is passed through to the error
message in case an ERR_INVALID_CALLBACK error is thrown.
PR-URL: https://github.com/nodejs/node/pull/27048
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Diffstat (limited to 'test/parallel/test-timers-refresh.js')
-rw-r--r-- | test/parallel/test-timers-refresh.js | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/test/parallel/test-timers-refresh.js b/test/parallel/test-timers-refresh.js index 597d5c9241..28267f2b7b 100644 --- a/test/parallel/test-timers-refresh.js +++ b/test/parallel/test-timers-refresh.js @@ -6,6 +6,7 @@ const common = require('../common'); const { strictEqual } = require('assert'); const { setUnrefTimeout } = require('internal/timers'); +const { inspect } = require('util'); // Schedule the unrefed cases first so that the later case keeps the event loop // active. @@ -32,14 +33,14 @@ const { setUnrefTimeout } = require('internal/timers'); // Should throw with non-functions { - const expectedError = { - code: 'ERR_INVALID_CALLBACK', - message: 'Callback must be a function' - }; - [null, true, false, 0, 1, NaN, '', 'foo', {}, Symbol()].forEach((cb) => { - common.expectsError(() => setUnrefTimeout(cb), - expectedError); + common.expectsError( + () => setUnrefTimeout(cb), + { + code: 'ERR_INVALID_CALLBACK', + message: `Callback must be a function. Received ${inspect(cb)}` + } + ); }); } |