diff options
author | Santiago Gimeno <santiago.gimeno@gmail.com> | 2018-02-10 14:52:18 +0100 |
---|---|---|
committer | Ruben Bridgewater <ruben@bridgewater.de> | 2018-02-22 11:06:21 +0100 |
commit | 45982de4184e66bb53577b9c07383189f3d83dde (patch) | |
tree | 6e4a28ebc2efdf3340fdd333d9ee51bdbfd2cf12 /test/sequential/test-inspector-port-cluster.js | |
parent | 2990429b16f32c216d357a5cbf3f81d43aa8abaf (diff) | |
download | android-node-v8-45982de4184e66bb53577b9c07383189f3d83dde.tar.gz android-node-v8-45982de4184e66bb53577b9c07383189f3d83dde.tar.bz2 android-node-v8-45982de4184e66bb53577b9c07383189f3d83dde.zip |
cluster: fix inspector port assignment
Make sure that inspector ports in cluster are inside the valid range:
`[1024, 65535]`.
Fixes flaky `test-inspector-port-zero-cluster`.
PR-URL: https://github.com/nodejs/node/pull/18696
Fixes: https://github.com/nodejs/node/issues/18303
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Diffstat (limited to 'test/sequential/test-inspector-port-cluster.js')
-rw-r--r-- | test/sequential/test-inspector-port-cluster.js | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/test/sequential/test-inspector-port-cluster.js b/test/sequential/test-inspector-port-cluster.js index 84ec408ebf..87469aa7ff 100644 --- a/test/sequential/test-inspector-port-cluster.js +++ b/test/sequential/test-inspector-port-cluster.js @@ -24,6 +24,16 @@ function testRunnerMain() { workers: [{ expectedPort: 9230 }] }); + spawnMaster({ + execArgv: ['--inspect=65534'], + workers: [ + { expectedPort: 65535 }, + { expectedPort: 1024 }, + { expectedPort: 1025 }, + { expectedPort: 1026 } + ] + }); + let port = debuggerPort + offset++ * 5; spawnMaster({ |