Skip to content

test: robust handling of env for npm-test-install

Rodrigo Muino Tomonari requested to merge github/fork/MylesBorins/robust-npm-test-env into master
Checklist
  • tests and code linting passes
  • a test and/or benchmark is included
  • the commit message follows commit guidelines
Affected core subsystem(s)

test

Description of change

Currently we are overwriting the entire env object of the child-process spawned in npm-test-install. This commit alternatively clones the process.env object and modifies it with the neccessary changes before passing it the the spawned process.

Fixes: https://github.com/nodejs/node/issues/6736

Merge request reports