fix: have toWin32Path bail out early if path is already Win32

GoogleChrome · May 21, 2020 · 35fc9ed · 35fc9ed
1 parent 11bc863
commit 35fc9ed
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/src/utils.ts b/src/utils.ts
@@ -74,6 +74,10 @@ export function makeTmpDir() {
 }
 
 export function toWin32Path(dir: string = ''): string {
+  if (/[a-z]:\\/iu.test(dir)) {
+    return dir;
+  }
+
   return execFileSync('wslpath', ['-w', dir]).toString().trim();
 }
 

diff --git a/test/utils-test.ts b/test/utils-test.ts
@@ -34,6 +34,17 @@ describe('toWin32Path', () => {
 
     assert.ok(execFileSyncStub.calledWith('wslpath', ['-w', '']));
   })
+
+  describe('when the path is already in Windows format', () => {
+    it('returns early', () => {
+      execFileSyncStub.returns(asBuffer(''));
+
+      assert.deepStrictEqual(toWin32Path('D:\\'), 'D:\\');
+      assert.deepStrictEqual(toWin32Path('C:\\'), 'C:\\');
+
+      assert.ok(execFileSyncStub.notCalled);
+    });
+  })
 })
 
 describe('toWSLPath', () => {