Skip to content

Commit

Permalink
test: add test for tls benchmarks
Browse files Browse the repository at this point in the history
PR-URL: #18489
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
apapirovski authored and MylesBorins committed Feb 21, 2018
1 parent 958f5ed commit 0d390f7
Show file tree
Hide file tree
Showing 2 changed files with 26 additions and 1 deletion.
2 changes: 1 addition & 1 deletion benchmark/tls/throughput.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,11 +40,11 @@ function main({ dur, type, size }) {
};

server = tls.createServer(options, onConnection);
setTimeout(done, dur * 1000);
var conn;
server.listen(common.PORT, function() {
const opt = { port: common.PORT, rejectUnauthorized: false };
conn = tls.connect(opt, function() {
setTimeout(done, dur * 1000);
bench.start();
conn.on('drain', write);
write();
Expand Down
25 changes: 25 additions & 0 deletions test/sequential/test-benchmark-tls.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
'use strict';

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

if (!common.enoughTestMem)
common.skip('Insufficient memory for TLS benchmark test');

// Because the TLS benchmarks use hardcoded ports, this should be in sequential
// rather than parallel to make sure it does not conflict with tests that choose
// random available ports.

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

runBenchmark('tls',
[
'concurrency=1',
'dur=0.1',
'n=1',
'size=2',
'type=asc'
],
{
NODEJS_BENCHMARK_ZERO_ALLOWED: 1,
duration: 0
});

0 comments on commit 0d390f7

Please sign in to comment.