Revert "test_packages_distributions_all_module_types: Create valid import names"

jaraco · jaraco · commit 5dbe83cdb056 · 2023-04-09T15:49:38.000-04:00
This reverts commit 70ff991. This behavior was adopted in 5e8260c and subsequently adapted as part of python#443.
diff --git a/tests/test_main.py b/tests/test_main.py
@@ -342,10 +342,10 @@ def test_packages_distributions_all_module_types(self):
         filenames = list(
             itertools.chain.from_iterable(
                 [
-                    f'top_level_{i}{suffix}',
-                    f'in_namespace_{i}/mod{suffix}',
-                    f'in_package_{i}/__init__.py',
-                    f'in_package_{i}/mod{suffix}',
+                    f'{i}-top-level{suffix}',
+                    f'{i}-in-namespace/mod{suffix}',
+                    f'{i}-in-package/__init__.py',
+                    f'{i}-in-package/mod{suffix}',
                 ]
                 for i, suffix in enumerate(suffixes)
             )
@@ -367,14 +367,9 @@ def test_packages_distributions_all_module_types(self):
         distributions = packages_distributions()
 
         for i in range(len(suffixes)):
-            assert distributions[f'top_level_{i}'] == ['all_distributions']
-            assert distributions[f'in_namespace_{i}'] == ['all_distributions']
-            assert distributions[f'in_package_{i}'] == ['all_distributions']
-
-        # All keys returned from packages_distributions() should be valid import
-        # names, which means that they must _at least_ be valid identifiers:
-        for import_name in distributions.keys():
-            assert import_name.isidentifier(), import_name
+            assert distributions[f'{i}-top-level'] == ['all_distributions']
+            assert distributions[f'{i}-in-namespace'] == ['all_distributions']
+            assert distributions[f'{i}-in-package'] == ['all_distributions']
 
 
 class PackagesDistributionsEggTest(
