fix: log unhandled error even if its stack is empty (#14619)

jestjs · Oct 13, 2023 · a30a52f · a30a52f
1 parent 5d63afa
commit a30a52f
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -31,6 +31,7 @@
 - `[jest-cli]` When specifying paths on the command line, only match against the relative paths of the test files ([#12519](https://github.com/facebook/jest/pull/12519))
   - [**BREAKING**] Changes `testPathPattern` configuration option to `testPathPatterns`, which now takes a list of patterns instead of the regex.
   - [**BREAKING**] `--testPathPattern` is now `--testPathPatterns`
+- `[jest-reporters, jest-runner]` Unhandled errors without stack get correctly logged to console ([#14619](https://github.com/facebook/jest/pull/14619))
 
 ### Performance
 

diff --git a/packages/jest-reporters/src/CoverageWorker.ts b/packages/jest-reporters/src/CoverageWorker.ts
@@ -33,7 +33,11 @@ export type CoverageWorkerData = {
 
 // Make sure uncaught errors are logged before we exit.
 process.on('uncaughtException', err => {
-  console.error(err.stack);
+  if (err.stack) {
+    console.error(err.stack);
+  } else {
+    console.error(err);
+  }
   exit(1);
 });
 

diff --git a/packages/jest-runner/src/testWorker.ts b/packages/jest-runner/src/testWorker.ts
@@ -35,7 +35,11 @@ type WorkerData = {
 
 // Make sure uncaught errors are logged before we exit.
 process.on('uncaughtException', err => {
-  console.error(err.stack);
+  if (err.stack) {
+    console.error(err.stack);
+  } else {
+    console.error(err);
+  }
   exit(1);
 });