From 1e35d5802a265d738f6fb45b7df1f6141dd6d676 Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Mon, 4 Jan 2021 19:21:43 -0800
Subject: [PATCH] benchmark: fix http/headers.js with test-double
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
The http/headers.js benchmark fails if wrk or autocannon are not
installed. This is because the benchmark exceeds the default permissible
length for HTTP headers. Reduce the largest benchmarks to fit within the
8192 default as that is what we should be optimizing for.
Fixes: https://github.com/nodejs/node/issues/31022
PR-URL: https://github.com/nodejs/node/pull/36794
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
---
benchmark/http/headers.js | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/benchmark/http/headers.js b/benchmark/http/headers.js
index b83ac17e742a2e..a3d2b7810be676 100644
--- a/benchmark/http/headers.js
+++ b/benchmark/http/headers.js
@@ -4,7 +4,7 @@ const common = require('../common.js');
const http = require('http');
const bench = common.createBenchmark(main, {
- n: [10, 1000],
+ n: [10, 600],
len: [1, 100],
duration: 5
});