From c8ee379d8582e4fbe446b98897932faa82519102 Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Wed, 30 May 2018 08:46:38 +0000
Subject: [PATCH] test: run crypto benchmark only once in tests

Prevent crypto benchmark files from running more than one benchmark
during tests.

PR-URL: https://github.com/nodejs/node/pull/21032
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
---
 benchmark/crypto/aes-gcm-throughput.js | 3 +++
 benchmark/crypto/cipher-stream.js      | 3 +++
 test/parallel/test-benchmark-crypto.js | 5 +++--
 3 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/benchmark/crypto/aes-gcm-throughput.js b/benchmark/crypto/aes-gcm-throughput.js
index 5c1e71e7280575..cd8f29c8c7d7b2 100644
--- a/benchmark/crypto/aes-gcm-throughput.js
+++ b/benchmark/crypto/aes-gcm-throughput.js
@@ -9,6 +9,9 @@ const bench = common.createBenchmark(main, {
 });
 
 function main({ n, len, cipher }) {
+  // Default cipher for tests.
+  if (cipher === '')
+    cipher = 'aes-128-gcm';
   const message = Buffer.alloc(len, 'b');
   const key = crypto.randomBytes(keylen[cipher]);
   const iv = crypto.randomBytes(12);
diff --git a/benchmark/crypto/cipher-stream.js b/benchmark/crypto/cipher-stream.js
index 64f6ff7b7292be..9f986dfb36b60d 100644
--- a/benchmark/crypto/cipher-stream.js
+++ b/benchmark/crypto/cipher-stream.js
@@ -10,6 +10,9 @@ const bench = common.createBenchmark(main, {
 });
 
 function main({ api, cipher, type, len, writes }) {
+  // Default cipher for tests.
+  if (cipher === '')
+    cipher = 'AES192';
   if (api === 'stream' && /^v0\.[0-8]\./.test(process.version)) {
     console.error('Crypto streams not available until v0.10');
     // use the legacy, just so that we can compare them.
diff --git a/test/parallel/test-benchmark-crypto.js b/test/parallel/test-benchmark-crypto.js
index 2e78d78bc93985..f24edf675f92f0 100644
--- a/test/parallel/test-benchmark-crypto.js
+++ b/test/parallel/test-benchmark-crypto.js
@@ -12,14 +12,15 @@ const runBenchmark = require('../common/benchmark');
 
 runBenchmark('crypto',
              [
-               'n=1',
                'algo=sha256',
                'api=stream',
+               'cipher=',
                'keylen=1024',
                'len=1',
+               'n=1',
                'out=buffer',
                'type=buf',
                'v=crypto',
-               'writes=1'
+               'writes=1',
              ],
              { NODEJS_BENCHMARK_ZERO_ALLOWED: 1 });