Skip to content

test: use duplex streams in duplex stream test

Rodrigo Muino Tomonari requested to merge github/fork/cjihrig/duplex into master

test-stream-duplex.js uses transform streams instead of the duplex stream base class. This leads to some code not being covered in the Duplex constructor.

This may lead to the equivalent code in the Transform constructor losing coverage. If that's the case, I'd prefer to fix that in a follow up PR.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
Affected core subsystem(s)

test

Merge request reports