]> granicus.if.org Git - python/commitdiff
#6022 fixed test_get_outputs so it doesn't leaves a test file in the cwd
authorTarek Ziadé <ziade.tarek@gmail.com>
Thu, 14 May 2009 20:14:13 +0000 (20:14 +0000)
committerTarek Ziadé <ziade.tarek@gmail.com>
Thu, 14 May 2009 20:14:13 +0000 (20:14 +0000)
Lib/distutils/tests/test_build_ext.py
Misc/NEWS

index 45c5b39e0f5fb24486082a8c3b83616551db6231..c6533ca9f79128d4355e5c03a743a3e17e650fcf 100644 (file)
@@ -312,12 +312,18 @@ class BuildExtTestCase(support.TempdirManager,
 
         # issue #5977 : distutils build_ext.get_outputs
         # returns wrong result with --inplace
-        cmd.inplace = 1
-        cmd.run()
-        so_file = cmd.get_outputs()[0]
+        other_tmp_dir = os.path.realpath(self.mkdtemp())
+        old_wd = os.getcwd()
+        os.chdir(other_tmp_dir)
+        try:
+            cmd.inplace = 1
+            cmd.run()
+            so_file = cmd.get_outputs()[0]
+        finally:
+            os.chdir(old_wd)
         self.assert_(os.path.exists(so_file))
         so_dir = os.path.dirname(so_file)
-        self.assertEquals(so_dir, os.getcwd())
+        self.assertEquals(so_dir, other_tmp_dir)
 
         cmd.inplace = 0
         cmd.run()
index 2203621b479785b15dcbba79d394bc7ed9161f94..5d7a81ea5c62121566d49265e624d70db8df11df 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -293,6 +293,9 @@ Core and Builtins
 Library
 -------
 
+- Issue #6022: a test file was created in the current working directory by
+  test_get_outputs in Distutils.
+
 - Issue #4050: inspect.findsource/getsource now raise an IOError if the 'source'
   file is a binary.  Patch by Brodie Rao, tests by Daniel Diniz.