]> granicus.if.org Git - python/commitdiff
bpo-30108: Fix test_site setUpModule() (#1460)
authorVictor Stinner <victor.stinner@gmail.com>
Thu, 4 May 2017 16:52:26 +0000 (18:52 +0200)
committerGitHub <noreply@github.com>
Thu, 4 May 2017 16:52:26 +0000 (18:52 +0200)
Oops, I forgot that PermissionError was introduced in Python 3.3!
Replace PermissionError with OSError and check on errno.

Lib/test/test_site.py

index 3ba5dca82269c6c12798aacf308b1e977231a209..9569135c2ca2f654a1d1973511a84bacce32936b 100644 (file)
@@ -8,6 +8,7 @@ import unittest
 from test.test_support import run_unittest, TESTFN, EnvironmentVarGuard
 from test.test_support import captured_output
 import __builtin__
+import errno
 import os
 import sys
 import re
@@ -38,9 +39,12 @@ def setUpModule():
             os.makedirs(site.USER_SITE)
             # modify sys.path: will be restored by tearDownModule()
             site.addsitedir(site.USER_SITE)
-        except PermissionError as exc:
-            raise unittest.SkipTest('unable to create user site directory (%r): %s'
-                                    % (site.USER_SITE, exc))
+        except OSError as exc:
+            if exc.errno in (errno.EACCES, errno.EPERM):
+                raise unittest.SkipTest('unable to create user site directory (%r): %s'
+                                        % (site.USER_SITE, exc))
+            else:
+                raise
 
 
 def tearDownModule():