Skip to content

test: refactor test-tls-enable-trace-cli.js

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

This test appears to be flakey on Unix. I'm not entirely sure why, but I think it may be related to the additional piping of the child process stderr (although this test is mostly a copy of another test that isn't exhibiting problems). This commit simplifies the test a bit.

Some preliminary testing seems promising:

If the stress test comes back green, I'd recommend fast tracking this.

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

Merge request reports

Loading