fix(#59): allow spaces in sanitizeFleName

owlplug-client/src/main/java/com/owlplug/core/utils/FileUtils.java

@@ -48,14 +48,9 @@ public static String convertPath(String path) {
     return path.replace("\\", "/");
   }
 
-  public static boolean isFilenameValid(String fileName) {
-
-    return fileName != null && !fileName.equals("") && fileName.matches("[-_.A-Za-z0-9]*");
-  }
-
   public static String sanitizeFileName(String fileName) {
 
-    return fileName.replaceAll("[^-_.A-Za-z0-9]", "");
+    return fileName.replaceAll("[^-_.A-Za-z0-9 ]", "").trim().replaceAll("\\s+"," ");
 
   }
 

owlplug-client/src/test/java/com/owlplug/core/utils/FileUtilsTest.java

@@ -0,0 +1,33 @@
+package com.owlplug.core.utils;
+
+import static org.junit.Assert.assertEquals;
+
+import org.junit.Test;
+
+public class FileUtilsTest {
+
+	@Test
+	public void testSanitizeFileNameWithRegularsChars() {
+		String sanitizedFileName = FileUtils.sanitizeFileName("File-name0_.test");
+		assertEquals("File-name0_.test", sanitizedFileName);
+	}
+
+	@Test
+	public void testSanitizeFileNameWithExtraSpaces() {
+		String sanitizedFileName = FileUtils.sanitizeFileName(" file   name ");
+		assertEquals("file name", sanitizedFileName);
+	}
+
+	@Test
+	public void testSanitizeFileNameWithWhitespacesChars() {
+		String sanitizedFileName = FileUtils.sanitizeFileName("\tfile\n\nname");
+		assertEquals("filename", sanitizedFileName);
+	}
+
+	@Test
+	public void testSanitizeFileNameWithIllegalChars() {
+		String sanitizedFileName = FileUtils.sanitizeFileName("fi/len%am[e]");
+		assertEquals("filename", sanitizedFileName);
+	}
+
+}