summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHai Shi <shihai1992@gmail.com>2020-08-08 11:05:24 (GMT)
committerGitHub <noreply@github.com>2020-08-08 11:05:24 (GMT)
commitc6f282f3b1cb6da6febc3b8b6d2dc1ef714dbbf7 (patch)
treec5f4f5c88c8a12e9daa8ca2786913c9beda7b4cb
parentd94af3f7ed98e6bfb4bf5f918f464b5e23d3ed1b (diff)
downloadcpython-c6f282f3b1cb6da6febc3b8b6d2dc1ef714dbbf7.zip
cpython-c6f282f3b1cb6da6febc3b8b6d2dc1ef714dbbf7.tar.gz
cpython-c6f282f3b1cb6da6febc3b8b6d2dc1ef714dbbf7.tar.bz2
bpo-40275: Use new test.support helper submodules in tests (GH-21785)
-rw-r--r--Lib/test/libregrtest/save_env.py6
-rw-r--r--Lib/test/test__osx_support.py17
-rw-r--r--Lib/test/test_dbm_ndbm.py5
-rw-r--r--Lib/test/test_zipfile64.py5
4 files changed, 17 insertions, 16 deletions
diff --git a/Lib/test/libregrtest/save_env.py b/Lib/test/libregrtest/save_env.py
index 50ed353..4c9c692 100644
--- a/Lib/test/libregrtest/save_env.py
+++ b/Lib/test/libregrtest/save_env.py
@@ -77,7 +77,7 @@ class saved_test_environment:
return list(urllib.request._url_tempfiles)
def restore_urllib_requests__url_tempfiles(self, tempfiles):
for filename in tempfiles:
- support.unlink(filename)
+ os_helper.unlink(filename)
def get_urllib_requests__opener(self):
return urllib.request._opener
@@ -245,9 +245,9 @@ class saved_test_environment:
fn = os_helper.TESTFN
if fn not in saved_value and (fn + '/') not in saved_value:
if os.path.isfile(fn):
- support.unlink(fn)
+ os_helper.unlink(fn)
elif os.path.isdir(fn):
- support.rmtree(fn)
+ os_helper.rmtree(fn)
_lc = [getattr(locale, lc) for lc in dir(locale)
if lc.startswith('LC_')]
diff --git a/Lib/test/test__osx_support.py b/Lib/test/test__osx_support.py
index 1a5d649..a3f41d2 100644
--- a/Lib/test/test__osx_support.py
+++ b/Lib/test/test__osx_support.py
@@ -9,6 +9,7 @@ import sys
import unittest
import test.support
+from test.support import os_helper
import _osx_support
@@ -39,9 +40,9 @@ class Test_OSXSupport(unittest.TestCase):
if self.env['PATH']:
self.env['PATH'] = self.env['PATH'] + ':'
self.env['PATH'] = self.env['PATH'] + os.path.abspath(self.temp_path_dir)
- test.support.unlink(self.prog_name)
+ os_helper.unlink(self.prog_name)
self.assertIsNone(_osx_support._find_executable(self.prog_name))
- self.addCleanup(test.support.unlink, self.prog_name)
+ self.addCleanup(os_helper.unlink, self.prog_name)
with open(self.prog_name, 'w') as f:
f.write("#!/bin/sh\n/bin/echo OK\n")
os.chmod(self.prog_name, stat.S_IRWXU)
@@ -52,8 +53,8 @@ class Test_OSXSupport(unittest.TestCase):
if self.env['PATH']:
self.env['PATH'] = self.env['PATH'] + ':'
self.env['PATH'] = self.env['PATH'] + os.path.abspath(self.temp_path_dir)
- test.support.unlink(self.prog_name)
- self.addCleanup(test.support.unlink, self.prog_name)
+ os_helper.unlink(self.prog_name)
+ self.addCleanup(os_helper.unlink, self.prog_name)
with open(self.prog_name, 'w') as f:
f.write("#!/bin/sh\n/bin/echo ExpectedOutput\n")
os.chmod(self.prog_name, stat.S_IRWXU)
@@ -143,8 +144,8 @@ class Test_OSXSupport(unittest.TestCase):
suffix = (':' + self.env['PATH']) if self.env['PATH'] else ''
self.env['PATH'] = os.path.abspath(self.temp_path_dir) + suffix
for c_name, c_output in compilers:
- test.support.unlink(c_name)
- self.addCleanup(test.support.unlink, c_name)
+ os_helper.unlink(c_name)
+ self.addCleanup(os_helper.unlink, c_name)
with open(c_name, 'w') as f:
f.write("#!/bin/sh\n/bin/echo " + c_output)
os.chmod(c_name, stat.S_IRWXU)
@@ -221,8 +222,8 @@ class Test_OSXSupport(unittest.TestCase):
suffix = (':' + self.env['PATH']) if self.env['PATH'] else ''
self.env['PATH'] = os.path.abspath(self.temp_path_dir) + suffix
c_name = 'clang'
- test.support.unlink(c_name)
- self.addCleanup(test.support.unlink, c_name)
+ os_helper.unlink(c_name)
+ self.addCleanup(os_helper.unlink, c_name)
# exit status 255 means no PPC support in this compiler chain
with open(c_name, 'w') as f:
f.write("#!/bin/sh\nexit 255")
diff --git a/Lib/test/test_dbm_ndbm.py b/Lib/test/test_dbm_ndbm.py
index e17a1d9..639c833 100644
--- a/Lib/test/test_dbm_ndbm.py
+++ b/Lib/test/test_dbm_ndbm.py
@@ -1,4 +1,3 @@
-from test import support
from test.support import import_helper
from test.support import os_helper
import_helper.import_module("dbm.ndbm") #skip if not supported
@@ -16,7 +15,7 @@ class DbmTestCase(unittest.TestCase):
def tearDown(self):
for suffix in ['', '.pag', '.dir', '.db']:
- support.unlink(self.filename + suffix)
+ os_helper.unlink(self.filename + suffix)
def test_keys(self):
self.d = dbm.ndbm.open(self.filename, 'c')
@@ -108,7 +107,7 @@ class DbmTestCase(unittest.TestCase):
def test_nonascii_filename(self):
filename = os_helper.TESTFN_NONASCII
for suffix in ['', '.pag', '.dir', '.db']:
- self.addCleanup(support.unlink, filename + suffix)
+ self.addCleanup(os_helper.unlink, filename + suffix)
with dbm.ndbm.open(filename, 'c') as db:
db[b'key'] = b'value'
self.assertTrue(any(os.path.exists(filename + suffix)
diff --git a/Lib/test/test_zipfile64.py b/Lib/test/test_zipfile64.py
index 3a788de..810fded 100644
--- a/Lib/test/test_zipfile64.py
+++ b/Lib/test/test_zipfile64.py
@@ -17,6 +17,7 @@ import sys
from tempfile import TemporaryFile
+from test.support import os_helper
from test.support import TESTFN, requires_zlib
TESTFN2 = TESTFN + "2"
@@ -138,8 +139,8 @@ class OtherTests(unittest.TestCase):
self.assertEqual(content, "%d" % (i**3 % 57))
def tearDown(self):
- support.unlink(TESTFN)
- support.unlink(TESTFN2)
+ os_helper.unlink(TESTFN)
+ os_helper.unlink(TESTFN2)
if __name__ == "__main__":
unittest.main()