Merge pull request #757 from metafacture/751-skipEmptyStringWhenCapitalize

Author: TobiasNx

metafix/src/main/java/org/metafacture/metafix/method/field/Capitalize.java

@@ -35,7 +35,7 @@ public Capitalize() {
 
     @Override
     public void apply(final Metafix metafix, final Record record, final List<String> params, final Map<String, String> options) {
-        record.transform(params.get(0), s -> s.substring(0, 1).toUpperCase() + s.substring(1));
+        record.transform(params.get(0), s -> s.isEmpty() ? s : s.substring(0, 1).toUpperCase() + s.substring(1));
     }
 
 }

metafix/src/test/java/org/metafacture/metafix/MetafixMethodTest.java

@@ -169,6 +169,24 @@ public void shouldCapitalizeString() {
         );
     }
 
+    @Test
+    public void shouldSkipEmptyStringWhenCapitalize() {
+        MetafixTestHelpers.assertFix(streamReceiver, Arrays.asList(
+                "capitalize('title')"
+            ),
+            i -> {
+                i.startRecord("1");
+                i.literal("title", "");
+                i.endRecord();
+            },
+            o -> {
+                o.get().startRecord("1");
+                o.get().literal("title", "");
+                o.get().endRecord();
+            }
+        );
+    }
+
     @Test
     public void shouldCapitalizeStringsInArray() {
         MetafixTestHelpers.assertFix(streamReceiver, Arrays.asList(