diff options
| author | Francesco Falanga <f.falanga@palmabit.com> | 2018-05-13 18:29:25 +0200 |
|---|---|---|
| committer | Rich Trott <rtrott@gmail.com> | 2018-05-13 14:06:26 -0700 |
| commit | eafb30ccbf12a4ded74a30d7634e84fbe4a5add3 (patch) | |
| tree | 218c091fa0a68c9946f444b1ddf5be7465c27f1b /test | |
| parent | 0e9ea35e391656d360ff428e1f9c5b86057df26e (diff) | |
| download | android-node-v8-eafb30ccbf12a4ded74a30d7634e84fbe4a5add3.tar.gz android-node-v8-eafb30ccbf12a4ded74a30d7634e84fbe4a5add3.tar.bz2 android-node-v8-eafb30ccbf12a4ded74a30d7634e84fbe4a5add3.zip | |
test: remove deepStrictEqual() third argument
The call to assert.deepStrictEqual() has a string literal for its third
argument. Unfortunately, a side effect of that is that the values of the
first two arguments are not displayed if there is an AssertionError.
That information is useful for debugging.
PR-URL: https://github.com/nodejs/node/pull/20702
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Diffstat (limited to 'test')
| -rw-r--r-- | test/parallel/test-net-socket-local-address.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/parallel/test-net-socket-local-address.js b/test/parallel/test-net-socket-local-address.js index c4f11db76b..1aa51e0c5c 100644 --- a/test/parallel/test-net-socket-local-address.js +++ b/test/parallel/test-net-socket-local-address.js @@ -17,8 +17,8 @@ const server = net.createServer((socket) => { }); server.on('close', common.mustCall(() => { - assert.deepStrictEqual(clientLocalPorts, serverRemotePorts, - 'client and server should agree on the ports used'); + // client and server should agree on the ports used + assert.deepStrictEqual(clientLocalPorts, serverRemotePorts); assert.strictEqual(2, conns); })); |