From 2bb93e11085ea582aea1636f544d900520d479ed Mon Sep 17 00:00:00 2001 From: Ben Noordhuis Date: Thu, 6 Jun 2019 12:59:30 +0200 Subject: [PATCH] test: set LC_ALL to known good value Set the locale to a known good value because it affects ICU's date string formatting. Setting LC_ALL needs to happen before the first call to `icu::Locale::getDefault()` because ICU caches the result. Fixes: https://github.com/nodejs/node/issues/27856 PR-URL: https://github.com/nodejs/node/pull/28096 Reviewed-By: Colin Ihrig Reviewed-By: Daniel Bevenius Reviewed-By: James M Snell Reviewed-By: Rich Trott Reviewed-By: Vladimir de Turckheim --- test/parallel/test-process-env-tz.js | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/test/parallel/test-process-env-tz.js b/test/parallel/test-process-env-tz.js index 25152a9b6c6ce1..da716299c9f622 100644 --- a/test/parallel/test-process-env-tz.js +++ b/test/parallel/test-process-env-tz.js @@ -1,5 +1,10 @@ 'use strict'; +// Set the locale to a known good value because it affects ICU's date string +// formatting. Setting LC_ALL needs to happen before the first call to +// `icu::Locale::getDefault()` because ICU caches the result. +process.env.LC_ALL = 'C'; + const common = require('../common'); const assert = require('assert');