]> granicus.if.org Git - python/commitdiff
Remove AIX workaround test_subprocess (GH-8939)
authorMichael Felt <aixtools@users.noreply.github.com>
Sun, 26 Aug 2018 17:29:36 +0000 (19:29 +0200)
committerGregory P. Smith <greg@krypto.org>
Sun, 26 Aug 2018 17:29:36 +0000 (10:29 -0700)
An old apparent AIX behavior workaround in test_subprocess's
test_undecodable_env is no longer needed.

Lib/test/test_subprocess.py
Misc/NEWS.d/next/Tests/2018-08-26-13-12-34.bpo-11193.H8fCGa.rst [new file with mode: 0644]

index 73b57b21db2cec9b2d17a2c218eb0f71ee3f9345..4719773b67b7d91aa107ed06ed027d128773b7fd 100644 (file)
@@ -2228,15 +2228,9 @@ class POSIXProcessTestCase(BaseTestCase):
             env = os.environ.copy()
             env[key] = value
             # Use C locale to get ASCII for the locale encoding to force
-            # surrogate-escaping of \xFF in the child process; otherwise it can
-            # be decoded as-is if the default locale is latin-1.
+            # surrogate-escaping of \xFF in the child process
             env['LC_ALL'] = 'C'
-            if sys.platform.startswith("aix"):
-                # On AIX, the C locale uses the Latin1 encoding
-                decoded_value = encoded_value.decode("latin1", "surrogateescape")
-            else:
-                # On other UNIXes, the C locale uses the ASCII encoding
-                decoded_value = value
+            decoded_value = value
             stdout = subprocess.check_output(
                 [sys.executable, "-c", script],
                 env=env)
diff --git a/Misc/NEWS.d/next/Tests/2018-08-26-13-12-34.bpo-11193.H8fCGa.rst b/Misc/NEWS.d/next/Tests/2018-08-26-13-12-34.bpo-11193.H8fCGa.rst
new file mode 100644 (file)
index 0000000..b31caa6
--- /dev/null
@@ -0,0 +1 @@
+Remove special condition for AIX in `test_subprocess.test_undecodable_env`