Skip to content

Commit d55bf6a

Browse files
KrayzeeKevFyko
authored andcommitted
test: ensure NODE_EXTRA_CA_CERTS not set before tests
The 'test-tls-env-extra-ca-file-load.js' test assumes that NODE_EXTRA_CA_CERTS is not set. If the build environment happens to have it set, this test will fail. This change deletes that env var before running the test. PR-URL: nodejs#43858 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Rich Trott <[email protected]>
1 parent 39666fd commit d55bf6a

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

test/parallel/test-tls-env-extra-ca-file-load.js

+2
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,8 @@ if (process.argv[2] !== 'child') {
2020
const NODE_EXTRA_CA_CERTS = fixtures.path('keys', 'ca1-cert.pem');
2121
const extendsEnv = (obj) => ({ ...process.env, ...obj });
2222

23+
// Remove any pre-existing extra CA certs.
24+
delete process.env.NODE_EXTRA_CA_CERTS;
2325
[
2426
extendsEnv({ CHILD_USE_EXTRA_CA_CERTS: 'yes', NODE_EXTRA_CA_CERTS }),
2527
extendsEnv({ CHILD_USE_EXTRA_CA_CERTS: 'no' }),

0 commit comments

Comments
 (0)