Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed file path issue on Windows #348

Merged
merged 1 commit into from
Apr 8, 2019

Conversation

NikolasKomonen
Copy link
Contributor

@NikolasKomonen NikolasKomonen commented Apr 8, 2019

@NikolasKomonen NikolasKomonen force-pushed the fileCachePathFix branch 2 times, most recently from def18ec to d61da95 Compare April 8, 2019 17:35
@NikolasKomonen NikolasKomonen force-pushed the fileCachePathFix branch 3 times, most recently from c991829 to 400b503 Compare April 8, 2019 19:25
@NikolasKomonen NikolasKomonen force-pushed the fileCachePathFix branch 2 times, most recently from 4c0bd96 to 55321bf Compare April 8, 2019 19:45
@NikolasKomonen NikolasKomonen force-pushed the fileCachePathFix branch 2 times, most recently from 18e38b8 to b1cb9ce Compare April 8, 2019 21:04
assertEquals("~/Test/Folder", FilesUtils.normalizePath("./~/Test/Folder"));
assertEquals("/Folder", FilesUtils.normalizePath("/Test/../Folder"));
assertEquals(Paths.get(System.getProperty("user.home"), "Test", "Folder").toString(), FilesUtils.normalizePath("~/Test/Folder"));
assertEquals(Paths.get(separator, "Test", "~", "Folder").toString(), FilesUtils.normalizePath("/Test/~/Folder"));
Copy link
Contributor Author

@NikolasKomonen NikolasKomonen Apr 8, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fbricon This test was failing for me on Windows, an extra \ was inserted at the beginning of the output of Paths.get(...) could you check if you have the same issue

@NikolasKomonen
Copy link
Contributor Author

@fbricon updated

@fbricon fbricon merged commit 9245f57 into eclipse-lemminx:master Apr 8, 2019
@fbricon fbricon added this to the v0.5.1 milestone Apr 8, 2019
@NikolasKomonen NikolasKomonen deleted the fileCachePathFix branch May 29, 2019 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants