For #887: Ignore lambdas on constructors.

Author: paulodamaso

qulice-pmd/src/test/java/com/qulice/pmd/PmdValidatorTest.java

@@ -39,6 +39,8 @@
 import java.util.Collections;
 import org.hamcrest.MatcherAssert;
 import org.hamcrest.Matchers;
+import org.hamcrest.core.IsEqual;
+import org.hamcrest.core.IsNot;
 import org.junit.Test;
 
 /**
@@ -310,6 +312,24 @@ public void forbidsCodeInConstructor()
         ).validate();
     }
 
+    /**
+     * PmdValidator allows lambda in constructor.
+     * @throws Exception If something wrong happens inside.
+     */
+    @Test
+    public void allowsLambdaInConstructor()
+        throws Exception {
+        final String file = "LambdaInConstructor.java";
+        new PmdAssert(
+            file, new IsEqual<>(true),
+            new IsNot<>(
+                RegexMatchers.containsPattern(
+                    String.format(PmdValidatorTest.CODE_IN_CON, file)
+                )
+            )
+        ).validate();
+    }
+
     /**
      * PmdValidator forbids usage of Files.createFile in tests.
      * @throws Exception If something wrong happens inside.

qulice-pmd/src/test/resources/com/qulice/pmd/LambdaInConstructor.java

@@ -0,0 +1,30 @@
+package foo;
+
+public final class LambdaInConstructor {
+    private final transient int number;
+    private final transient int another;
+
+    public LambdaInConstructor(final int parameter) {
+        this(
+            parameter,
+             () -> {
+                final int ret;
+                if (parameter % 2 == 0){
+                    ret = 10;
+                } else {
+                    ret = 20;
+                }
+                return ret;
+             }
+        );
+    }
+
+    public LambdaInConstructor(final int parameter, final int other) {
+        this.number = parameter;
+        this.another = other;
+    }
+
+    public int num() {
+        return number + another;
+    }
+}