Fix pkgutil.iter_modules regression

Author: miguendes

Lib/pkgutil.py

@@ -413,6 +413,7 @@ def get_importer(path_item):
     The cache (or part of it) can be cleared manually if a
     rescan of sys.path_hooks is necessary.
     """
+    path_item = os.fsdecode(path_item)
     try:
         importer = sys.path_importer_cache[path_item]
     except KeyError:

Lib/test/test_pkgutil.py

@@ -1,3 +1,4 @@
+from pathlib import Path
 from test.support import run_unittest
 from test.support.import_helper import unload, CleanImport
 from test.support.warnings_helper import check_warnings
@@ -92,6 +93,15 @@ def test_getdata_zipfile(self):
 
         del sys.modules[pkg]
 
+    def test_iter_modules(self):
+        #see: issue44061
+        actual = list(pkgutil.iter_modules([Path("/home")], 'somepackage.somesubpackage'))
+        self.assertListEqual([], actual)
+
+        expected_msg = "path must be None or list of paths to look for modules in"
+        with self.assertRaisesRegex(ValueError, expected_msg):
+            list(pkgutil.iter_modules("invalid_path"))
+
     def test_unreadable_dir_on_syspath(self):
         # issue7367 - walk_packages failed if unreadable dir on sys.path
         package_name = "unreadable_package"
@@ -574,6 +584,12 @@ def test_get_importer_avoids_emulation(self):
             self.assertIsNone(pkgutil.get_importer("*??"))
             self.assertEqual(len(w.warnings), 0)
 
+    def test_issue44061(self):
+        try:
+            pkgutil.get_importer(Path("/home"))
+        except AttributeError:
+            self.fail("Unexpected AttributeError when calling get_importer")
+
     def test_iter_importers_avoids_emulation(self):
         with check_warnings() as w:
             for importer in pkgutil.iter_importers(): pass

Misc/NEWS.d/next/Library/2021-05-07-08-39-23.bpo-44061.MvElG6.rst

@@ -0,0 +1,2 @@
+Fix regression when calling :func:`pkgutil.iter_modules` with a list of
+:class:`pathlib.Path` objects