Remove unnecessary parens, add unit test (#102, #149, #170)

Author: fsteeg

metafix/src/main/java/org/metafacture/metafix/FixPath.java

@@ -136,7 +136,7 @@ public String toString() {
             result = new FixPath(replaceInPath(ASTERISK, i));
         }
         // Multiple * or wildcards: use the old value's path
-        else if (value.getPath() != null && (countAsterisks() >= 2) || hasWildcard()) {
+        else if (value.getPath() != null && countAsterisks() >= 2 || hasWildcard()) {
             result = new FixPath(value.getPath());
         }
         else {

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

@@ -369,6 +369,36 @@ public void shouldNotDeleteNonFoundLookupExplicitly() {
         );
     }
 
+    @Test
+    // See https://github.com/metafacture/metafacture-fix/issues/149
+    public void shouldDeleteNonFoundLookupOnDemandNonRepeatedField() {
+        MetafixTestHelpers.assertFix(streamReceiver, Arrays.asList(
+                "lookup('title', Aloha: Alohaeha, 'Moin': 'Moin zäme', delete: 'true')"
+            ),
+            i -> {
+                i.startRecord("1");
+                i.literal("title", "Aloha");
+                i.endRecord();
+                i.startRecord("2");
+                i.literal("title", "Moin");
+                i.endRecord();
+                i.startRecord("3");
+                i.literal("title", "Yo");
+                i.endRecord();
+            },
+            o -> {
+                o.get().startRecord("1");
+                o.get().literal("title", "Alohaeha");
+                o.get().endRecord();
+                o.get().startRecord("2");
+                o.get().literal("title", "Moin zäme");
+                o.get().endRecord();
+                o.get().startRecord("3");
+                o.get().endRecord();
+            }
+        );
+    }
+
     @Test
     public void shouldDeleteNonFoundLookupOnDemandAndVacuum() {
         MetafixTestHelpers.assertFix(streamReceiver, Arrays.asList(