From d0943bcb693dc8a0cf0b903c04918c4ac6e694ea Mon Sep 17 00:00:00 2001 From: Anatoli Papirovski Date: Mon, 4 Jun 2018 15:08:16 +0200 Subject: [PATCH] test: move benchmark-dgram to sequential MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This test uses hardcoded ports, it should not be located in parallel. PR-URL: https://github.com/nodejs/node/pull/21144 Reviewed-By: Michaël Zasso Reviewed-By: Colin Ihrig Reviewed-By: Joyee Cheung Reviewed-By: Trivikram Kamat Reviewed-By: Rich Trott --- test/{parallel => sequential}/test-benchmark-dgram.js | 4 ++++ 1 file changed, 4 insertions(+) rename test/{parallel => sequential}/test-benchmark-dgram.js (63%) diff --git a/test/parallel/test-benchmark-dgram.js b/test/sequential/test-benchmark-dgram.js similarity index 63% rename from test/parallel/test-benchmark-dgram.js rename to test/sequential/test-benchmark-dgram.js index 69eafbcb13b..8c93cd5a0c9 100644 --- a/test/parallel/test-benchmark-dgram.js +++ b/test/sequential/test-benchmark-dgram.js @@ -4,6 +4,10 @@ require('../common'); const runBenchmark = require('../common/benchmark'); +// Because the dgram benchmarks use hardcoded ports, this should be in +// sequential rather than parallel to make sure it does not conflict with +// tests that choose random available ports. + runBenchmark('dgram', ['address=true', 'chunks=2', 'dur=0.1',