Merge pull request #226 from ajoberstar/consumer

Generate correct Consumer type for ops

src/main/groovy/org/ajoberstar/grgit/internal/WithOperationsASTTransformation.java

@@ -101,7 +101,7 @@ private MethodNode makeMapMethod(ClassNode targetClass, String opName, ClassNode
 
   private MethodNode makeConsumerMethod(ClassNode targetClass, String opName, ClassNode opClass, ClassNode opReturn, boolean isStatic) {
     ClassNode parmType = classFromType(Consumer.class);
-    GenericsType[] generics = new GenericsType[] {new GenericsType(opReturn)};
+    GenericsType[] generics = new GenericsType[] {new GenericsType(opClass)};
     parmType.setGenericsTypes(generics);
     Parameter[] parms = new Parameter[] {new Parameter(parmType, "arg")};
 

src/test/groovy/org/ajoberstar/grgit/operation/GenericOpTest.java

@@ -0,0 +1,32 @@
+package org.ajoberstar.grgit.operation;
+
+import java.io.File;
+import java.io.IOException;
+import java.util.Arrays;
+import java.util.HashSet;
+import org.junit.Test;
+import static org.junit.Assert.*;
+import org.junit.Rule;
+import org.junit.rules.TemporaryFolder;
+import org.ajoberstar.grgit.Grgit;
+
+public class GenericOpTest {
+  @Rule
+  public TemporaryFolder tempDir = new TemporaryFolder();
+
+  @Test
+  public void consumerOperationWorks() throws IOException {
+    File dir = tempDir.newFolder();
+    Grgit grgit = Grgit.init(op -> {
+      op.setDir(dir);
+    });
+    grgit.add(op -> {
+      op.setPatterns(new HashSet<>(Arrays.asList(".")));
+    });
+    grgit.commit(op -> {
+      op.setMessage("First commit");
+    });
+    assertEquals(1, grgit.log().size());
+    grgit.close();
+  }
+}