Merged revisions 84909-84913 via svnmerge from

svn+ssh://pythondev@svn.python.org/python/branches/py3k

........
  r84909 | antoine.pitrou | 2010-09-20 00:46:05 +0200 (lun., 20 sept. 2010) | 3 lines

  Try to fix test_subprocess on "x86 debian parallel 3.x" buildbot
........
  r84910 | antoine.pitrou | 2010-09-20 01:06:53 +0200 (lun., 20 sept. 2010) | 3 lines

  Try to make signal-sending tests in test_subprocess more robust on slow machines
........
  r84911 | antoine.pitrou | 2010-09-20 01:28:30 +0200 (lun., 20 sept. 2010) | 3 lines

  Make error more explicit in test_finalize_with_trace
........
  r84912 | antoine.pitrou | 2010-09-20 02:12:19 +0200 (lun., 20 sept. 2010) | 3 lines

  Try to fix buildbot failure (#9902)
........
  r84913 | antoine.pitrou | 2010-09-20 03:33:21 +0200 (lun., 20 sept. 2010) | 3 lines

  Try a more robust implementation of _kill_process
........

diff --git a/Lib/test/test_subprocess.py b/Lib/test/test_subprocess.py
index 552b9f220d27fa373a930dd57f5246ba1ac4256d..f2dbdc23fb158c44aeb93865f969dedc5dc2ca8d 100644 (file)
--- a/Lib/test/test_subprocess.py
+++ b/Lib/test/test_subprocess.py
@@ -765,12 +765,16 @@ class ProcessTestCase(BaseTestCase):
 
         def test_undecodable_env(self):
             for key, value in (('test', 'abc\uDCFF'), ('test\uDCFF', '42')):
-                value_repr = repr(value).encode("ascii")
+                value_repr = ascii(value).encode("ascii")
 
                 # test str with surrogates
-                script = "import os; print(repr(os.getenv(%s)))" % repr(key)
+                script = "import os; print(ascii(os.getenv(%s)))" % repr(key)
                 env = os.environ.copy()
                 env[key] = value
+                # Force surrogate-escaping of \xFF in the child process;
+                # otherwise it can be decoded as-is if the default locale
+                # is latin-1.
+                env['PYTHONFSENCODING'] = 'ascii'
                 stdout = subprocess.check_output(
                     [sys.executable, "-c", script],
                     env=env)
@@ -780,7 +784,7 @@ class ProcessTestCase(BaseTestCase):
                 # test bytes
                 key = key.encode("ascii", "surrogateescape")
                 value = value.encode("ascii", "surrogateescape")
-                script = "import os; print(repr(os.getenv(%s)))" % repr(key)
+                script = "import os; print(ascii(os.getenv(%s)))" % repr(key)
                 env = os.environ.copy()
                 env[key] = value
                 stdout = subprocess.check_output(

diff --git a/Lib/test/test_threading.py b/Lib/test/test_threading.py
index 71135cc2e040fa029d36638b3a4b99eff85f8fa6..713ea9cd336652fd18196933907ee6907536b8d8 100644 (file)
--- a/Lib/test/test_threading.py
+++ b/Lib/test/test_threading.py
@@ -279,7 +279,7 @@ class ThreadTests(unittest.TestCase):
         # Issue1733757
         # Avoid a deadlock when sys.settrace steps into threading._shutdown
         import subprocess
-        rc = subprocess.call([sys.executable, "-c", """if 1:
+        p = subprocess.Popen([sys.executable, "-c", """if 1:
             import sys, threading
 
             # A deadlock-killer, to prevent the
@@ -299,9 +299,14 @@ class ThreadTests(unittest.TestCase):
                 return func
 
             sys.settrace(func)
-            """])
+            """],
+            stdout=subprocess.PIPE,
+            stderr=subprocess.PIPE)
+        stdout, stderr = p.communicate()
+        rc = p.returncode
         self.assertFalse(rc == 2, "interpreted was blocked")
-        self.assertTrue(rc == 0, "Unexpected error")
+        self.assertTrue(rc == 0,
+                        "Unexpected error: " + ascii(stderr))
 
     def test_join_nondaemon_on_shutdown(self):
         # Issue 1722344