Move cwd ENOENT test to where it matters

Fix: #515

Author: isaacs

package-lock.json

@@ -69,6 +69,7 @@
     "@types/tap": "^15.0.7",
     "c8": "^7.12.0",
     "eslint-config-prettier": "^8.6.0",
+    "memfs": "^3.4.13",
     "mkdirp": "^2.1.4",
     "prettier": "^2.8.3",
     "rimraf": "^4.1.3",

package.json

@@ -136,6 +136,8 @@ export class Processor {
         }
       }
 
+      if (t.isENOENT()) continue
+
       let p: MMPattern
       let rest: Pattern | null
       let changed = false

src/processor.ts

@@ -405,30 +405,30 @@ export class GlobWalker<
 
   async walk(): Promise<Matches<O>> {
     if (this.signal?.aborted) throw this.signal.reason
-    const t = this.path.isUnknown() ? await this.path.lstat() : this.path
-    if (t) {
-      await new Promise((res, rej) => {
-        this.walkCB(t, this.patterns, () => {
-          if (this.signal?.aborted) {
-            rej(this.signal.reason)
-          } else {
-            res(this.matches)
-          }
-        })
-      })
+    if (this.path.isUnknown()) {
+      await this.path.lstat()
     }
+    await new Promise((res, rej) => {
+      this.walkCB(this.path, this.patterns, () => {
+        if (this.signal?.aborted) {
+          rej(this.signal.reason)
+        } else {
+          res(this.matches)
+        }
+      })
+    })
     return this.matches
   }
 
   walkSync(): Matches<O> {
     if (this.signal?.aborted) throw this.signal.reason
-    const t = this.path.isUnknown() ? this.path.lstatSync() : this.path
-    // nothing for the callback to do, because this never pauses
-    if (t) {
-      this.walkCBSync(t, this.patterns, () => {
-        if (this.signal?.aborted) throw this.signal.reason
-      })
+    if (this.path.isUnknown()) {
+      this.path.lstatSync()
     }
+    // nothing for the callback to do, because this never pauses
+    this.walkCBSync(this.path, this.patterns, () => {
+      if (this.signal?.aborted) throw this.signal.reason
+    })
     return this.matches
   }
 }
@@ -463,12 +463,8 @@ export class GlobStream<
   stream(): MatchStream<O> {
     const target = this.path
     if (target.isUnknown()) {
-      target.lstat().then(e => {
-        if (e) {
-          this.walkCB(target, this.patterns, () => this.results.end())
-        } else {
-          this.results.end()
-        }
+      target.lstat().then(() => {
+        this.walkCB(target, this.patterns, () => this.results.end())
       })
     } else {
       this.walkCB(target, this.patterns, () => this.results.end())
@@ -477,14 +473,10 @@ export class GlobStream<
   }
 
   streamSync(): MatchStream<O> {
-    const target = this.path.isUnknown()
-      ? this.path.lstatSync()
-      : this.path
-    if (target) {
-      this.walkCBSync(target, this.patterns, () => this.results.end())
-    } else {
-      this.results.end()
+    if (this.path.isUnknown()) {
+      this.path.lstatSync()
     }
+    this.walkCBSync(this.path, this.patterns, () => this.results.end())
     return this.results
   }
 }

src/walker.ts

@@ -0,0 +1,16 @@
+import { fs as memfs, vol } from 'memfs'
+import t from 'tap'
+import { glob } from '../'
+t.beforeEach(() => vol.fromJSON({ '/x': 'abc' }))
+
+const fs = memfs as unknown as typeof import('fs')
+
+t.test('should match single file with pattern', async t => {
+  const expandedFiles = await glob(['.', '/**/*'], { nodir: true, fs })
+  t.strictSame(expandedFiles, ['/x'])
+})
+
+t.test('should match single file without pattern', async t => {
+  const expandedFiles = await glob(['.', '/x'], { nodir: true, fs })
+  t.strictSame(expandedFiles, ['/x'])
+})