REPO-239 Change FileType lookup logic to fit with the type strings that FileApi accept

FileType.lookup() stays backward compatible, but now it should also accept snake-cased type names

Author: Denis Kurochkin

api-sdk/src/main/java/com/smartling/api/sdk/file/FileType.java

@@ -17,6 +17,9 @@
 
 import org.apache.commons.lang3.StringUtils;
 
+import java.util.HashMap;
+import java.util.Map;
+
 public enum FileType
 {
     JAVA_PROPERTIES("text/plain", true),            // Java resources
@@ -50,7 +53,19 @@ public enum FileType
     private final String mimeType;
     private final boolean isTextFormat;
 
-    private static String createIdentifier(String s)
+    // Lookup map
+    public final static Map<String, FileType> BY_NAME_LOOKUP = new HashMap<>();
+
+    static
+    {
+        for (final FileType value : FileType.values())
+        {
+            BY_NAME_LOOKUP.put(value.identifier.toLowerCase(), value);
+            BY_NAME_LOOKUP.put(value.name().toLowerCase(), value);
+        }
+    }
+
+    private static String toLowerCamel(String s)
     {
         StringBuilder buf = new StringBuilder();
         String[] parts = s.split("_");
@@ -61,9 +76,9 @@ private static String createIdentifier(String s)
         return buf.toString();
     }
 
-    private FileType(final String mimeType, final boolean isTextFormat)
+    FileType(final String mimeType, final boolean isTextFormat)
     {
-        this.identifier = createIdentifier(name());
+        this.identifier = toLowerCamel(name());
         this.mimeType = mimeType;
         this.isTextFormat = isTextFormat;
     }
@@ -85,12 +100,6 @@ public boolean isTextFormat()
 
     public static FileType lookup(final String fileTypeString)
     {
-        for (final FileType fileType : FileType.values())
-        {
-            if (fileType.identifier.equalsIgnoreCase(fileTypeString))
-                return fileType;
-        }
-
-        return null;
+        return BY_NAME_LOOKUP.get(StringUtils.lowerCase(fileTypeString));
     }
 }

api-sdk/src/test/java/com/smartling/api/sdk/file/FileTypeTest.java

@@ -0,0 +1,20 @@
+package com.smartling.api.sdk.file;
+
+import org.junit.Assert;
+import org.junit.Test;
+
+import static com.smartling.api.sdk.file.FileType.JAVA_PROPERTIES;
+
+public class FileTypeTest
+{
+    @Test
+    public void testLookup()
+    {
+        Assert.assertEquals(JAVA_PROPERTIES, FileType.lookup("javaProperties"));
+        Assert.assertEquals(JAVA_PROPERTIES, FileType.lookup("JaVaPrOpErTiEs"));
+        Assert.assertEquals(JAVA_PROPERTIES, FileType.lookup("JaVa_PrOpErTiEs"));
+        Assert.assertEquals(JAVA_PROPERTIES, FileType.lookup(JAVA_PROPERTIES.name()));
+        Assert.assertEquals(JAVA_PROPERTIES, FileType.lookup(JAVA_PROPERTIES.toString()));
+        Assert.assertNull(FileType.lookup("A_NON_EXISTING_CONTENT_TYPE"));
+    }
+}