fix(fs): encode glob returned path (#469)

Fixes #422

Author: autozimu

src/fs.ts

@@ -78,7 +78,7 @@ export class LocalFileSystem implements FileSystem {
                 follow: true,
             })
             globber.on('match', (file: string) => {
-                subscriber.next(normalizeUri(base + file))
+                subscriber.next(file)
             })
             globber.on('error', (err: any) => {
                 subscriber.error(err)
@@ -89,6 +89,12 @@ export class LocalFileSystem implements FileSystem {
             return () => {
                 globber.abort()
             }
+        }).map(file => {
+            const encodedPath = file
+                .split('/')
+                .map(encodeURIComponent)
+                .join('/')
+            return normalizeUri(base + encodedPath)
         })
     }
 

src/test/memfs.test.ts

@@ -18,6 +18,14 @@ describe('memfs.ts', () => {
                 sinon.assert.calledOnce(listener)
                 sinon.assert.calledWithExactly(listener, 'file:///foo/bar.txt', undefined)
             })
+            it('should add just a URI and emit an event when URI has encoded char', () => {
+                const listener = sinon.spy()
+                const fs = new InMemoryFileSystem('/')
+                fs.on('add', listener)
+                fs.add('file:///foo/%25bar.txt')
+                sinon.assert.calledOnce(listener)
+                sinon.assert.calledWithExactly(listener, 'file:///foo/%25bar.txt', undefined)
+            })
             it('should add content for a URI and emit an event', () => {
                 const listener = sinon.spy()
                 const fs = new InMemoryFileSystem('/')