@empiricalrun/test-run 0.7.5 → 0.7.6

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @empiricalrun/test-run
 
+## 0.7.6
+
+### Patch Changes
+
+- d04190f: fix: remove process.on listeners to avoid leaks
+
 ## 0.7.5
 
 ### Patch Changes

package/dist/lib/run-specific-test.d.ts

@@ -1,9 +1,4 @@
 import { TestRunParameters } from "../types";
-/**
- *
- * @export
- * @param {TestRunParameters} { name, dir, pwOptions }
- */
 export declare function runSpecificTests({ tests, projects, pwOptions, platform, }: TestRunParameters): Promise<{
     hasTestPassed: boolean;
 }>;

package/dist/lib/run-specific-test.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"run-specific-test.d.ts","sourceRoot":"","sources":["../../src/lib/run-specific-test.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,iBAAiB,EAAE,MAAM,UAAU,CAAC;AAY7C;;;;GAIG;AACH,wBAAsB,gBAAgB,CAAC,EACrC,KAAU,EACV,QAAQ,EACR,SAAS,EACT,QAAQ,GACT,EAAE,iBAAiB,GAAG,OAAO,CAAC;IAC7B,aAAa,EAAE,OAAO,CAAC;CACxB,CAAC,CA0FD"}
+{"version":3,"file":"run-specific-test.d.ts","sourceRoot":"","sources":["../../src/lib/run-specific-test.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,iBAAiB,EAAE,MAAM,UAAU,CAAC;AAoB7C,wBAAsB,gBAAgB,CAAC,EACrC,KAAU,EACV,QAAQ,EACR,SAAS,EACT,QAAQ,GACT,EAAE,iBAAiB,GAAG,OAAO,CAAC;IAC7B,aAAa,EAAE,OAAO,CAAC;CACxB,CAAC,CAuFD"}

package/dist/lib/run-specific-test.js

@@ -7,11 +7,13 @@ exports.runSpecificTests = void 0;
 const fs_1 = require("fs");
 const promises_1 = __importDefault(require("fs/promises"));
 const utils_1 = require("../utils");
-/**
- *
- * @export
- * @param {TestRunParameters} { name, dir, pwOptions }
- */
+function setupProcessListeners(cleanup) {
+    const events = ["beforeExit", "exit", "SIGINT", "SIGTERM"];
+    events.forEach((event) => process.on(event, cleanup));
+    return () => {
+        events.forEach((event) => process.removeListener(event, cleanup));
+    };
+}
 async function runSpecificTests({ tests = [], projects, pwOptions, platform, }) {
     if (!tests || tests.length === 0) {
         throw new Error("No tests found");
@@ -72,11 +74,7 @@ async function runSpecificTests({ tests = [], projects, pwOptions, platform, })
             });
         }
     }
-    // TODO: check if this works for multiple listeners
-    process.on("beforeExit", revertFileContent);
-    process.on("exit", revertFileContent);
-    process.on("SIGINT", revertFileContent);
-    process.on("SIGTERM", revertFileContent);
+    const removeListeners = setupProcessListeners(revertFileContent);
     let hasTestPassed = true;
     try {
         const testRunner = (0, utils_1.getTestRunner)(platform);
@@ -91,6 +89,7 @@ async function runSpecificTests({ tests = [], projects, pwOptions, platform, })
     catch (e) {
         hasTestPassed = false;
     }
+    removeListeners();
     revertFileContent();
     return {
         hasTestPassed,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@empiricalrun/test-run",
-  "version": "0.7.5",
+  "version": "0.7.6",
   "publishConfig": {
     "registry": "https://registry.npmjs.org/",
     "access": "public"