delta/node-new.git/benchmark/tls, branch try_Python3_on_macOS

test: move test_[key|ca|cert] to fixtures/keys/

2019-06-10T16:56:55+00:00

Lots of changes, but mostly just search/replace of
fixtures.readSync(...) to fixtures.readKey([new key]...)

Benchmarks modified to use fixtures.readKey(...):
benchmark/tls/throughput.js
benchmark/tls/tls-connect.js
benchmark/tls/secure-pair.js

Also be sure to review the change to L16 of
test/parallel/test-crypto-sign-verify.js

PR-URL: https://github.com/nodejs/node/pull/27962
Reviewed-By: Sam Roberts 
Reviewed-By: Ujjwal Sharma 
Reviewed-By: Rich Trott

Revert "test: move all test keys/certs under `test/fixtures/keys/`"

2019-06-05T23:01:13+00:00

PR-URL: https://github.com/nodejs/node/pull/28083
Reviewed-By: Anna Henningsen 
Reviewed-By: Ali Ijaz Sheikh 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Refael Ackermann (רפאל פלחי)

test: move test_[key|ca|cert] to fixtures/keys/

2019-06-05T18:37:43+00:00

Lots of changes, but mostly just search/replace of
fixtures.readSync(...) to fixtures.readKey([new key]...)

Benchmarks modified to use fixtures.readKey(...):
benchmark/tls/throughput.js
benchmark/tls/tls-connect.js
benchmark/tls/secure-pair.js

Also be sure to review the change to L16 of
test/parallel/test-crypto-sign-verify.js

PR-URL: https://github.com/nodejs/node/pull/27962
Reviewed-By: Sam Roberts 
Reviewed-By: Ujjwal Sharma 
Reviewed-By: Rich Trott

benchmark,lib: change var to const

2019-03-30T12:16:39+00:00

Refs: https://github.com/nodejs/node/pull/26679

PR-URL: https://github.com/nodejs/node/pull/26915
Reviewed-By: Michaël Zasso 
Reviewed-By: Tobias Nießen 
Reviewed-By: Refael Ackermann

benchmark: refactor for consistent style

2019-02-07T06:18:31+00:00

Code in benchmark directory sometimes uses `function () {}` for
anonymous callbacks and sometimes uses `() => {}`. Multi-line arrays
sometimes have a trailing comma and sometimes do not. Update to always
use arrow functions for anonymous callbacks and trailing commas for
multiline arrays.

PR-URL: https://github.com/nodejs/node/pull/25944
Reviewed-By: Anto Aravinth 
Reviewed-By: James M Snell

doc,lib,test: capitalize comment sentences

2018-12-17T16:14:35+00:00

This activates the eslint capitalize comment rule for comments
above 50 characters.

PR-URL: https://github.com/nodejs/node/pull/24996
Reviewed-By: Ujjwal Sharma 
Reviewed-By: Anna Henningsen 
Reviewed-By: Sakthipriyan Vairamani 
Reviewed-By: James M Snell

tls: move convertNPNProtocols to End-of-Life

2018-05-17T16:53:44+00:00

This was deprecated in 10.0.0 because NPN support was removed.
It does not make sense to keep this around longer than 10.x

PR-URL: https://github.com/nodejs/node/pull/20736
Reviewed-By: Rich Trott 
Reviewed-By: Colin Ihrig 
Reviewed-By: Daniel Bevenius 
Reviewed-By: Yuta Hiroto 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Tobias Nießen 
Reviewed-By: Michaël Zasso 
Reviewed-By: Matteo Collina 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Ali Ijaz Sheikh

benchmark: add tls benchmark for legacy SecurePair

2018-05-16T04:54:36+00:00

PR-URL: https://github.com/nodejs/node/pull/20344
Refs: https://github.com/nodejs/node/issues/20263
Reviewed-By: James M Snell 
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: Tobias Nießen

tools: enable eslint one-var rule

2018-02-22T14:56:15+00:00

PR-URL: https://github.com/nodejs/node/pull/18831
Reviewed-By: Luigi Pinca 
Reviewed-By: Matheus Marchini 
Reviewed-By: Anna Henningsen 
Reviewed-By: Michaël Zasso 
Reviewed-By: Anatoli Papirovski 
Reviewed-By: Colin Ihrig

test: add test for tls benchmarks

2018-02-04T13:14:44+00:00

PR-URL: https://github.com/nodejs/node/pull/18489
Reviewed-By: Ruben Bridgewater 
Reviewed-By: James M Snell