fix: signal listener removal using different function references

Author: edilson258

lib/internal/test_runner/harness.js

@@ -285,8 +285,8 @@ function setupProcessState(root, globalOptions) {
     process.removeListener('unhandledRejection', rejectionHandler);
     process.removeListener('beforeExit', exitHandler);
     if (globalOptions.isTestRunner) {
-      process.removeListener('SIGINT', () => terminationHandler(kSigInt));
-      process.removeListener('SIGTERM', () => terminationHandler(kSigTerm));
+      process.removeListener('SIGINT', sigIntHandler);
+      process.removeListener('SIGTERM', sigTermHandler);
     }
   };
 
@@ -321,13 +321,16 @@ function setupProcessState(root, globalOptions) {
     process.exit(exitCode);
   };
 
+  const sigIntHandler = () => terminationHandler(kSigInt);
+  const sigTermHandler = () => terminationHandler(kSigTerm);
+
   process.on('uncaughtException', exceptionHandler);
   process.on('unhandledRejection', rejectionHandler);
   process.on('beforeExit', exitHandler);
   // TODO(MoLow): Make it configurable to hook when isTestRunner === false.
   if (globalOptions.isTestRunner) {
-    process.on('SIGINT', () => terminationHandler(kSigInt));
-    process.on('SIGTERM', () => terminationHandler(kSigTerm));
+    process.on('SIGINT', sigIntHandler);
+    process.on('SIGTERM', sigTermHandler);
   }
 
   root.harness.coverage = FunctionPrototypeBind(collectCoverage, null, root, coverage);