Skip to content

Commit 5386b23

Browse files
bzozorangemocha
authored andcommitted
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 60891c6 commit 5386b23

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

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

Lines changed: 6 additions & 1 deletion
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)