From 5473a5cb4a2ac35878fd7766eb519f8ede245e4e Mon Sep 17 00:00:00 2001 From: Daniel Bevenius Date: Mon, 12 Feb 2018 07:28:41 +0100 Subject: [PATCH] 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. I'll take a closer look at the check and see what can be done but I don't have time straight away, so creating this commit to avoid breakage in the short term. --- test/sequential/test-benchmark-tls.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/test/sequential/test-benchmark-tls.js b/test/sequential/test-benchmark-tls.js index 7c87aa3cbcd89e..3545955e3ab5b0 100644 --- a/test/sequential/test-benchmark-tls.js +++ b/test/sequential/test-benchmark-tls.js @@ -2,6 +2,9 @@ const common = require('../common'); +if (!common.hasCrypto) + common.skip('missing crypto'); + if (!common.enoughTestMem) common.skip('Insufficient memory for TLS benchmark test');