Filter non-identifiers from module names. Fixes #442.

jaraco · jaraco · commit f9dd1640fcef · 2023-03-25T11:19:29.000-04:00
diff --git a/importlib_metadata/__init__.py b/importlib_metadata/__init__.py
@@ -902,4 +902,9 @@ def _top_level_inferred(dist):
         f.parts[0] if len(f.parts) > 1 else inspect.getmodulename(f)
         for f in always_iterable(dist.files)
     }
-    return filter(None, opt_names)
+
+    @pass_none
+    def valid_module(name):
+        return name.isidentifier()
+
+    return filter(valid_module, opt_names)
diff --git a/tests/test_main.py b/tests/test_main.py
@@ -323,7 +323,6 @@ def test_packages_distributions_neither_toplevel_nor_files(self):
         )
         packages_distributions()
 
-    @__import__('pytest').mark.xfail(reason="442")
     def test_packages_distributions_all_module_types(self):
         """
         Test top-level modules detected on a package without 'top-level.txt'.

Original file line number	Diff line number	Diff line change
`@@ -323,7 +323,6 @@ def test_packages_distributions_neither_toplevel_nor_files(self):`
`323`	`323`	`)`
`324`	`324`	`packages_distributions()`
`325`	`325`
`326`		`- @__import__('pytest').mark.xfail(reason="442")`
`327`	`326`	`def test_packages_distributions_all_module_types(self):`
`328`	`327`	`"""`
`329`	`328`	`Test top-level modules detected on a package without 'top-level.txt'.`