fix(node): setting process.exitCode should change exit code of process

cli/tests/unit_node/process_test.ts

@@ -681,9 +681,52 @@ Deno.test("process.memoryUsage.rss()", () => {
 });
 
 Deno.test("process.exitCode", () => {
- assert(process.exitCode === undefined);
+ assertEquals(process.exitCode, undefined);
  process.exitCode = 127;
- assert(process.exitCode === 127);
+ assertEquals(process.exitCode, 127);
+ process.exitCode = "asdf";
+ assertEquals(process.exitCode, "asdf");
+ process.exitCode = "10";
+ assertEquals(process.exitCode, "10");
+ process.exitCode = undefined; // reset
+});
+
+async function exitCodeTest(codeText: string, expectedExitCode: number) {
+ const command = new Deno.Command(Deno.execPath(), {
+ args: [
+ "eval",
+ codeText,
+ ],
+ cwd: testDir,
+ });
+ const { code } = await command.output();
+ assertEquals(code, expectedExitCode);
+}
+
+Deno.test("process.exitCode in should change exit code", async () => {
+ await exitCodeTest(
+ "import process from 'node:process'; process.exitCode = 127;",
+ 127,
+ );
+ await exitCodeTest(
+ "import process from 'node:process'; process.exitCode = '10';",
+ 10,
+ );
+ await exitCodeTest(
+ "import process from 'node:process'; process.exitCode = '0x10';",
+ 16,
+ );
+ await exitCodeTest(
+ "import process from 'node:process'; process.exitCode = NaN;",
+ 0,
+ );
+});
+
+Deno.test("Deno.exit should override process exit", async () => {
+ await exitCodeTest(
+ "import process from 'node:process'; process.exitCode = 10; Deno.exit(12);",
+ );
+ assertEquals(code, 12);
 });
 
 Deno.test("process.config", () => {

ext/node/polyfills/process.ts

@@ -76,15 +76,16 @@ const notImplementedEvents = [
 ];
 
 export const argv: string[] = [];
+let globalProcessExitCode: number | undefined = undefined;
 
 /** https://nodejs.org/api/process.html#process_process_exit_code */
 export const exit = (code?: number | string) => {
  if (code || code === 0) {
  if (typeof code === "string") {
  const parsedCode = parseInt(code);
- process.exitCode = isNaN(parsedCode) ? undefined : parsedCode;
+ globalProcessExitCode = isNaN(parsedCode) ? undefined : parsedCode;
  } else {
- process.exitCode = code;
+ globalProcessExitCode = code;
  }
  }
 
@@ -426,7 +427,17 @@ class Process extends EventEmitter {
  _exiting = _exiting;
 
  /** https://nodejs.org/api/process.html#processexitcode_1 */
- exitCode: undefined | number = undefined;
+ get exitCode() {
+ return globalProcessExitCode;
+ }
+
+ set exitCode(code: number | undefined) {
+ globalProcessExitCode = code;
+ code = (typeof code === "string" ? parseInt(code) : code) ?? 0;
+ if (typeof code === "number" && !isNaN(code)) {
+ ops.op_set_exit_code(code);
+ }
+ }
 
  // Typed as any to avoid importing "module" module for types
  // deno-lint-ignore no-explicit-any