]> granicus.if.org Git - python/commitdiff
Issue #17169: Restore errno in tempfile exceptions.
authorSerhiy Storchaka <storchaka@gmail.com>
Sat, 9 Feb 2013 20:25:49 +0000 (22:25 +0200)
committerSerhiy Storchaka <storchaka@gmail.com>
Sat, 9 Feb 2013 20:25:49 +0000 (22:25 +0200)
Lib/tempfile.py
Lib/test/test_tempfile.py

index 61b0df17a61b1754be895c9b48061eedd5fed656..0cc5c04405a8302c79d6fe4264e9c0e1980d8993 100644 (file)
@@ -31,6 +31,7 @@ import warnings as _warnings
 import sys as _sys
 import io as _io
 import os as _os
+import errno as _errno
 from random import Random as _Random
 
 try:
@@ -181,7 +182,9 @@ def _get_default_tempdir():
                 pass
             except OSError:
                 break   # no point trying more names in this directory
-    raise FileNotFoundError("No usable temporary directory found in %s" % dirlist)
+    raise FileNotFoundError(_errno.ENOENT,
+                            "No usable temporary directory found in %s" %
+                            dirlist)
 
 _name_sequence = None
 
@@ -214,7 +217,8 @@ def _mkstemp_inner(dir, pre, suf, flags):
         except FileExistsError:
             continue    # try again
 
-    raise FileExistsError("No usable temporary file name found")
+    raise FileExistsError(_errno.EEXIST,
+                          "No usable temporary file name found")
 
 
 # User visible interfaces.
@@ -301,7 +305,8 @@ def mkdtemp(suffix="", prefix=template, dir=None):
         except FileExistsError:
             continue    # try again
 
-    raise FileExistsError("No usable temporary directory name found")
+    raise FileExistsError(_errno.EEXIST,
+                          "No usable temporary directory name found")
 
 def mktemp(suffix="", prefix=template, dir=None):
     """User-callable function to return a unique temporary file name.  The
@@ -330,7 +335,8 @@ def mktemp(suffix="", prefix=template, dir=None):
         if not _exists(file):
             return file
 
-    raise FileExistsError("No usable temporary filename found")
+    raise FileExistsError(_errno.EEXIST,
+                          "No usable temporary filename found")
 
 
 class _TemporaryFileWrapper:
index 40e5e6f94b91e76244ae373d0ae06c1374664ca8..7f4d5b9ec786e1f688d6541d0e121ef4b61aa7b7 100644 (file)
@@ -1,5 +1,6 @@
 # tempfile.py unit tests.
 import tempfile
+import errno
 import os
 import signal
 import sys
@@ -963,8 +964,9 @@ class TestTemporaryDirectory(BaseTestCase):
         # (noted as part of Issue #10188)
         with tempfile.TemporaryDirectory() as nonexistent:
             pass
-        with self.assertRaises(os.error):
+        with self.assertRaises(FileNotFoundError) as cm:
             tempfile.TemporaryDirectory(dir=nonexistent)
+        self.assertEqual(cm.exception.errno, errno.ENOENT)
 
     def test_explicit_cleanup(self):
         # A TemporaryDirectory is deleted when cleaned up