test: move require of https to after crypto check

danbev · targos · commit 3baeb82f4375 · 2018-08-11T18:10:10.000+02:00
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: #22148 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com>
diff --git 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'),

-Original file line number
+Diff line change
@@ @@ -1,11 +1,11 @@ @@
 'use strict';
 constcommon=require('../common');
 constassert=require('assert');
 -consthttps=require('https');
 constfixtures=require('../common/fixtures');
 if(!common.hasCrypto)
 common.skip('missing crypto');
 +consthttps=require('https');
 constoptions={
 key: fixtures.readKey('agent1-key.pem'),