fixup! Some platforms need the full env

nodejs · Apr 25, 2019 · d93152d · d93152d
1 parent c369ef0
commit d93152d
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/test/parallel/test-intl.js b/test/parallel/test-intl.js
@@ -127,10 +127,13 @@ if (!common.hasIntl) {
   // `ffi` ligature (contraction)
   assert.strictEqual(coll.compare('\ufb03', 'ffi'), 0);
 
-  // Regression test for https://github.com/nodejs/node/issues/27379
-  execFile(
-    process.execPath, ['-p', 'new Date().toLocaleString()'],
-    { env: { LC_ALL: 'ja' }, encoding: 'utf-8' },
-    common.mustCall((e) => assert.ifError(e))
-  );
+  {
+    // Regression test for https://github.com/nodejs/node/issues/27379
+    const env = { ...process.env, LC_ALL: 'ja' };
+    execFile(
+      process.execPath, ['-p', 'new Date().toLocaleString()'],
+      { env },
+      common.mustCall((e) => assert.ifError(e))
+    );
+  }
 }