Merge ~ines-almeida/launchpad:force-tests-container-locale into launchpad:master

[Ines Almeida]

Force test container locale

This prevents local setups from influencing test results regarding locale setup.

Added a quick locale setup in the environment configuration in `lp/scripts/utilities/test`

Tested it locally in tests that were failing, and it "fixed" the issue

[Ines Almeida (ines-almeida)]

FYI, had a look into using `os.environ` but for that we would need to change all locale variables, this seemed much cleaner.

[Ines Almeida (ines-almeida)]

Is this tiny change worth a test to check the locale, or is all tests passing enough testing?

[Colin Watson (cjwatson)]

LGTM, thanks. I don't think we need a separate test for this.

review: Approve

diff --git a/lib/lp/scripts/utilities/test.py b/lib/lp/scripts/utilities/test.py
index d997a8a..93d16cf 100755
--- a/lib/lp/scripts/utilities/test.py
+++ b/lib/lp/scripts/utilities/test.py
@@ -15,6 +15,7 @@
 
 import argparse
 import doctest
+import locale
 import os
 import random
 import re
@@ -52,6 +53,9 @@ def configure_environment():
     os.environ["TZ"] = "Asia/Calcutta"
     time.tzset()
 
+    # Forces tests to run with a sensible locale encoding
+    locale.setlocale(locale.LC_ALL, "C.UTF-8")
+
     # Httplib2 0.7 started validating SSL certificates, and the test suite
     # uses a self-signed certificate, so disable it with an env variable.
     os.environ["LP_DISABLE_SSL_CERTIFICATE_VALIDATION"] = "1"