node/test/parallel/test-worker-cli-options.js
theanarkh c137d6eb31
src: fix execArgv in worker
PR-URL: https://github.com/nodejs/node/pull/53029
Fixes: https://github.com/nodejs/node/issues/53011
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
2024-05-23 15:27:50 +00:00

31 lines
1.3 KiB
JavaScript

// Flags: --expose-internals --expose-gc
'use strict';
require('../common');
const { Worker } = require('worker_threads');
const assert = require('assert');
const CODE = `
// If the --expose-internals flag does not pass to worker
// require function will throw an error
require('internal/options');
global.gc();
`;
// Test if the flags is passed to worker threads correctly
// and do not throw an error with the invalid execArgv
// when execArgv is inherited from parent
// See https://github.com/nodejs/node/issues/52825
// See https://github.com/nodejs/node/issues/53011
// Inherited env, execArgv from the parent will be ok
new Worker(CODE, { eval: true });
// Pass process.env explicitly and inherited execArgv from parent will be ok
new Worker(CODE, { eval: true, env: process.env });
// Inherited env from the parent and pass execArgv (Node.js options) explicitly will be ok
new Worker(CODE, { eval: true, execArgv: ['--expose-internals'] });
// Pass process.env and execArgv (Node.js options) explicitly will be ok
new Worker(CODE, { eval: true, env: process.env, execArgv: ['--expose-internals'] });
// Pass execArgv (V8 options) explicitly will throw an error
assert.throws(() => {
new Worker(CODE, { eval: true, execArgv: ['--expose-gc'] });
}, /ERR_WORKER_INVALID_EXEC_ARGV/);