Commit 4f5f0e4
test: refactor test-net-keepalive.js
- Replace require() vars with const.
- Replace assert.equal() with assert.strictEqual().
- Add common.mustCall() to the setTimeout() callback.
PR-URL: #9995
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>1 parent 23a573f commit 4f5f0e4
1 file changed
Lines changed: 7 additions & 7 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1,20 +1,20 @@ | |||
| 1 | 1 | 'use strict'; | |
| 2 | - var common = require('../common'); | ||
| 3 | - var assert = require('assert'); | ||
| 4 | - var net = require('net'); | ||
| 2 | + const common = require('../common'); | ||
| 3 | + const assert = require('assert'); | ||
| 4 | + const net = require('net'); | ||
| 5 | 5 | ||
| 6 | 6 | var serverConnection; | |
| 7 | 7 | var clientConnection; | |
| 8 | 8 | var echoServer = net.createServer(function(connection) { | |
| 9 | 9 | serverConnection = connection; | |
| 10 | - setTimeout(function() { | ||
| 10 | + setTimeout(common.mustCall(function() { | ||
| 11 | 11 | // make sure both connections are still open | |
| 12 | - assert.equal(serverConnection.readyState, 'open'); | ||
| 13 | - assert.equal(clientConnection.readyState, 'open'); | ||
| 12 | + assert.strictEqual(serverConnection.readyState, 'open'); | ||
| 13 | + assert.strictEqual(clientConnection.readyState, 'open'); | ||
| 14 | 14 | serverConnection.end(); | |
| 15 | 15 | clientConnection.end(); | |
| 16 | 16 | echoServer.close(); | |
| 17 | - }, common.platformTimeout(100)); | ||
| 17 | + }, 1), common.platformTimeout(100)); | ||
| 18 | 18 | connection.setTimeout(0); | |
| 19 | 19 | assert.notEqual(connection.setKeepAlive, undefined); | |
| 20 | 20 | // send a keepalive packet after 50 ms | |
0 commit comments