test: ignore process.exit errors

Author: stefan-guggisberg

test/core/h2c.test.js

@@ -43,7 +43,9 @@ describe('unencrypted HTTP/2 (h2c)-specific Core Tests', () => {
 
   after(async () => {
     await reset();
-    process.kill(server.pid);
+    try {
+      process.kill(server.pid);
+    } catch (ignore) { /* ignore */ }
   });
 
   it('supports unencrypted HTTP/2 (h2c)', async () => {

test/core/index.test.js

@@ -60,7 +60,9 @@ describe('Core Tests', () => {
 
   after(async () => {
     await defaultCtx.reset();
-    process.kill(server.pid);
+    try {
+      process.kill(server.pid);
+    } catch (ignore) { /* ignore */ }
   });
 
   it('supports HTTP/1(.1)', async () => {
@@ -72,7 +74,9 @@ describe('Core Tests', () => {
       assert.strictEqual(resp.statusCode, 200);
       assert.strictEqual(resp.httpVersionMajor, 1);
     } finally {
-      process.kill(h1Server.pid);
+      try {
+        process.kill(h1Server.pid);
+      } catch (ignore) { /* ignore */ }
     }
   });
 
@@ -336,7 +340,9 @@ describe('Core Tests', () => {
       assert.strictEqual(ok.length, N);
     } finally {
       await ctx.reset();
-      process.kill(testServer.pid);
+      try {
+        process.kill(testServer.pid);
+      } catch (ignore) { /* ignore */ }
     }
   });
 

test/core/misc.test.js

@@ -65,7 +65,9 @@ describe('Misc. Core Tests (edge cases to improve code coverage)', () => {
       assert(err instanceof RequestAbortedError);
     } finally {
       await customCtx.reset();
-      process.kill(server.pid);
+      try {
+        process.kill(server.pid);
+      } catch (ignore) { /* ignore */ }
     }
     const ts1 = Date.now();
     assert((ts1 - ts0) < 10);
@@ -93,7 +95,9 @@ describe('Misc. Core Tests (edge cases to improve code coverage)', () => {
       assert.strictEqual(json.body, body);
     } finally {
       await customCtx.reset();
-      process.kill(server.pid);
+      try {
+        process.kill(server.pid);
+      } catch (ignore) { /* ignore */ }
     }
   });
 });

test/fetch/cache.test.js

@@ -71,7 +71,9 @@ describe('Cache Tests', () => {
   });
 
   after(async () => {
-    process.kill(server.pid);
+    try {
+      process.kill(server.pid);
+    } catch (ignore) { /* ignore */ }
   });
 
   afterEach(async () => {

test/fetch/index.http1.test.js

@@ -52,7 +52,9 @@ testParams.forEach((params) => {
     });
 
     after(async () => {
-      process.kill(server.pid);
+      try {
+        process.kill(server.pid);
+      } catch (ignore) { /* ignore */ }
     });
 
     it(`forcing HTTP/1.1 using context option works' (${name})`, async () => {

test/fetch/index.http2.test.js

@@ -38,7 +38,9 @@ describe('HTTP/2-specific Fetch Tests', () => {
   });
 
   after(async () => {
-    process.kill(server.pid);
+    try {
+      process.kill(server.pid);
+    } catch (ignore) { /* ignore */ }
   });
 
   afterEach(async () => {

test/fetch/index.test.js

@@ -577,7 +577,9 @@ testParams.forEach((params) => {
         assert(json !== null && typeof json === 'object');
         assert.strictEqual(json.headers.authorization, undefined);
       } finally {
-        process.kill(targetServer.pid);
+        try {
+          process.kill(targetServer.pid);
+        } catch (ignore) { /* ignore */ }
       }
     });
 
@@ -776,7 +778,9 @@ testParams.forEach((params) => {
           assert.strictEqual(body, HELLO_WORLD);
         } finally {
           await ctx.reset();
-          process.kill(server.pid);
+          try {
+            process.kill(server.pid);
+          } catch (ignore) { /* ignore */ }
         }
       });
     }

test/fetch/redirect.test.js

@@ -40,7 +40,9 @@ describe('Redirect-specific Fetch Tests', () => {
     } finally {
       await ctx.reset();
       // shutdown server
-      process.kill(server.pid);
+      try {
+        process.kill(server.pid);
+      } catch (ignore) { /* ignore */ }
     }
   });
 
@@ -66,7 +68,9 @@ describe('Redirect-specific Fetch Tests', () => {
     } finally {
       await ctx.reset();
       // shutdown server
-      process.kill(server.pid);
+      try {
+        process.kill(server.pid);
+      } catch (ignore) { /* ignore */ }
     }
   });
 });

test/fetch/resiliance.test.js

@@ -33,7 +33,9 @@ describe('Fetch Resiliance Tests', () => {
       assert.strictEqual(body, HELLO_MSG);
 
       // restart server
-      process.kill(server.pid);
+      try {
+        process.kill(server.pid);
+      } catch (ignore) { /* ignore */ }
       server = await Server.launch(2, true, HELLO_MSG, server.port);
 
       resp = await ctx.fetch(`${server.origin}/hello`);
@@ -43,7 +45,9 @@ describe('Fetch Resiliance Tests', () => {
       assert.strictEqual(body, HELLO_MSG);
     } finally {
       await ctx.reset();
-      process.kill(server.pid);
+      try {
+        process.kill(server.pid);
+      } catch (ignore) { /* ignore */ }
     }
   });
 
@@ -60,7 +64,9 @@ describe('Fetch Resiliance Tests', () => {
       assert.strictEqual(body, HELLO_MSG);
 
       // stop h2 server
-      process.kill(server.pid);
+      try {
+        process.kill(server.pid);
+      } catch (ignore) { /* ignore */ }
       // start h1 server
       server = await Server.launch(1, true, HELLO_MSG, server.port);
       // expect FetchError: Protocol error (message depends on node version)
@@ -73,7 +79,9 @@ describe('Fetch Resiliance Tests', () => {
       assert.strictEqual(body, HELLO_MSG);
     } finally {
       await ctx.reset();
-      process.kill(server.pid);
+      try {
+        process.kill(server.pid);
+      } catch (ignore) { /* ignore */ }
     }
   });
 
@@ -100,7 +108,9 @@ describe('Fetch Resiliance Tests', () => {
       assert.strictEqual(body, HELLO_MSG);
     } finally {
       await ctx.reset();
-      process.kill(server.pid);
+      try {
+        process.kill(server.pid);
+      } catch (ignore) { /* ignore */ }
     }
   });
 });