Merge pull request #762 from gemini-testing/HERMIONE-818.debug

fix: handle unhandled rejections with circular refs on timeout error
gemini-testing · Apr 27, 2023 · 17d9aa0 · 17d9aa0
2 parents d59ac1a + 8c38c66
commit 17d9aa0
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 2 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -67,6 +67,7 @@
     "mocha": "^10.2.0",
     "plugins-loader": "^1.1.0",
     "png-validator": "1.1.0",
+    "serialize-error": "^8.1.0",
     "sharp": "~0.30.7",
     "sizzle": "^2.3.6",
     "temp": "^0.8.3",

diff --git a/src/utils/processor.js b/src/utils/processor.js
@@ -1,5 +1,6 @@
 "use strict";
 
+const { serializeError, deserializeError } = require("serialize-error");
 const { WORKER_UNHANDLED_REJECTION } = require("../constants/process-messages");
 const ipc = require("./ipc");
 
@@ -15,7 +16,16 @@ module.exports = async (module, methodName, args, cb) => {
     try {
         const result = await require(module)[methodName](...args);
         cb(null, result);
-    } catch (e) {
-        cb(e);
+    } catch (err) {
+        sendError(err, cb);
     }
 };
+
+function sendError(err, cb) {
+    try {
+        cb(err);
+    } catch {
+        const errWithoutCircularRefs = deserializeError(serializeError(err));
+        cb(errWithoutCircularRefs);
+    }
+}