From f11d4a1556a5536d69c1ae524822a709689c46ac Mon Sep 17 00:00:00 2001 From: weewey Date: Wed, 26 Apr 2017 22:20:35 +0800 Subject: [PATCH] test: use common.mustCall in test-https-strict PR-URL: https://github.com/nodejs/node/pull/12668 Reviewed-By: Rich Trott Reviewed-By: Yuta Hiroto Reviewed-By: James M Snell --- test/parallel/test-https-strict.js | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/test/parallel/test-https-strict.js b/test/parallel/test-https-strict.js index 215ce7fe007c4e..70e11d2c8b2ab1 100644 --- a/test/parallel/test-https-strict.js +++ b/test/parallel/test-https-strict.js @@ -91,8 +91,6 @@ server2.listen(0, listening()); server3.listen(0, listening()); const responseErrors = {}; -let expectResponseCount = 0; -let responseCount = 0; let pending = 0; @@ -147,17 +145,14 @@ function makeReq(path, port, error, host, ca) { options.headers = { host: host }; } const req = https.get(options); - expectResponseCount++; const server = port === server1.address().port ? server1 : port === server2.address().port ? server2 : port === server3.address().port ? server3 : null; - if (!server) throw new Error('invalid port: ' + port); server.expectCount++; - req.on('response', (res) => { - responseCount++; + req.on('response', common.mustCall((res) => { assert.strictEqual(res.connection.authorizationError, error); responseErrors[path] = res.connection.authorizationError; pending--; @@ -167,7 +162,7 @@ function makeReq(path, port, error, host, ca) { server3.close(); } res.resume(); - }); + })); } function allListening() { @@ -220,5 +215,4 @@ process.on('exit', () => { assert.strictEqual(server1.requests.length, server1.expectCount); assert.strictEqual(server2.requests.length, server2.expectCount); assert.strictEqual(server3.requests.length, server3.expectCount); - assert.strictEqual(responseCount, expectResponseCount); });