build: Set LC_ALL env var in setup file, not shell

lerna · Oct 12, 2018 · 349d43a · 349d43a
1 parent 433162b
commit 349d43a
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 13 deletions.
diff --git a/helpers/set-npm-userconfig/index.js b/helpers/set-npm-userconfig/index.js
@@ -3,3 +3,6 @@
 // Overwrite npm userconfig to avoid test pollution
 // https://docs.npmjs.com/misc/config#npmrc-files
 process.env.npm_config_userconfig = require("path").resolve(__dirname, "test-user.ini");
+
+// use consistent locale for all tests
+process.env.LC_ALL = "en_US.UTF-8";
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -6,10 +6,10 @@
   "scripts": {
     "ci": "npm test -- --ci --maxWorkers=2 && npm run integration -- --ci",
     "fix": "npm run lint -- --fix",
-    "integration": "cross-env LC_ALL=en_US.UTF-8 jest --config jest.integration.js --maxWorkers=2",
+    "integration": "jest --config jest.integration.js --maxWorkers=2",
     "lint": "eslint . --ignore-path .gitignore --cache",
     "pretest": "npm run lint",
-    "test": "cross-env LC_ALL=en_US.UTF-8 jest"
+    "test": "jest"
   },
   "repository": {
     "type": "git",
@@ -53,7 +53,6 @@
     "@lerna-test/silence-logging": "file:helpers/silence-logging",
     "@lerna-test/update-lerna-config": "file:helpers/update-lerna-config",
     "camelcase": "^4.1.0",
-    "cross-env": "^5.2.0",
     "eslint": "^5.6.1",
     "eslint-config-airbnb-base": "^13.1.0",
     "eslint-config-prettier": "^3.1.0",