test: edge cases for map fs

harttle · Jul 9, 2024 · fad00aa · fad00aa
1 parent df27ac6
commit fad00aa
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/src/fs/map-fs.spec.ts b/src/fs/map-fs.spec.ts
@@ -14,4 +14,13 @@ describe('MapFS', () => {
   it('should resolve exceeding root', () => {
     expect(fs.resolve('foo/bar', '../../../coo', '')).toEqual('coo')
   })
+  it('should resolve from absolute path', () => {
+    expect(fs.resolve('/foo/bar', '../../coo', '')).toEqual('/coo')
+  })
+  it('should resolve exceeding root from absolute path', () => {
+    expect(fs.resolve('/foo/bar', '../../../coo', '')).toEqual('/coo')
+  })
+  it('should resolve from invalid path', () => {
+    expect(fs.resolve('foo//bar', '../coo', '')).toEqual('foo/coo')
+  })
 })
diff --git a/src/fs/map-fs.ts b/src/fs/map-fs.ts
@@ -32,10 +32,12 @@ export class MapFS {
   resolve (dir: string, file: string, ext: string) {
     file += ext
     if (dir === '.') return file
-    const segments = dir.split(this.sep)
+    const segments = dir.split(/\/+/)
     for (const segment of file.split(this.sep)) {
       if (segment === '.' || segment === '') continue
-      else if (segment === '..') segments.pop()
+      else if (segment === '..') {
+        if (segments.length > 1 || segments[0] !== '') segments.pop()
+      }
       else segments.push(segment)
     }
     return segments.join(this.sep)