Commit 7ae2ca4
test: removed default message from assert.strictEqual
Use the `assert.strictEqual()` default message instead of a static
message to include the actual and expected value.
PR-URL: #19660
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent a89ba21 commit 7ae2ca4
1 file changed
Lines changed: 3 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -34,7 +34,7 @@ const server1ConnHandler = (socket) => { | |||
| 34 | 34 | assert.fail('data event should not have happened yet'); | |
| 35 | 35 | } | |
| 36 | 36 | ||
| 37 | - assert.strictEqual(data.toString(), msg, 'invalid data received'); | ||
| 37 | + assert.strictEqual(data.toString(), msg); | ||
| 38 | 38 | socket.end(); | |
| 39 | 39 | server1.close(); | |
| 40 | 40 | }); | |
@@ -46,12 +46,11 @@ const server1 = net.createServer({ pauseOnConnect: true }, server1ConnHandler); | |||
| 46 | 46 | ||
| 47 | 47 | const server2ConnHandler = (socket) => { | |
| 48 | 48 | socket.on('data', function(data) { | |
| 49 | - assert.strictEqual(data.toString(), msg, 'invalid data received'); | ||
| 49 | + assert.strictEqual(data.toString(), msg); | ||
| 50 | 50 | socket.end(); | |
| 51 | 51 | server2.close(); | |
| 52 | 52 | ||
| 53 | - assert.strictEqual(server1Sock.bytesRead, 0, | ||
| 54 | - 'no data should have been read yet'); | ||
| 53 | + assert.strictEqual(server1Sock.bytesRead, 0); | ||
| 55 | 54 | server1Sock.resume(); | |
| 56 | 55 | stopped = false; | |
| 57 | 56 | }); | |
0 commit comments