From eef94a8bf827b98cd27e8aaab6bb20791a46bace Mon Sep 17 00:00:00 2001 From: Gibson Fahnestock Date: Fri, 2 Jun 2017 10:54:03 +0100 Subject: [PATCH] test: pass env vars through to test-benchmark-http Allows NODE_TEST_DIR to be set (necessary to avoid path length issues with common.PIPE). PR-URL: https://github.com/nodejs/node/pull/13390 Refs: https://github.com/nodejs/node/issues/12708#issuecomment-297847882 Reviewed-By: Joyee Cheung Reviewed-By: Refael Ackermann Reviewed-By: Colin Ihrig Reviewed-By: Rich Trott Reviewed-By: James M Snell Reviewed-By: Michael Dawson --- test/sequential/test-benchmark-http.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/test/sequential/test-benchmark-http.js b/test/sequential/test-benchmark-http.js index cbeafcb8e6f19a..ce3d58e5d8d4b8 100644 --- a/test/sequential/test-benchmark-http.js +++ b/test/sequential/test-benchmark-http.js @@ -20,6 +20,9 @@ const path = require('path'); const runjs = path.join(__dirname, '..', '..', 'benchmark', 'run.js'); +const env = Object.assign({}, process.env, + { NODEJS_BENCHMARK_ZERO_ALLOWED: 1 }); + const child = fork(runjs, ['--set', 'benchmarker=test-double', '--set', 'c=1', '--set', 'chunks=0', @@ -28,7 +31,7 @@ const child = fork(runjs, ['--set', 'benchmarker=test-double', '--set', 'len=1', '--set', 'n=1', 'http'], - {env: {NODEJS_BENCHMARK_ZERO_ALLOWED: 1}}); + {env}); child.on('exit', (code, signal) => { assert.strictEqual(code, 0); assert.strictEqual(signal, null);