Create path using File.separator instead of "/"

malaporte / nashorn-commonjs-modules

CommonJS modules support for Nashorn

MIT License

108 stars 31 forks source link

Closed arisone closed 6 years ago

arisone commented 6 years ago

In order to fix #28 I propose:

in FilesystemFolderuse File.separator instead of "/"
in FilesystemFolderTestremove all assert based on string constants and instead create Fileobject for the test folders and use File.getPath and File.separator to create the root path to use when assert Folder.getPath.startsWith and instead use File.separator to assert Folder.getPath.endsWith

malaporte commented 6 years ago

Much better than my blind attempt, thanks!