Commit fe9b6ee
test: move require('https') to after crypto check
Currently, test-https-client-override-global-agent.js fails with the
following error when configured --without-ssl:
Error [ERR_NO_CRYPTO]:
Node.js is not compiled with OpenSSL crypto support
at Object.assertCrypto (internal/util.js:101:11)
...
at Object.<anonymous>
(/node/test/parallel/test-https-client-override-global-agent.js:5:15)
This commit moves the require statement to after the crypto check.
PR-URL: #25388
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent dcbf1d9 commit fe9b6ee
1 file changed
Lines changed: 2 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1,12 +1,11 @@ | |||
| 1 | 1 | 'use strict'; | |
| 2 | 2 | const common = require('../common'); | |
| 3 | + if (!common.hasCrypto) | ||
| 4 | + common.skip('missing crypto'); | ||
| 3 | 5 | const fixtures = require('../common/fixtures'); | |
| 4 | 6 | const assert = require('assert'); | |
| 5 | 7 | const https = require('https'); | |
| 6 | 8 | ||
| 7 | - if (!common.hasCrypto) | ||
| 8 | - common.skip('missing crypto'); | ||
| 9 | - | ||
| 10 | 9 | // Disable strict server certificate validation by the client | |
| 11 | 10 | process.env.NODE_TLS_REJECT_UNAUTHORIZED = '0'; | |
| 12 | 11 | ||
0 commit comments