Skip to content

Commit

Permalink
test: fix NODE_OPTIONS feature check
Browse files Browse the repository at this point in the history
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 18, 2019
1 parent 6015627 commit aa3c41f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/sequential/test-set-http-max-http-headers.js
Original file line number Diff line number Diff line change
Expand Up @@ -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`
});
Expand Down

0 comments on commit aa3c41f

Please sign in to comment.