Skip to content
This repository was archived by the owner on May 28, 2025. It is now read-only.

Commit d597369

Browse files
committed
Auto merge of rust-lang#15175 - lnicola:cross-env, r=Veykril
internal: Drop `cross-env` npm dep
2 parents 40b2a68 + 477a75d commit d597369

File tree

3 files changed

+3
-21
lines changed

3 files changed

+3
-21
lines changed

editors/code/package-lock.json

Lines changed: 0 additions & 19 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

editors/code/package.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@
3838
"lint": "prettier --check . && eslint -c .eslintrc.js --ext ts ./src ./tests",
3939
"fix": "prettier --write . && eslint -c .eslintrc.js --ext ts ./src ./tests --fix",
4040
"pretest": "tsc && npm run build",
41-
"test": "cross-env TEST_VARIABLE=test node ./out/tests/runTests.js"
41+
"test": "node ./out/tests/runTests.js"
4242
},
4343
"dependencies": {
4444
"@hpcc-js/wasm": "^2.13.0",
@@ -55,7 +55,6 @@
5555
"@typescript-eslint/parser": "^5.60.1",
5656
"@vscode/test-electron": "^2.3.3",
5757
"@vscode/vsce": "^2.19.0",
58-
"cross-env": "^7.0.3",
5958
"esbuild": "^0.18.10",
6059
"eslint": "^8.43.0",
6160
"eslint-config-prettier": "^8.8.0",

editors/code/tests/unit/settings.test.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@ export async function getTests(ctx: Context) {
3939
});
4040

4141
suite.addTest("Should support external variables", async () => {
42+
process.env["TEST_VARIABLE"] = "test";
4243
const envJson = {
4344
USING_EXTERNAL_VAR: "${env:TEST_VARIABLE} test ${env:TEST_VARIABLE}",
4445
};
@@ -48,6 +49,7 @@ export async function getTests(ctx: Context) {
4849

4950
const actualEnv = await substituteVariablesInEnv(envJson);
5051
assert.deepStrictEqual(actualEnv, expectedEnv);
52+
delete process.env["TEST_VARIABLE"];
5153
});
5254

5355
suite.addTest("should support VSCode variables", async () => {

0 commit comments

Comments
 (0)