Skip to content

Commit 1d79af6

Browse files
oliversalzburgMylesBorins
authored andcommitted
test: case sensitivity of env variables
Environment variables should be treated case-insensitive on Windows platforms and case-sensitive on UNIX platforms. This commit ensures this behavior persists. Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> PR-URL: #9166 Fixes: #9157
1 parent 18a235b commit 1d79af6

File tree

1 file changed

+15
-1
lines changed

1 file changed

+15
-1
lines changed

test/parallel/test-process-env.js

+15-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
'use strict';
22

3-
require('../common');
3+
const common = require('../common');
44
const assert = require('assert');
55

66
// changes in environment should be visible to child processes
@@ -67,3 +67,17 @@ assert.equal(3, date.getUTCHours());
6767
assert.equal(5, date.getHours());
6868
*/
6969
/* eslint-enable max-len */
70+
71+
// Environment variables should be case-insensitive on Windows, and
72+
// case-sensitive on other platforms.
73+
process.env.TEST = 'test';
74+
assert.strictEqual(process.env.TEST, 'test');
75+
// Check both mixed case and lower case, to avoid any regressions that might
76+
// simply convert input to lower case.
77+
if (common.isWindows) {
78+
assert.strictEqual(process.env.test, 'test');
79+
assert.strictEqual(process.env.teST, 'test');
80+
} else {
81+
assert.strictEqual(process.env.test, undefined);
82+
assert.strictEqual(process.env.teST, undefined);
83+
}

0 commit comments

Comments
 (0)