test: accept SIGILL aborts in async-hooks tests

trivikr · aduh95 · commit 85487ceaf69c · 2026-06-25T16:07:34.000+02:00
Use common.nodeProcessAborted() for the intentional abort path so platform-specific abort signals such as SIGILL are accepted. Signed-off-by: Kamat, Trivikram <16024985+trivikr@users.noreply.github.com> Assisted-by: openai:gpt-5.5 PR-URL: #63687 Refs: https://github.com/nodejs/reliability/blob/main/reports/2026-06-01.md#jstest-failure Reviewed-By: Richard Lau <richard.lau@ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
diff --git a/test/async-hooks/test-emit-after-on-destroyed.js b/test/async-hooks/test-emit-after-on-destroyed.js
@@ -55,7 +55,7 @@ if (process.argv[2] === 'child') {
   child.on('close', common.mustCall((code, signal) => {
     if ((common.isAIX ||
          (common.isLinux && process.arch === 'x64')) &&
-         signal === 'SIGABRT') {
+        common.nodeProcessAborted(code, signal)) {
       // XXX: The child process could be aborted due to unknown reasons. Work around it.
     } else {
       assert.strictEqual(signal, null);
diff --git a/test/async-hooks/test-improper-order.js b/test/async-hooks/test-improper-order.js
@@ -53,8 +53,8 @@ if (process.argv[2] === 'child') {
 
   child.on('close', common.mustCall((code, signal) => {
     if ((common.isAIX ||
-        (common.isLinux && process.arch === 'x64')) &&
-        signal === 'SIGABRT') {
+         (common.isLinux && process.arch === 'x64')) &&
+        common.nodeProcessAborted(code, signal)) {
       // XXX: The child process could be aborted due to unknown reasons. Work around it.
     } else {
       assert.strictEqual(signal, null);
diff --git a/test/async-hooks/test-improper-unwind.js b/test/async-hooks/test-improper-unwind.js
@@ -58,7 +58,7 @@ if (process.argv[2] === 'child') {
   child.on('close', common.mustCall((code, signal) => {
     if ((common.isAIX ||
          (common.isLinux && process.arch === 'x64')) &&
-         signal === 'SIGABRT') {
+        common.nodeProcessAborted(code, signal)) {
       // XXX: The child process could be aborted due to unknown reasons. Work around it.
     } else {
       assert.strictEqual(signal, null);
