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 <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>
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)