Discussion:
[Python-3000-checkins] r65164 - in python/branches/py3k: Lib/test/test_importhooks.py
georg.brandl
2008-07-20 23:21:58 UTC
Permalink
Author: georg.brandl
Date: Mon Jul 21 01:21:49 2008
New Revision: 65164

Log:
Merged revisions 65163 via svnmerge from
svn+ssh://pythondev at svn.python.org/python/trunk

........
r65163 | georg.brandl | 2008-07-21 01:18:55 +0200 (Mon, 21 Jul 2008) | 4 lines

Save the whole of sys.modules instead of using an import tracker.
This, when merged to py3k, will fix the spurious buildbot failure
in test_urllib2 ("<urlopen error unknown url type: do>").
........


Modified:
python/branches/py3k/ (props changed)
python/branches/py3k/Lib/test/test_importhooks.py

Modified: python/branches/py3k/Lib/test/test_importhooks.py
==============================================================================
--- python/branches/py3k/Lib/test/test_importhooks.py (original)
+++ python/branches/py3k/Lib/test/test_importhooks.py Mon Jul 21 01:21:49 2008
@@ -33,15 +33,6 @@
test_path = "!!!_test_!!!"


-class ImportTracker:
- """Importer that only tracks attempted imports."""
- def __init__(self):
- self.imports = []
- def find_module(self, fullname, path=None):
- self.imports.append(fullname)
- return None
-
-
class TestImporter:

modules = {
@@ -152,17 +143,15 @@
self.meta_path = sys.meta_path[:]
self.path_hooks = sys.path_hooks[:]
sys.path_importer_cache.clear()
- self.tracker = ImportTracker()
- sys.meta_path.insert(0, self.tracker)
+ self.modules_before = sys.modules.copy()

def tearDown(self):
sys.path[:] = self.path
sys.meta_path[:] = self.meta_path
sys.path_hooks[:] = self.path_hooks
sys.path_importer_cache.clear()
- for fullname in self.tracker.imports:
- if fullname in sys.modules:
- del sys.modules[fullname]
+ sys.modules.clear()
+ sys.modules.update(self.modules_before)


class ImportHooksTestCase(ImportHooksBaseTestCase):
@@ -256,13 +245,7 @@
for mname in mnames:
m = __import__(mname, globals(), locals(), ["__dummy__"])
m.__loader__ # to make sure we actually handled the import
-## # Delete urllib from modules because urlparse was imported above.
-## # Without this hack, test_socket_ssl fails if run in this order:
-## # regrtest.py test_codecmaps_tw test_importhooks test_socket_ssl
-## try:
-## del sys.modules['urllib']
-## except KeyError:
-## pass
+

def test_main():
support.run_unittest(ImportHooksTestCase)

Loading...