fs: correctly pass direct to exclude withFileTypes

Author: avivkeller

lib/internal/fs/glob.js

@@ -543,7 +543,7 @@ class Glob {
         const fromSymlink = pattern.symlinks.has(index);
 
         if (current === lazyMinimatch().GLOBSTAR) {
-          if (entry.name[0] === '.' || (this.#exclude && this.#exclude(entry.name))) {
+          if (entry.name[0] === '.' || (this.#exclude && this.#exclude(this.#withFileTypes ? entry : entry.name))) {
             continue;
           }
           if (!fromSymlink && entry.isDirectory()) {

test/parallel/test-fs-glob.mjs

@@ -342,7 +342,7 @@ describe('glob - withFileTypes', function() {
   const promisified = promisify(glob);
   for (const [pattern, expected] of Object.entries(patterns)) {
     test(pattern, async () => {
-      const actual = await promisified(pattern, { cwd: fixtureDir, withFileTypes: true });
+      const actual = await promisified(pattern, { cwd: fixtureDir, withFileTypes: true, exclude: (dirent) => assert.ok(dirent instanceof Dirent) });
       assertDirents(actual);
       const normalized = expected.filter(Boolean).map((item) => basename(item)).sort();
       assert.deepStrictEqual(actual.map((dirent) => dirent.name).sort(), normalized.sort());
@@ -353,7 +353,7 @@ describe('glob - withFileTypes', function() {
 describe('globSync - withFileTypes', function() {
   for (const [pattern, expected] of Object.entries(patterns)) {
     test(pattern, () => {
-      const actual = globSync(pattern, { cwd: fixtureDir, withFileTypes: true });
+      const actual = globSync(pattern, { cwd: fixtureDir, withFileTypes: true, exclude: (dirent) => assert.ok(dirent instanceof Dirent) });
       assertDirents(actual);
       const normalized = expected.filter(Boolean).map((item) => basename(item)).sort();
       assert.deepStrictEqual(actual.map((dirent) => dirent.name).sort(), normalized.sort());
@@ -365,7 +365,7 @@ describe('fsPromises glob - withFileTypes', function() {
   for (const [pattern, expected] of Object.entries(patterns)) {
     test(pattern, async () => {
       const actual = [];
-      for await (const item of asyncGlob(pattern, { cwd: fixtureDir, withFileTypes: true })) actual.push(item);
+      for await (const item of asyncGlob(pattern, { cwd: fixtureDir, withFileTypes: true, exclude: (dirent) => assert.ok(dirent instanceof Dirent) })) actual.push(item);
       assertDirents(actual);
       const normalized = expected.filter(Boolean).map((item) => basename(item)).sort();
       assert.deepStrictEqual(actual.map((dirent) => dirent.name).sort(), normalized.sort());