Skip to content

Commit

Permalink
fix(windows): closes #362
Browse files Browse the repository at this point in the history
  • Loading branch information
Mayank1791989 committed Sep 9, 2019
1 parent 53b388e commit f1879e1
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 2 deletions.
10 changes: 10 additions & 0 deletions lib/flowLSP/utils/__tests__/uriToString.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,16 @@ test('[windows] upper-case drive letter', () => {
expect(uriToString(uri)).toEqual('file:///C:/flow/test');
});

test('[windows] upper-case drive letter', () => {
const uri = URI.file('e:/flow/test');
expect(uriToString(uri)).toEqual('file:///E:/flow/test');
});

test('[windows] upper-case drive letter', () => {
const uri = URI.file('/e:/flow/test');
expect(uriToString(uri)).toEqual('file:///E:/flow/test');
});

test('linux path', () => {
const uri = URI.file('/flow/test');
expect(uriToString(uri)).toEqual('file:///flow/test');
Expand Down
4 changes: 2 additions & 2 deletions lib/flowLSP/utils/uriToString.js
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ function makeWindowsDriveLetterUppercase(
const lowerCaseDriveLetter = windowsDriveLetter.toLowerCase();
const upperCaseDriveLetter = windowsDriveLetter.toUpperCase();
return uriString.replace(
`${lowerCaseDriveLetter}:`,
`${upperCaseDriveLetter}:`,
`/${lowerCaseDriveLetter}:`,
`/${upperCaseDriveLetter}:`,
);
}

0 comments on commit f1879e1

Please sign in to comment.