Skip to content

Commit 5b88cb7

Browse files
danbevMylesBorins
authored andcommitted
test: add crypto check to test-benchmark-tls
Currently when building --without-ssl a 'ERR_NO_CRYPTO' error is reported. This is not currently being picked up by the crypto-check lint rule as it does not actually require any crypto modules directly, but instead this is done by common/benchmark. PR-URL: #18724 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent 5331454 commit 5b88cb7

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

test/sequential/test-benchmark-tls.js

+3
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,9 @@
22

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

5+
if (!common.hasCrypto)
6+
common.skip('missing crypto');
7+
58
if (!common.enoughTestMem)
69
common.skip('Insufficient memory for TLS benchmark test');
710

0 commit comments

Comments
 (0)