From b1cbbbc7af7303cc913f501d7296e80f34eca7a1 Mon Sep 17 00:00:00 2001 From: Daniel Bevenius Date: Mon, 6 Aug 2018 08:54:04 +0200 Subject: [PATCH] test: move require of https to after crypto check Currently when configured without-ssl test-request-arguments.js will fail with a missing crypto message. This commit moves the require of https to after the crypto check. PR-URL: https://github.com/nodejs/node/pull/22148 Reviewed-By: James M Snell Reviewed-By: Luigi Pinca Reviewed-By: Colin Ihrig Reviewed-By: Michael Dawson Reviewed-By: Richard Lau --- test/parallel/test-https-request-arguments.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/parallel/test-https-request-arguments.js b/test/parallel/test-https-request-arguments.js index 44037ddd6de90b..9dc80094be0d2c 100644 --- a/test/parallel/test-https-request-arguments.js +++ b/test/parallel/test-https-request-arguments.js @@ -1,11 +1,11 @@ 'use strict'; const common = require('../common'); const assert = require('assert'); -const https = require('https'); const fixtures = require('../common/fixtures'); if (!common.hasCrypto) common.skip('missing crypto'); +const https = require('https'); const options = { key: fixtures.readKey('agent1-key.pem'),