[ 
https://issues.apache.org/jira/browse/SLING-1113?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12756548#action_12756548
 ] 

Mike Müller commented on SLING-1113:
------------------------------------

That solves the problem. thanks.
Just one little thing: The test now covers not all possible cases. As described 
in [1] on a windows system the separator is "\" but if you load the file 
through getClass().getResource the path contains only "/" even on a windows 
system.
So I would like to include this case in the test cases also. 
WDYT?

[1] http://markmail.org/message/3hxln3pnxdtz47ak

> BootstrapInstallerTest fails on Windows systems
> -----------------------------------------------
>
>                 Key: SLING-1113
>                 URL: https://issues.apache.org/jira/browse/SLING-1113
>             Project: Sling
>          Issue Type: Bug
>          Components: Testing
>         Environment: Windows XP SP3
>            Reporter: Mike Müller
>            Assignee: Mike Müller
>            Priority: Trivial
>         Attachments: BootstrapInstallerTest.java.patch, 
> SLING-1113-fmeschbe.patch
>
>
> The BootstrapInstallerTest.testExtractFileName fails because on a Windows 
> system the separator char for a directory is \ and not /. The test does not 
> use File.separatorChar to get the right separator but the implementation in 
> BootstrapInstaller does.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to