benchmark: shorten config name in http benchmark

Shorten the config name in check_invalid_header_char so it would not result in long lines that make the benchmark result hard to read. PR-URL: #18452 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Weijia Wang <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
nodejs · Feb 21, 2018 · 9ae513a · 9ae513a
1 parent 8c9b41a
commit 9ae513a
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/benchmark/http/check_invalid_header_char.js b/benchmark/http/check_invalid_header_char.js
@@ -3,6 +3,10 @@
 const common = require('../common.js');
 const _checkInvalidHeaderChar = require('_http_common')._checkInvalidHeaderChar;
 
+// Put it here so the benchmark result lines will not be super long.
+const LONG_AND_INVALID = 'Here is a value that is really a folded header ' +
+  'value\r\n  this should be supported, but it is not currently';
+
 const bench = common.createBenchmark(main, {
   key: [
     // Valid
@@ -21,8 +25,7 @@ const bench = common.createBenchmark(main, {
     'en-US',
 
     // Invalid
-    'Here is a value that is really a folded header value\r\n  this should be \
-     supported, but it is not currently',
+    'LONG_AND_INVALID',
     '中文呢', // unicode
     'foo\nbar',
     '\x7F'
@@ -31,6 +34,9 @@ const bench = common.createBenchmark(main, {
 });
 
 function main({ n, key }) {
+  if (key === 'LONG_AND_INVALID') {
+    key = LONG_AND_INVALID;
+  }
   bench.start();
   for (var i = 0; i < n; i++) {
     _checkInvalidHeaderChar(key);