Skip to content

Commit

Permalink
test: fix flaky test-benchmark-events
Browse files Browse the repository at this point in the history
Make sure each benchmark file only runs one combination of options.
Allow for zero iterations for short benchmark.

PR-URL: #17472
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
  • Loading branch information
Trott authored and gibfahn committed Dec 20, 2017
1 parent 0f6ce05 commit d12d86e
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion test/parallel/test-benchmark-events.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,4 +4,6 @@ require('../common');

const runBenchmark = require('../common/benchmark');

runBenchmark('events', ['n=1']);
runBenchmark('events',
['argc=0', 'listeners=1', 'n=1'],
{ NODEJS_BENCHMARK_ZERO_ALLOWED: 1 });

0 comments on commit d12d86e

Please sign in to comment.