IO-163 Change FileUtils.toURLs() to use File.toURI().toURL() rather than File.toURL() - thanks to Alex Miller for the suggestion

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/io/trunk@647000 13f79535-47bb-0310-9956-ffa450edef68

Author: Niall Pemberton

src/java/org/apache/commons/io/FileUtils.java

@@ -532,7 +532,7 @@ public static URL[] toURLs(File[] files) throws IOException {
         URL[] urls = new URL[files.length];
 
         for (int i = 0; i < urls.length; i++) {
-            urls[i] = files[i].toURL();
+            urls[i] = files[i].toURI().toURL();
         }
 
         return urls;

src/test/org/apache/commons/io/FileUtilsTestCase.java

@@ -328,6 +328,7 @@ public void testToURLs1() throws Exception {
         File[] files = new File[] {
             new File(getTestDirectory(), "file1.txt"),
             new File(getTestDirectory(), "file2.txt"),
+            new File(getTestDirectory(), "test file.txt"),
         };
         URL[] urls = FileUtils.toURLs(files);
 
@@ -336,6 +337,10 @@ public void testToURLs1() throws Exception {
         assertEquals(true, urls[0].toExternalForm().indexOf("file1.txt") >= 0);
         assertEquals(true, urls[1].toExternalForm().startsWith("file:"));
         assertEquals(true, urls[1].toExternalForm().indexOf("file2.txt") >= 0);
+
+        // Test escaped char
+        assertEquals(true, urls[2].toExternalForm().startsWith("file:"));
+        assertEquals(true, urls[2].toExternalForm().indexOf("test%20file.txt") >= 0);
     }
 
 //    public void testToURLs2() throws Exception {