diff options
author | ZYSzys <17367077526@163.com> | 2019-01-15 11:39:20 +0800 |
---|---|---|
committer | Anna Henningsen <anna@addaleax.net> | 2019-01-27 17:20:14 +0100 |
commit | 22a9fe35520f7eb89845237582262457721afc70 (patch) | |
tree | 79e2d411daaeebe020200d2b9836e01381081f37 /test/parallel | |
parent | 802ea05a37e5d342ef26970daeb876f4f8405ce5 (diff) | |
download | android-node-v8-22a9fe35520f7eb89845237582262457721afc70.tar.gz android-node-v8-22a9fe35520f7eb89845237582262457721afc70.tar.bz2 android-node-v8-22a9fe35520f7eb89845237582262457721afc70.zip |
test: add test for net-socket-setTimeout callback
PR-URL: https://github.com/nodejs/node/pull/25084
Refs: https://github.com/nodejs/node/issues/19060
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Diffstat (limited to 'test/parallel')
-rw-r--r-- | test/parallel/test-net-socket-timeout.js | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/test/parallel/test-net-socket-timeout.js b/test/parallel/test-net-socket-timeout.js index a30ab8ce29..f0f5f194bf 100644 --- a/test/parallel/test-net-socket-timeout.js +++ b/test/parallel/test-net-socket-timeout.js @@ -31,6 +31,9 @@ const nonNumericDelays = [ ]; const badRangeDelays = [-0.001, -1, -Infinity, Infinity, NaN]; const validDelays = [0, 0.001, 1, 1e6]; +const invalidCallbacks = [ + 1, '100', true, false, null, {}, [], Symbol('test') +]; for (let i = 0; i < nonNumericDelays.length; i++) { @@ -49,6 +52,19 @@ for (let i = 0; i < validDelays.length; i++) { s.setTimeout(validDelays[i], () => {}); } +for (let i = 0; i < invalidCallbacks.length; i++) { + [0, 1].forEach((mesc) => + common.expectsError( + () => s.setTimeout(mesc, invalidCallbacks[i]), + { + code: 'ERR_INVALID_CALLBACK', + type: TypeError, + message: 'Callback must be a function' + } + ) + ); +} + const server = net.Server(); server.listen(0, common.mustCall(() => { const socket = net.createConnection(server.address().port); |