From 067d1f16de6120f233c11e7b6e6b137a65a7daeb Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Sun, 8 Oct 2017 21:23:49 -0700 Subject: [PATCH] test: allow short benchmarks for tests Enable short benchmarks for process and path benchmark tests. These have been observed to fail in CI due to returning with zero operations performed in the allotted time. We have a special environment variable for other benchmark tests that can be set to make that allowable in benchmarks. Set it for path and process. PR-URL: https://github.com/nodejs/node/pull/16097 Reviewed-By: Evan Lucas Reviewed-By: Minwoo Jung Reviewed-By: Gibson Fahnestock --- test/parallel/test-benchmark-path.js | 2 +- test/parallel/test-benchmark-process.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/test/parallel/test-benchmark-path.js b/test/parallel/test-benchmark-path.js index 9b73b921008..890fd7ae41f 100644 --- a/test/parallel/test-benchmark-path.js +++ b/test/parallel/test-benchmark-path.js @@ -11,4 +11,4 @@ runBenchmark('path', 'pathext=', 'paths=', 'props=' - ]); + ], { NODEJS_BENCHMARK_ZERO_ALLOWED: 1 }); diff --git a/test/parallel/test-benchmark-process.js b/test/parallel/test-benchmark-process.js index 08b28269159..c11d82b327f 100644 --- a/test/parallel/test-benchmark-process.js +++ b/test/parallel/test-benchmark-process.js @@ -9,4 +9,4 @@ runBenchmark('process', 'millions=0.000001', 'n=1', 'type=raw' - ]); + ], { NODEJS_BENCHMARK_ZERO_ALLOWED: 1 });