diff options
author | Marvin Hagemeister <marvin@deno.com> | 2024-05-20 15:24:13 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-05-20 15:24:13 +0200 |
commit | fb3f82b9eafef7da732c0af7e8a86c3b87075e3c (patch) | |
tree | cffcb5edd9e5c8b3c7128a685ab26c5cf30af371 /tests/unit_node/process_test.ts | |
parent | d7709daaa09ebd40b5756a2ce6d3e1c0c3bf7984 (diff) |
fix(node): track `SIG*` listeners in `process.listeners` (#23890)
Some npm libraries like `signal-exit` rely on the length of the listener
array returned by `process.listeners("SIGNT")` to be correct to
function. We weren't tracking `SIG*` events there, which broke those npm
libraries.
Fixes https://github.com/denoland/deno/issues/22892
Diffstat (limited to 'tests/unit_node/process_test.ts')
-rw-r--r-- | tests/unit_node/process_test.ts | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/tests/unit_node/process_test.ts b/tests/unit_node/process_test.ts index 0eadb0a16..8f56c92a0 100644 --- a/tests/unit_node/process_test.ts +++ b/tests/unit_node/process_test.ts @@ -1108,3 +1108,19 @@ Deno.test({ process.constructor.call({}); }, }); + +// Test for https://github.com/denoland/deno/issues/22892 +Deno.test("process.listeners - include SIG* events", () => { + const listener = () => console.log("SIGINT"); + process.on("SIGINT", listener); + assertEquals(process.listeners("SIGINT").length, 1); + + const listener2 = () => console.log("SIGINT"); + process.prependListener("SIGINT", listener2); + assertEquals(process.listeners("SIGINT").length, 2); + + process.off("SIGINT", listener); + assertEquals(process.listeners("SIGINT").length, 1); + process.off("SIGINT", listener2); + assertEquals(process.listeners("SIGINT").length, 0); +}); |