Skip to content

Commit

Permalink
test: fix typo in test-cli-node-options.js
Browse files Browse the repository at this point in the history
`expect` was probably a typo, as it is a function
and will always be `undefined` as JSON.

The previous `test()` check infers it should be `want`.

PR-URL: #13558
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
  • Loading branch information
vsemozhetbyt authored and addaleax committed Jun 12, 2017
1 parent 32c87ac commit e713482
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/parallel/test-cli-node-options.js
Original file line number Diff line number Diff line change
Expand Up @@ -72,8 +72,8 @@ function expect(opt, want) {
assert.ifError(err);
if (!RegExp(want).test(stdout)) {
console.error('For %j, failed to find %j in: <\n%s\n>',
opt, expect, stdout);
assert(false, `Expected ${expect}`);
opt, want, stdout);
assert.fail(`Expected ${want}`);
}
}));
}

0 comments on commit e713482

Please sign in to comment.