mirror of
https://github.com/nodejs/node.git
synced 2024-11-21 10:59:27 +00:00
2bcf9995d2
Internal workers are essential to load user scripts and bootstrapped with internal entrypoints. They should not be waiting for inspectors even when `--inspect-brk` and `--inspect-wait` were specified, and avoid blocking main thread to bootstrap. IsolateData can be created with a specified PerIsolateOptions instead of creating a copy from the per_process namespace. This also avoids creating a copy bypassing the parent env's modified options, like creating a worker thread from a worker thread. PR-URL: https://github.com/nodejs/node/pull/54219 Fixes: https://github.com/nodejs/node/issues/53681 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
35 lines
1.0 KiB
JavaScript
35 lines
1.0 KiB
JavaScript
// This tests esm loader's internal worker will not be blocked by --inspect-brk.
|
|
// Regression: https://github.com/nodejs/node/issues/53681
|
|
|
|
'use strict';
|
|
const common = require('../common');
|
|
|
|
common.skipIfInspectorDisabled();
|
|
|
|
const assert = require('assert');
|
|
const fixtures = require('../common/fixtures');
|
|
const { NodeInstance } = require('../common/inspector-helper.js');
|
|
|
|
async function runIfWaitingForDebugger(session) {
|
|
const commands = [
|
|
{ 'method': 'Runtime.enable' },
|
|
{ 'method': 'Debugger.enable' },
|
|
{ 'method': 'Runtime.runIfWaitingForDebugger' },
|
|
];
|
|
|
|
await session.send(commands);
|
|
await session.waitForNotification('Debugger.paused');
|
|
}
|
|
|
|
async function runTest() {
|
|
const main = fixtures.path('es-module-loaders', 'register-loader.mjs');
|
|
const child = new NodeInstance(['--inspect-brk=0'], '', main);
|
|
|
|
const session = await child.connectInspectorSession();
|
|
await runIfWaitingForDebugger(session);
|
|
await session.runToCompletion();
|
|
assert.strictEqual((await child.expectShutdown()).exitCode, 0);
|
|
}
|
|
|
|
runTest();
|