(original) (raw)

changeset: 84154:bc18b5d4920e user: Brett Cannon brett@python.org date: Sat Jun 15 17:11:25 2013 -0400 files: Lib/test/script_helper.py Lib/test/support.py Lib/test/test_pdb.py Lib/test/test_pkgimport.py Lib/test/test_pkgutil.py Lib/test/test_reprlib.py Lib/test/test_zipimport.py description: Issue #17177: Stop using imp in a bunch of tests diff -r fcb686e720ff -r bc18b5d4920e Lib/test/script_helper.py --- a/Lib/test/script_helper.py Sat Jun 15 15:10:08 2013 -0400 +++ b/Lib/test/script_helper.py Sat Jun 15 17:11:25 2013 -0400 @@ -12,7 +12,7 @@ import shutil import zipfile -from imp import source_from_cache +from importlib.util import source_from_cache from test.support import make_legacy_pyc, strip_python_stderr # Executing the interpreter in a subprocess diff -r fcb686e720ff -r bc18b5d4920e Lib/test/support.py --- a/Lib/test/support.py Sat Jun 15 15:10:08 2013 -0400 +++ b/Lib/test/support.py Sat Jun 15 17:11:25 2013 -0400 @@ -15,10 +15,10 @@ import warnings import unittest import importlib +import importlib.util import collections.abc import re import subprocess -import imp import time import sysconfig import fnmatch @@ -316,7 +316,7 @@ does not need to exist, however the PEP 3147 pyc file must exist. :return: The file system path to the legacy pyc file. """ - pyc_file = imp.cache_from_source(source) + pyc_file = importlib.util.cache_from_source(source) up_one = os.path.dirname(os.path.abspath(source)) legacy_pyc = os.path.join(up_one, source + ('c' if __debug__ else 'o')) os.rename(pyc_file, legacy_pyc) @@ -335,8 +335,8 @@ # combinations of PEP 3147 and legacy pyc and pyo files. unlink(source + 'c') unlink(source + 'o') - unlink(imp.cache_from_source(source, debug_override=True)) - unlink(imp.cache_from_source(source, debug_override=False)) + unlink(importlib.util.cache_from_source(source, debug_override=True)) + unlink(importlib.util.cache_from_source(source, debug_override=False)) # On some platforms, should not run gui test even if it is allowed # in `use_resources'. diff -r fcb686e720ff -r bc18b5d4920e Lib/test/test_pdb.py --- a/Lib/test/test_pdb.py Sat Jun 15 15:10:08 2013 -0400 +++ b/Lib/test/test_pdb.py Sat Jun 15 17:11:25 2013 -0400 @@ -1,9 +1,9 @@ # A test suite for pdb; not very comprehensive at the moment. import doctest -import imp import pdb import sys +import types import unittest import subprocess import textwrap @@ -464,7 +464,7 @@ # Module for testing skipping of module that makes a callback -mod = imp.new_module('module_to_skip') +mod = types.ModuleType('module_to_skip') exec('def foo_pony(callback): x = 1; callback(); return None', mod.__dict__) diff -r fcb686e720ff -r bc18b5d4920e Lib/test/test_pkgimport.py --- a/Lib/test/test_pkgimport.py Sat Jun 15 15:10:08 2013 -0400 +++ b/Lib/test/test_pkgimport.py Sat Jun 15 17:11:25 2013 -0400 @@ -6,7 +6,7 @@ import tempfile import unittest -from imp import cache_from_source +from importlib.util import cache_from_source from test.support import run_unittest, create_empty_file class TestImport(unittest.TestCase): diff -r fcb686e720ff -r bc18b5d4920e Lib/test/test_pkgutil.py --- a/Lib/test/test_pkgutil.py Sat Jun 15 15:10:08 2013 -0400 +++ b/Lib/test/test_pkgutil.py Sat Jun 15 17:11:25 2013 -0400 @@ -1,12 +1,12 @@ from test.support import run_unittest, unload, check_warnings import unittest import sys -import imp import importlib import pkgutil import os import os.path import tempfile +import types import shutil import zipfile @@ -105,7 +105,7 @@ class MyTestLoader(object): def load_module(self, fullname): # Create an empty module - mod = sys.modules.setdefault(fullname, imp.new_module(fullname)) + mod = sys.modules.setdefault(fullname, types.ModuleType(fullname)) mod.__file__ = "<%s>" % self.__class__.__name__ mod.__loader__ = self # Make it a package diff -r fcb686e720ff -r bc18b5d4920e Lib/test/test_reprlib.py --- a/Lib/test/test_reprlib.py Sat Jun 15 15:10:08 2013 -0400 +++ b/Lib/test/test_reprlib.py Sat Jun 15 17:11:25 2013 -0400 @@ -3,11 +3,11 @@ Nick Mathewson """ -import imp import sys import os import shutil import importlib +import importlib.util import unittest from test.support import run_unittest, create_empty_file, verbose @@ -241,7 +241,8 @@ source_path_len += 2 * (len(self.longname) + 1) # a path separator + `module_name` + ".py" source_path_len += len(module_name) + 1 + len(".py") - cached_path_len = source_path_len + len(imp.cache_from_source("x.py")) - len("x.py") + cached_path_len = (source_path_len + + len(importlib.util.cache_from_source("x.py")) - len("x.py")) if os.name == 'nt' and cached_path_len >= 258: # Under Windows, the max path len is 260 including C's terminating # NUL character. diff -r fcb686e720ff -r bc18b5d4920e Lib/test/test_zipimport.py --- a/Lib/test/test_zipimport.py Sat Jun 15 15:10:08 2013 -0400 +++ b/Lib/test/test_zipimport.py Sat Jun 15 17:11:25 2013 -0400 @@ -1,7 +1,7 @@ import sys import os import marshal -import imp +import importlib.util import struct import time import unittest @@ -34,7 +34,8 @@ mtime = int(mtime) else: mtime = int(-0x100000000 + int(mtime)) - pyc = imp.get_magic() + struct.pack("/brett@python.org