diff options
author | Bartosz Sosnowski <bartosz@janeasystems.com> | 2016-05-31 19:25:15 +0300 |
---|---|---|
committer | Myles Borins <mborins@us.ibm.com> | 2016-07-14 22:45:24 +0300 |
commit | 046d651118d91c2bc04ae3d9913eb2167d02ad43 (patch) | |
tree | c1ef08e97c0f2f1b3250b99942a2023c0af65810 | |
parent | 236491e6981fe4ef291e130122b3c8465bc3cc6c (diff) |
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: https://github.com/nodejs/node/pull/7178
Reviewed-By: Trott - Rich Trott <rtrott@gmail.com>
Reviewed-By: orangemocha - Alexis Campailla <orangemocha@nodejs.org>
-rw-r--r-- | benchmark/child_process/child-process-exec-stdout.js | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/benchmark/child_process/child-process-exec-stdout.js b/benchmark/child_process/child-process-exec-stdout.js index 79efb6fadf2..6194f66b55d 100644 --- a/benchmark/child_process/child-process-exec-stdout.js +++ b/benchmark/child_process/child-process-exec-stdout.js @@ -1,7 +1,12 @@ 'use strict'; const common = require('../common.js'); + +var messagesLength = [64, 256, 1024, 4096]; +// Windows does not support that long arguments +if (process.platform !== 'win32') + messagesLength.push(32768); const bench = common.createBenchmark(main, { - len: [64, 256, 1024, 4096, 32768], + len: messagesLength, dur: [5] }); |