fix: allow withdirs parameter in glob for AbstractFileSystem

fsspec/spec.py

@@ -611,6 +611,7 @@ def glob(self, path, maxdepth=None, **kwargs):
         min_idx = min(idx_star, idx_qmark, idx_brace)
 
         detail = kwargs.pop("detail", False)
+        withdirs = kwargs.pop("withdirs", True)
 
         if not has_magic(path):
             if self.exists(path, **kwargs):
@@ -639,7 +640,9 @@ def glob(self, path, maxdepth=None, **kwargs):
             else:
                 depth = None
 
-        allpaths = self.find(root, maxdepth=depth, withdirs=True, detail=True, **kwargs)
+        allpaths = self.find(
+            root, maxdepth=depth, withdirs=withdirs, detail=True, **kwargs
+        )
 
         pattern = glob_translate(path + ("/" if ends_with_sep else ""))
         pattern = re.compile(pattern)

fsspec/tests/test_spec.py

@@ -1376,6 +1376,15 @@ def test_glob_posix_rules(path, expected, glob_fs):
     for name, info in _clean_paths(detailed_output).items():
         assert info == glob_fs[name]
 
+    withdirs_output = glob_fs.glob(path=f"mock://{path}", detail=True, withdirs=False)
+    path_output = _clean_paths(withdirs_output)
+    for name, info in path_output.items():
+        # withdirs only respected when path has magic
+        # otherwise glob returns the path regardless of the type of the path
+        if glob.has_magic(path):
+            assert info["type"] == "file"
+        assert info == glob_fs[name]
+
 
 @pytest.fixture(scope="function")
 def tmpfs(tmpdir):