Skip to content

test: add missing console.error to exec-maxBuffer

Rodrigo Muino Tomonari requested to merge github/fork/BethGriggs/fix-child-process-exec-maxbuff into master

While looking at this test I noticed that the final cmd calls console.('${unicode}'). To me, it wasn't clear if this was intentional or if it should be console.error('${unicode}').

https://github.com/nodejs/node/blob/master/test/parallel/test-child-process-exec-maxBuffer.js#L39

If it was intentional I am happy for this to be closed and/or I can add a comment to the test.

/cc @trott as author of the test case

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

test

Merge request reports