fix path inferring with expandDirectories disabled

SuperchupuDev · Aug 24, 2024 · f9fe994 · f9fe994
1 parent ae772b9
commit f9fe994
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/src/index.ts b/src/index.ts
@@ -62,11 +62,12 @@ function normalizePattern(
       if (properties.commonPath[i] === part && !/[\!\*\{\}\(\)]/.test(part)) {
         newCommonPath.push(part);
       } else {
-        newCommonPath.pop();
         break;
       }
     }
 
+    newCommonPath.pop();
+
     properties.depthOffset = newCommonPath.length;
     properties.commonPath = newCommonPath;
 

diff --git a/test/index.test.ts b/test/index.test.ts
@@ -68,6 +68,16 @@ test('expandDirectories true', async () => {
   assert.deepEqual(files.sort(), ['a/a.ts', 'a/b.ts']);
 });
 
+test("no expandDirectories doesn't break common path inferring", async () => {
+  const files = await glob({ patterns: ['a/a.ts'], expandDirectories: false, cwd });
+  assert.deepEqual(files.sort(), ['a/a.ts']);
+});
+
+test("expandDirectories doesn't break common path inferring either", async () => {
+  const files = await glob({ patterns: ['a/a.ts'], expandDirectories: true, cwd });
+  assert.deepEqual(files.sort(), ['a/a.ts']);
+});
+
 test('handle absolute patterns to some extent', async () => {
   const files = await glob({ patterns: [`${cwd.replaceAll('\\', '/')}/a/a.ts`], cwd });
   assert.deepEqual(files.sort(), ['a/a.ts']);