diff options
| author | Anna Henningsen <anna@addaleax.net> | 2020-04-09 18:44:28 +0200 |
|---|---|---|
| committer | Beth Griggs <Bethany.Griggs@uk.ibm.com> | 2020-04-14 11:03:55 +0100 |
| commit | 7147df53e820063c99258e307b07febbb03554b1 (patch) | |
| tree | 2b3c5becc843b33e1fa361d55b5ccada18cc3f0a /test/parallel | |
| parent | 0538dd11824cacf7c1ada2a0ef628fe04094cf5c (diff) | |
| download | node-new-7147df53e820063c99258e307b07febbb03554b1.tar.gz | |
worker: fix type check in receiveMessageOnPort
Use the same type check we use in `MoveToContext()` in
`ReceiveMessage()`.
Fixes: https://github.com/nodejs/node/issues/32742
PR-URL: https://github.com/nodejs/node/pull/32745
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Zeyu Yang <himself65@outlook.com>
Diffstat (limited to 'test/parallel')
| -rw-r--r-- | test/parallel/test-worker-message-port-receive-message.js | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/test/parallel/test-worker-message-port-receive-message.js b/test/parallel/test-worker-message-port-receive-message.js index 9bd8e7ed64..e7078ea1e6 100644 --- a/test/parallel/test-worker-message-port-receive-message.js +++ b/test/parallel/test-worker-message-port-receive-message.js @@ -23,3 +23,11 @@ port2.on('message', common.mustNotCall()); port1.postMessage(message1); assert.deepStrictEqual(receiveMessageOnPort(port2), { message: message1 }); port1.close(); + +for (const value of [null, 0, -1, {}, []]) { + assert.throws(() => receiveMessageOnPort(value), { + name: 'TypeError', + code: 'ERR_INVALID_ARG_TYPE', + message: 'First argument needs to be a MessagePort instance' + }); +} |