Skip to content

test: fix truncation of argv

Rodrigo Muino Tomonari requested to merge github/fork/danbev/node_test_fixture_args_fix into master

Currently argv_[1] and argv_[2] are getting truncated by one character because of an incorrect addition of one to account for the null character. I only noticed this when working on #12087, but that fix will probably not get included in favor of a JavaScript test so I'm adding this separate commit for it.

Refs: https://github.com/nodejs/node/pull/12087

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

test

Merge request reports