Report unhandled promise rejection on exit and make exit code 1 instead of 0 (#1734)

Co-authored-by: Jarred Sumner <709451+Jarred-Sumner@users.noreply.github.com>

5 files changed, 43 insertions, 0 deletions

diff --git a/test/bun.js/exit-code-0.js b/test/bun.js/exit-code-0.js
new file mode 100644
index 000000000..dcbbff6c9
--- /dev/null
+++ b/test/bun.js/exit-code-0.js
@@ -0,0 +1 @@
+process.exit(0);
diff --git a/test/bun.js/exit-code-1.js b/test/bun.js/exit-code-1.js
new file mode 100644
index 000000000..6cee2e1e7
--- /dev/null
+++ b/test/bun.js/exit-code-1.js
@@ -0,0 +1 @@
+process.exit(1);
diff --git a/test/bun.js/exit-code-await-throw-1.js b/test/bun.js/exit-code-await-throw-1.js
new file mode 100644
index 000000000..6b8c42eab
--- /dev/null
+++ b/test/bun.js/exit-code-await-throw-1.js
@@ -0,0 +1,3 @@
+await (async function () {
+  throw 42;
+})();
diff --git a/test/bun.js/exit-code-unhandled-throw.js b/test/bun.js/exit-code-unhandled-throw.js
new file mode 100644
index 000000000..e8f5ca4cb
--- /dev/null
+++ b/test/bun.js/exit-code-unhandled-throw.js
@@ -0,0 +1,3 @@
+(async function () {
+  throw 42;
+})();
diff --git a/test/bun.js/exit-code.test.ts b/test/bun.js/exit-code.test.ts
new file mode 100644
index 000000000..b7d91f8e9
--- /dev/null
+++ b/test/bun.js/exit-code.test.ts
@@ -0,0 +1,35 @@
+import { describe, expect, it, test } from "bun:test";
+import { bunExe } from "bunExe";
+import { spawnSync } from "bun";
+
+it("process.exit(1) works", () => {
+  const { exitCode } = spawnSync([
+    bunExe(),
+    import.meta.dir + "/exit-code-1.js",
+  ]);
+  expect(exitCode).toBe(1);
+});
+
+it("await on a thrown value reports exit code 1", () => {
+  const { exitCode } = spawnSync([
+    bunExe(),
+    import.meta.dir + "/exit-code-await-throw-1.js",
+  ]);
+  expect(exitCode).toBe(1);
+});
+
+it("unhandled promise rejection reports exit code 1", () => {
+  const { exitCode } = spawnSync([
+    bunExe(),
+    import.meta.dir + "/exit-code-unhandled-throw.js",
+  ]);
+  expect(exitCode).toBe(1);
+});
+
+it("process.exit(0) works", () => {
+  const { exitCode } = spawnSync([
+    bunExe(),
+    import.meta.dir + "/exit-code-0.js",
+  ]);
+  expect(exitCode).toBe(0);
+});