mirror of
https://github.com/nodejs/node.git
synced 2024-11-21 10:59:27 +00:00
abe82110b7
Read-only access to `process.env` does not have side effects. Refs: https://github.com/nodejs/node/pull/27523 PR-URL: https://github.com/nodejs/node/pull/27684 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Aleksei Koziatinskii <ak239spb@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com> Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
25 lines
591 B
JavaScript
25 lines
591 B
JavaScript
'use strict';
|
|
const common = require('../common');
|
|
common.skipIfInspectorDisabled();
|
|
|
|
// Test that read-only process.env access is considered to have no
|
|
// side-effects by the inspector.
|
|
|
|
const assert = require('assert');
|
|
const inspector = require('inspector');
|
|
|
|
const session = new inspector.Session();
|
|
session.connect();
|
|
|
|
process.env.TESTVAR = 'foobar';
|
|
|
|
session.post('Runtime.evaluate', {
|
|
expression: 'process.env.TESTVAR',
|
|
throwOnSideEffect: true
|
|
}, (error, res) => {
|
|
assert.ifError(error);
|
|
assert.deepStrictEqual(res, {
|
|
result: { type: 'string', value: 'foobar' }
|
|
});
|
|
});
|