Expand test to include importable names that aren't identifiers and honor that expectation.

Author: jaraco

importlib_metadata/__init__.py

@@ -904,7 +904,7 @@ def _top_level_inferred(dist):
     }
 
     @pass_none
-    def valid_module(name):
-        return name.isidentifier()
+    def importable_name(name):
+        return '.' not in name
 
-    return filter(valid_module, opt_names)
+    return filter(importable_name, opt_names)

tests/test_main.py

@@ -336,7 +336,7 @@ def test_packages_distributions_all_module_types(self):
                         Version: 1.0.0
                     """,
                     'RECORD': ''.join(
-                        f'top_level_{i}{suffix},,\n'
+                        f'importable-name {i}{suffix},,\n'
                         f'in_namespace_{i}/mod{suffix},,\n'
                         f'in_package_{i}/__init__.py,,\n'
                         f'in_package_{i}/mod{suffix},,\n'
@@ -350,7 +350,7 @@ 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'importable-name {i}'] == ['all_distributions']
             assert distributions[f'in_namespace_{i}'] == ['all_distributions']
             assert distributions[f'in_package_{i}'] == ['all_distributions']