test: do not force the process to exit

Using `process.exit()` in these tests is unnecessary and may mask other problems. Refs: #29873 (review)
nodejs · lpinca · Oct 10, 2019 · Oct 11, 2019 · 55273296e172dfb6a0679eb876ed0501fee9b93e
commit 55273296e172dfb6a0679eb876ed0501fee9b93e
diff --git a/test/parallel/test-cluster-net-send.js b/test/parallel/test-cluster-net-send.js
@@ -72,7 +72,6 @@ if (process.argv[2] !== 'child') {
   });
 
   process.on('disconnect', function() {
-    process.exit();
     server.close();
   });
 }
diff --git a/test/parallel/test-http-date-header.js b/test/parallel/test-http-date-header.js
@@ -48,7 +48,6 @@ server.addListener('listening', () => {
               'Response headers didn\'t contain a Date.');
     res.addListener('end', () => {
       server.close();
-      process.exit();
     });
     res.resume();
   });

diff --git a/test/parallel/test-http-expect-continue.js b/test/parallel/test-http-expect-continue.js
@@ -76,7 +76,6 @@ server.on('listening', common.mustCall(() => {
       assert.strictEqual(body, test_res_body);
       assert.ok('abcd' in res.headers, 'Response headers missing.');
       server.close();
-      process.exit();
     }));
   }));
 }));
diff --git a/test/parallel/test-http-localaddress.js b/test/parallel/test-http-localaddress.js
@@ -49,7 +49,6 @@ server.listen(0, '127.0.0.1', () => {
   const req = http.request(options, function(res) {
     res.on('end', () => {
       server.close();
-      process.exit();
     });
     res.resume();
   });

diff --git a/test/parallel/test-http-set-trailers.js b/test/parallel/test-http-set-trailers.js
@@ -61,7 +61,6 @@ server.on('listening', function() {
     outstanding_reqs--;
     if (outstanding_reqs === 0) {
       server.close();
-      process.exit();
     }
   });
 });
@@ -91,7 +90,6 @@ server.on('listening', function() {
       );
       if (outstanding_reqs === 0) {
         server.close();
-        process.exit();
       }
     }
   });
@@ -110,7 +108,6 @@ server.on('listening', function() {
       outstanding_reqs--;
       if (outstanding_reqs === 0) {
         server.close();
-        process.exit();
       }
     });
     res.resume();

diff --git a/test/parallel/test-https-localaddress.js b/test/parallel/test-https-localaddress.js
@@ -61,7 +61,6 @@ server.listen(0, '127.0.0.1', function() {
   const req = https.request(options, function(res) {
     res.on('end', function() {
       server.close();
-      process.exit();
     });
     res.resume();
   });

diff --git a/test/parallel/test-https-timeout.js b/test/parallel/test-https-timeout.js
@@ -51,10 +51,15 @@ const server = https.createServer(options, function() {
     console.log('got response');
   });
 
+  req.on('error', common.expectsError({
+    message: 'socket hang up',
+    code: 'ECONNRESET',
+    type: Error
+  }));
+
   req.on('timeout', common.mustCall(function() {
     console.log('timeout occurred outside');
     req.destroy();
     server.close();
-    process.exit(0);
   }));
 }));
diff --git a/test/parallel/test-timers-non-integer-delay.js b/test/parallel/test-timers-non-integer-delay.js
@@ -45,7 +45,6 @@ let N = 50;
 const interval = setInterval(common.mustCall(() => {
   if (--N === 0) {
     clearInterval(interval);
-    process.exit(0);
   }
 }, N), TIMEOUT_DELAY);