feat: remove fast-glob package in favor of Node.js fs.glob

Breaking change: The globs support behavior may different. However, we expect most Japa
projects to use standard glob expressions and must work without any issues

Author: thetutlage

package.json

@@ -61,7 +61,6 @@
     "@poppinss/hooks": "^7.3.0",
     "@poppinss/string": "^1.7.1",
     "error-stack-parser-es": "^1.0.5",
-    "fast-glob": "^3.3.3",
     "find-cache-directory": "^6.0.0",
     "getopts": "^2.3.0",
     "supports-color": "^10.2.2"

src/files_manager.ts

@@ -7,8 +7,9 @@
  * file that was distributed with this source code.
  */
 
+import { join } from 'node:path'
 import string from '@poppinss/string'
-import fastGlob from 'fast-glob'
+import { glob } from 'node:fs/promises'
 import { pathToFileURL } from 'node:url'
 import type { TestFiles } from './types.js'
 
@@ -29,13 +30,18 @@ export class FilesManager {
    */
   async getFiles(cwd: string, files: TestFiles, excludes: string[]): Promise<URL[]> {
     if (Array.isArray(files) || typeof files === 'string') {
-      const testFiles = await fastGlob(files, {
-        absolute: true,
-        onlyFiles: true,
+      const matchingFiles = glob(files, {
+        withFileTypes: false,
         cwd: cwd,
-        ignore: excludes,
+        exclude: excludes,
       })
-      return testFiles.map((file) => pathToFileURL(file))
+
+      const testFiles = await Array.fromAsync(matchingFiles)
+      return testFiles
+        .sort((current, next) => {
+          return current.localeCompare(next, undefined, { numeric: true, sensitivity: 'base' })
+        })
+        .map((file) => pathToFileURL(join(cwd, file)))
     }
 
     return await files()

tests/files_manager.spec.ts

@@ -129,6 +129,9 @@ test.describe('Files manager | getFiles', () => {
 
     await wrapAssertions(() => {
       assert.deepEqual(files, [
+        new URL('modules/core/main.ts', cwd),
+        new URL('modules/core/reporters/base.ts', cwd),
+        new URL('modules/core/types.ts', cwd),
         new URL('tests/base_reporter.spec.ts', cwd),
         new URL('tests/cli_parser.spec.ts', cwd),
         new URL('tests/config_manager.spec.ts', cwd),
@@ -138,9 +141,6 @@ test.describe('Files manager | getFiles', () => {
         new URL('tests/helpers.spec.ts', cwd),
         new URL('tests/planner.spec.ts', cwd),
         new URL('tests/runner.spec.ts', cwd),
-        new URL('modules/core/main.ts', cwd),
-        new URL('modules/core/types.ts', cwd),
-        new URL('modules/core/reporters/base.ts', cwd),
       ])
     })
   })