test: refactoring test with common.mustCall

PR-URL: #12702 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
nodejs · Jul 11, 2017 · 8414659 · 8414659
1 parent 608c309
commit 8414659
Showing 1 changed file with 13 additions and 10 deletions.
diff --git a/test/parallel/test-https-simple.js b/test/parallel/test-https-simple.js
@@ -34,28 +34,31 @@ const serverCallback = common.mustCall(function(req, res) {
 
 const server = https.createServer(options, serverCallback);
 
-server.listen(0, function() {
+server.listen(0, common.mustCall(() => {
   // Do a request ignoring the unauthorized server certs
+  const port = server.address().port;
+
   const noCertCheckOptions = {
     hostname: '127.0.0.1',
-    port: this.address().port,
+    port: port,
     path: '/',
     method: 'GET',
     rejectUnauthorized: false
   };
+
   noCertCheckOptions.Agent = new https.Agent(noCertCheckOptions);
 
-  const req = https.request(noCertCheckOptions, function(res) {
+  const req = https.request(noCertCheckOptions, common.mustCall((res) => {
     let responseBody = '';
     res.on('data', function(d) {
       responseBody = responseBody + d;
     });
 
-    res.on('end', function() {
+    res.on('end', common.mustCall(() => {
       assert.strictEqual(responseBody, body);
       testSucceeded();
-    });
-  });
+    }));
+  }));
   req.end();
 
   req.on('error', function(e) {
@@ -65,7 +68,7 @@ server.listen(0, function() {
   // Do a request that throws error due to the invalid server certs
   const checkCertOptions = {
     hostname: '127.0.0.1',
-    port: this.address().port,
+    port: port,
     path: '/',
     method: 'GET'
   };
@@ -81,11 +84,11 @@ server.listen(0, function() {
   });
   checkCertReq.end();
 
-  checkCertReq.on('error', function(e) {
+  checkCertReq.on('error', common.mustCall((e) => {
     assert.strictEqual(e.code, 'UNABLE_TO_VERIFY_LEAF_SIGNATURE');
     testSucceeded();
-  });
-});
+  }));
+}));
 
 process.on('exit', function() {
   assert.strictEqual(successful, tests);