mirror of
https://github.com/nodejs/node.git
synced 2024-12-01 16:10:02 +01:00
7f496fefb6
`Environment::GetCurrent` may not available in the context of OOM. Removes the cyclic `Environment::GetCurrent` and `env->isolate()` calls to ensure both `isolate` and `env` is present if available. However, this behavior is not guaranteed. As `Environment::GetCurrent` didn't allocate new handles in the heap, when a Context is entered it can still get the valid env pointer. Removes the unstable assertion of the absence of env in the test. PR-URL: https://github.com/nodejs/node/pull/44398 Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
41 lines
1.3 KiB
JavaScript
41 lines
1.3 KiB
JavaScript
'use strict';
|
|
|
|
// Testcases for situations involving fatal errors, like Javascript heap OOM
|
|
|
|
require('../common');
|
|
const assert = require('assert');
|
|
const fs = require('fs');
|
|
const helper = require('../common/report.js');
|
|
const spawnSync = require('child_process').spawnSync;
|
|
const tmpdir = require('../common/tmpdir');
|
|
const fixtures = require('../common/fixtures');
|
|
|
|
// Common args that will cause an out-of-memory error for child process.
|
|
const ARGS = [
|
|
'--max-heap-size=20',
|
|
fixtures.path('report-oom'),
|
|
];
|
|
const REPORT_FIELDS = [
|
|
['header.trigger', 'OOMError'],
|
|
['javascriptHeap.memoryLimit', 20 * 1024 * 1024 /* 20MB */],
|
|
];
|
|
|
|
{
|
|
// Verify that --report-directory is respected when set.
|
|
// Verify that --report-compact is respected when not set.
|
|
tmpdir.refresh();
|
|
const dir = '--report-directory=' + tmpdir.path;
|
|
const args = ['--report-on-fatalerror', dir, ...ARGS];
|
|
const child = spawnSync(process.execPath, args, { });
|
|
assert.notStrictEqual(child.status, 0, 'Process exited unexpectedly');
|
|
|
|
const reports = helper.findReports(child.pid, tmpdir.path);
|
|
assert.strictEqual(reports.length, 1);
|
|
|
|
const report = reports[0];
|
|
helper.validate(report, REPORT_FIELDS);
|
|
|
|
const lines = fs.readFileSync(report, 'utf8').split('\n').length - 1;
|
|
assert(lines > 10);
|
|
}
|