fix(node): don't emit exit twice (#1753)

node/process.ts

@@ -218,11 +218,11 @@ class Process extends EventEmitter {
   constructor() {
     super();
 
-    //This causes the exit event to be binded to the unload event
     globalThis.addEventListener("unload", () => {
-      //TODO(Soremwar)
-      //Get the exit code from the unload event
-      super.emit("exit", 0);
+      if (!process._exiting) {
+        process._exiting = true;
+        super.emit("exit", process.exitCode || 0);
+      }
     });
   }
 

node/process_test.ts

@@ -412,3 +412,30 @@ Deno.test("process._exiting", () => {
 Deno.test("process.execPath", () => {
   assertEquals(process.execPath, process.argv[0]);
 });
+
+Deno.test({
+  name: "process.exit",
+  async fn() {
+    const cwd = path.dirname(path.fromFileUrl(import.meta.url));
+
+    const p = Deno.run({
+      cmd: [
+        Deno.execPath(),
+        "run",
+        "--quiet",
+        "--unstable",
+        "./testdata/process_exit2.ts",
+      ],
+      cwd,
+      stdout: "piped",
+    });
+
+    const decoder = new TextDecoder();
+    const rawOutput = await p.output();
+    assertEquals(
+      stripColor(decoder.decode(rawOutput).trim()),
+      "exit",
+    );
+    p.close();
+  },
+});

node/testdata/process_exit2.ts

@@ -0,0 +1,5 @@
+import "../global.ts";
+
+//deno-lint-ignore no-undef
+process.on("exit", () => console.log("exit"));
+process.exit();