Skip to content

Commit 046d651

Browse files
bzozMyles Borins
authored and
Myles Borins
committed
benchmark: fix child-process-exec-stdout on win
This benchmark fails on Windows when trying to execute command which is more than 32k in size. This commits skips this one case when running under Windows. PR-URL: #7178 Reviewed-By: Trott - Rich Trott <[email protected]> Reviewed-By: orangemocha - Alexis Campailla <[email protected]>
1 parent 236491e commit 046d651

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

benchmark/child_process/child-process-exec-stdout.js

+6-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,12 @@
11
'use strict';
22
const common = require('../common.js');
3+
4+
var messagesLength = [64, 256, 1024, 4096];
5+
// Windows does not support that long arguments
6+
if (process.platform !== 'win32')
7+
messagesLength.push(32768);
38
const bench = common.createBenchmark(main, {
4-
len: [64, 256, 1024, 4096, 32768],
9+
len: messagesLength,
510
dur: [5]
611
});
712

0 commit comments

Comments
 (0)