refactor http.get expectedCount with common.mustCall

nodejs · Apr 27, 2017 · 6306067 · 6306067
1 parent 1d96803
commit 6306067
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/test/parallel/test-https-strict.js b/test/parallel/test-https-strict.js
@@ -156,8 +156,7 @@ function makeReq(path, port, error, host, ca) {
   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 +166,7 @@ function makeReq(path, port, error, host, ca) {
       server3.close();
     }
     res.resume();
-  });
+  }));
 }
 
 function allListening() {
@@ -220,5 +219,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);
 });