pythongh-117089: Move importlib.metadata tests to their own package (p…

…ython#117092) * Ensure importlib.metadata tests do not leak references in sys.modules. * Move importlib.metadata tests to their own package for easier syncing with importlib_metadata. * Update owners and makefile for new directories. * Add blurb
diegorusso · Apr 17, 2024 · 14b08e3 · 14b08e3
1 parent b70389c
commit 14b08e3
Show file tree

Hide file tree

Showing 15 changed files with 7 additions and 3 deletions.
diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS
@@ -119,7 +119,7 @@ Python/dynload_*.c            @ericsnowcurrently
 Lib/test/test_module/         @ericsnowcurrently
 Doc/c-api/module.rst          @ericsnowcurrently
 **/*importlib/resources/*     @jaraco @warsaw @FFY00
-**/importlib/metadata/*       @jaraco @warsaw
+**/*importlib/metadata/*       @jaraco @warsaw
 
 # Dates and times
 **/*datetime*                 @pganssle @abalkin

diff --git a/Lib/test/test_importlib/data/__init__.py → Lib/test/test_importlib/metadata/__init__.py b/Lib/test/test_importlib/data/__init__.py → Lib/test/test_importlib/metadata/__init__.py
diff --git a/Lib/test/test_importlib/_context.py → Lib/test/test_importlib/metadata/_context.py b/Lib/test/test_importlib/_context.py → Lib/test/test_importlib/metadata/_context.py
diff --git a/Lib/test/test_importlib/_path.py → Lib/test/test_importlib/metadata/_path.py b/Lib/test/test_importlib/_path.py → Lib/test/test_importlib/metadata/_path.py
diff --git a/Lib/test/test_importlib/metadata/data/__init__.py b/Lib/test/test_importlib/metadata/data/__init__.py
diff --git a/...rtlib/data/example-21.12-py3-none-any.whl → ...adata/data/example-21.12-py3-none-any.whl b/...rtlib/data/example-21.12-py3-none-any.whl → ...adata/data/example-21.12-py3-none-any.whl
diff --git a/...st_importlib/data/example-21.12-py3.6.egg → ...lib/metadata/data/example-21.12-py3.6.egg b/...st_importlib/data/example-21.12-py3.6.egg → ...lib/metadata/data/example-21.12-py3.6.egg
diff --git a/...tlib/data/example2-1.0.0-py3-none-any.whl → ...data/data/example2-1.0.0-py3-none-any.whl b/...tlib/data/example2-1.0.0-py3-none-any.whl → ...data/data/example2-1.0.0-py3-none-any.whl
diff --git a/Lib/test/test_importlib/fixtures.py → Lib/test/test_importlib/metadata/fixtures.py b/Lib/test/test_importlib/fixtures.py → Lib/test/test_importlib/metadata/fixtures.py
@@ -9,6 +9,7 @@
 import functools
 import contextlib
 
+from test.support import import_helper
 from test.support.os_helper import FS_NONASCII
 from test.support import requires_zlib
 
@@ -85,6 +86,7 @@ def add_sys_path(dir):
     def setUp(self):
         super().setUp()
         self.fixtures.enter_context(self.add_sys_path(self.site_dir))
+        self.fixtures.enter_context(import_helper.isolated_modules())
 
 
 class SiteBuilder(SiteDir):
@@ -348,7 +350,7 @@ def DALS(str):
 
 @requires_zlib()
 class ZipFixtures:
-    root = 'test.test_importlib.data'
+    root = 'test.test_importlib.metadata.data'
 
     def _fixture_on_path(self, filename):
         pkg_file = resources.files(self.root).joinpath(filename)

diff --git a/Lib/test/test_importlib/stubs.py → Lib/test/test_importlib/metadata/stubs.py b/Lib/test/test_importlib/stubs.py → Lib/test/test_importlib/metadata/stubs.py
diff --git a/Lib/test/test_importlib/test_metadata_api.py → Lib/test/test_importlib/metadata/test_api.py b/Lib/test/test_importlib/test_metadata_api.py → Lib/test/test_importlib/metadata/test_api.py
diff --git a/Lib/test/test_importlib/test_main.py → ...test/test_importlib/metadata/test_main.py b/Lib/test/test_importlib/test_main.py → ...test/test_importlib/metadata/test_main.py
diff --git a/Lib/test/test_importlib/test_zip.py → Lib/test/test_importlib/metadata/test_zip.py b/Lib/test/test_importlib/test_zip.py → Lib/test/test_importlib/metadata/test_zip.py
diff --git a/Makefile.pre.in b/Makefile.pre.in
@@ -2351,10 +2351,11 @@ TESTSUBDIRS=	idlelib/idle_test \
 		test/test_import/data/unwritable \
 		test/test_importlib \
 		test/test_importlib/builtin \
-		test/test_importlib/data \
 		test/test_importlib/extension \
 		test/test_importlib/frozen \
 		test/test_importlib/import_ \
+		test/test_importlib/metadata \
+		test/test_importlib/metadata/data \
 		test/test_importlib/namespace_pkgs \
 		test/test_importlib/namespace_pkgs/both_portions \
 		test/test_importlib/namespace_pkgs/both_portions/foo \

diff --git a/Misc/NEWS.d/next/Tests/2024-03-20-14-19-32.gh-issue-117089.WwR1Z1.rst b/Misc/NEWS.d/next/Tests/2024-03-20-14-19-32.gh-issue-117089.WwR1Z1.rst
@@ -0,0 +1 @@
+Consolidated tests for importlib.metadata in their own ``metadata`` package.