test: move net bytes-per-chunk test to parallel

Browse Source

This was added in 3217e8e66fa81e787b9f3b18c0c09235f050acee as
a regression test for a security patch. We moved it to
`sequential` to lower the risk of creating a flaky test,
because an earlier version of it was failing one some platforms.

There is no known reason why te test should be flaky in this form,
though, and moving it to parallel would be good because it does take
around 3 seconds that would otherwise fully count towards the
test run time.

PR-URL: https://github.com/nodejs/node/pull/21322
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>

...

This commit is contained in:

Anna Henningsen 2018-06-13 16:22:23 +02:00

parent 7aa95670a1

commit d362b07e78

No known key found for this signature in database

GPG Key ID: 9C63F3A6CD2AD8F9

1 changed files with 0 additions and 0 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Show Stats Download Patch File Download Diff File Expand all files Collapse all files

0

test/sequential/test-net-bytes-per-incoming-chunk-overhead.js → test/parallel/test-net-bytes-per-incoming-chunk-overhead.js