summaryrefslogtreecommitdiff
path: root/test
diff options
context:
space:
mode:
Diffstat (limited to 'test')
-rw-r--r--test/abort/test-abort-uncaught-exception.js2
-rw-r--r--test/async-hooks/test-callback-error.js2
-rw-r--r--test/common/index.js6
-rw-r--r--test/parallel/test-windows-abort-exitcode.js2
-rw-r--r--test/parallel/test-windows-failed-heap-allocation.js24
5 files changed, 30 insertions, 6 deletions
diff --git a/test/abort/test-abort-uncaught-exception.js b/test/abort/test-abort-uncaught-exception.js
index f984719376..fb30cfba3c 100644
--- a/test/abort/test-abort-uncaught-exception.js
+++ b/test/abort/test-abort-uncaught-exception.js
@@ -21,7 +21,7 @@ function run(flags, signals) {
child.on('exit', common.mustCall(function(code, sig) {
if (common.isWindows) {
if (signals)
- assert.strictEqual(code, 3);
+ assert.strictEqual(code, 0xC0000005);
else
assert.strictEqual(code, 1);
} else {
diff --git a/test/async-hooks/test-callback-error.js b/test/async-hooks/test-callback-error.js
index a4b8a99f33..b2093f74e2 100644
--- a/test/async-hooks/test-callback-error.js
+++ b/test/async-hooks/test-callback-error.js
@@ -101,7 +101,7 @@ assert.ok(!arg);
child.on('close', (code, signal) => {
clearTimeout(tO);
if (common.isWindows) {
- assert.strictEqual(code, 3);
+ assert.strictEqual(code, 134);
assert.strictEqual(signal, null);
} else {
assert.strictEqual(code, null);
diff --git a/test/common/index.js b/test/common/index.js
index 9358f86f65..54742319d2 100644
--- a/test/common/index.js
+++ b/test/common/index.js
@@ -605,10 +605,10 @@ exports.nodeProcessAborted = function nodeProcessAborted(exitCode, signal) {
// On Windows, 'aborts' are of 2 types, depending on the context:
// (i) Forced access violation, if --abort-on-uncaught-exception is on
// which corresponds to exit code 3221225477 (0xC0000005)
- // (ii) raise(SIGABRT) or abort(), which lands up in CRT library calls
- // which corresponds to exit code 3.
+ // (ii) Otherwise, _exit(134) which is called in place of abort() due to
+ // raising SIGABRT exiting with ambiguous exit code '3' by default
if (exports.isWindows)
- expectedExitCodes = [3221225477, 3];
+ expectedExitCodes = [0xC0000005, 134];
// When using --abort-on-uncaught-exception, V8 will use
// base::OS::Abort to terminate the process.
diff --git a/test/parallel/test-windows-abort-exitcode.js b/test/parallel/test-windows-abort-exitcode.js
index d61a91315b..e2e6570c1c 100644
--- a/test/parallel/test-windows-abort-exitcode.js
+++ b/test/parallel/test-windows-abort-exitcode.js
@@ -16,7 +16,7 @@ if (process.argv[2] === 'child') {
} else {
const child = spawn(process.execPath, [__filename, 'child']);
child.on('exit', common.mustCall((code, signal) => {
- assert.strictEqual(code, 3);
+ assert.strictEqual(code, 134);
assert.strictEqual(signal, null);
}));
}
diff --git a/test/parallel/test-windows-failed-heap-allocation.js b/test/parallel/test-windows-failed-heap-allocation.js
new file mode 100644
index 0000000000..465dcae2b6
--- /dev/null
+++ b/test/parallel/test-windows-failed-heap-allocation.js
@@ -0,0 +1,24 @@
+'use strict';
+const common = require('../common');
+
+// This test ensures that an out of memory error exits with code 134 on Windows
+
+if (!common.isWindows) return common.skip('Windows-only');
+
+const assert = require('assert');
+const { exec } = require('child_process');
+
+if (process.argv[2] === 'heapBomb') {
+ // heap bomb, imitates a memory leak quickly
+ const fn = (nM) => [...Array(nM)].map((i) => fn(nM ** 2));
+ fn(2);
+}
+
+// --max-old-space-size=3 is the min 'old space' in V8, explodes fast
+const cmd = `"${process.execPath}" --max-old-space-size=3 "${__filename}"`;
+exec(`${cmd} heapBomb`, common.mustCall((err) => {
+ const msg = `Wrong exit code of ${err.code}! Expected 134 for abort`;
+ // Note: common.nodeProcessAborted() is not asserted here because it
+ // returns true on 134 as well as 0xC0000005 (V8's base::OS::Abort)
+ assert.strictEqual(err.code, 134, msg);
+}));