Skip to content
Permalink
Browse files

test: fix NODE_OPTIONS feature check

The configuration variable being tested is `true` if Node.js was
compiled without support for NODE_OPTIONS.

PR-URL: #28225
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
  • Loading branch information...
richardlau authored and targos committed Jun 14, 2019
1 parent 6015627 commit aa3c41fe406ccacfe29f4517de53e5965f5fa3c8
Showing with 1 addition and 1 deletion.
  1. +1 −1 test/sequential/test-set-http-max-http-headers.js
@@ -60,7 +60,7 @@ parsers.forEach((parser) => {
});

// Next, repeat the same checks using NODE_OPTIONS if it is supported.
if (process.config.variables.node_without_node_options) {
if (!process.config.variables.node_without_node_options) {
const env = Object.assign({}, process.env, {
NODE_OPTIONS: `--http-parser=${parser} --max-http-header-size=1024`
});

0 comments on commit aa3c41f

Please sign in to comment.
You can’t perform that action at this time.