pathlib: Fix glob/rglob to return Path objects rather than strings.

This now matches CPython behaviour.

Signed-off-by: Damien George <damien@micropython.org>

Author: dpgeorge

python-stdlib/pathlib/manifest.py

@@ -1,3 +1,3 @@
-metadata(version="0.0.1")
+metadata(version="0.0.2")
 
 module("pathlib.py")

python-stdlib/pathlib/pathlib.py

@@ -126,7 +126,7 @@ def _glob(self, path, pattern, recursive):
         for name, mode, *_ in os.ilistdir(path):
             full_path = path + _SEP + name
             if name.startswith(prefix) and name.endswith(suffix):
-                yield full_path
+                yield Path(full_path)
             if recursive and mode & 0x4000:  # is_dir
                 yield from self._glob(full_path, pattern, recursive=recursive)
 

python-stdlib/pathlib/tests/test_pathlib.py

@@ -164,7 +164,10 @@ def test_glob(self):
         glob_gen = path.glob("*.txt")
         self.assertTrue(_isgenerator(glob_gen))
 
-        res = [str(x) for x in glob_gen]
+        res = list(glob_gen)
+        for p in res:
+            self.assertIsInstance(p, Path)
+
         self.assertTrue(len(res) == 2)
         self.assertTrue(foo_txt in res)
         self.assertTrue(bar_txt in res)
@@ -190,7 +193,10 @@ def test_rglob(self):
         glob_gen = path.rglob("*.txt")
         self.assertTrue(_isgenerator(glob_gen))
 
-        res = [str(x) for x in glob_gen]
+        res = list(glob_gen)
+        for p in res:
+            self.assertIsInstance(p, Path)
+
         self.assertTrue(len(res) == 3)
         self.assertTrue(foo_txt in res)
         self.assertTrue(bar_txt in res)