Fix generic return type lambda breakpoint issue. Fix #1359 (#499)

* Fix generic return type lambda breakpoint issue. Fix #1359

The fix tries to compare the runtime lambda method signature with none
generic version of the method signature found from AST traversal as a
additional comparison to what is there.

Author: gayanper

com.microsoft.java.debug.core/src/main/java/com/microsoft/java/debug/core/Breakpoint.java

@@ -325,7 +325,8 @@ private Location findMethodLocaiton(ReferenceType refType, String methodName, St
         for (Method method : methods) {
             if (!method.isAbstract() && !method.isNative()
                     && methodName.equals(method.name())
-                    && (methodSiguature.equals(method.genericSignature()) || methodSiguature.equals(method.signature()))) {
+                    && (methodSiguature.equals(method.genericSignature()) || methodSiguature.equals(method.signature())
+                            || toNoneGeneric(methodSiguature).equals(method.signature()))) {
                 location = method.location();
                 break;
             }
@@ -334,6 +335,28 @@ private Location findMethodLocaiton(ReferenceType refType, String methodName, St
         return location;
     }
 
+    static String toNoneGeneric(String genericSig) {
+        StringBuilder builder = new StringBuilder();
+        boolean append = true;
+        int depth = 0;
+        char[] chars = genericSig.toCharArray();
+        for (int i = 0; i < chars.length; i++) {
+            char c = chars[i];
+            if (c == '<') {
+                depth++;
+                append = (depth == 0);
+            }
+            if (append) {
+                builder.append(c);
+            }
+            if (c == '>') {
+                depth--;
+                append = (depth == 0);
+            }
+        }
+        return builder.toString();
+    }
+
     private List<Location> findLocaitonsOfLine(Method method, int lineNumber) {
         try {
             return method.locationsOfLine(lineNumber);

com.microsoft.java.debug.core/src/test/java/com/microsoft/java/debug/core/BreakpointTest.java

@@ -0,0 +1,17 @@
+package com.microsoft.java.debug.core;
+
+import static org.junit.Assert.assertEquals;
+
+import org.junit.Test;
+
+public class BreakpointTest {
+    @Test
+    public void testToNoneGeneric() {
+        assertEquals("Ljava.util.List;", Breakpoint.toNoneGeneric("Ljava.util.List<java.lang.String;>;"));
+        assertEquals("(Ljava/util/Map;)Ljava/util/Map;", Breakpoint.toNoneGeneric(
+                "(Ljava/util/Map<Ljava/lang/String;Ljava/util/List<Ljava/lang/Integer;>;>;)Ljava/util/Map<Ljava/lang/String;Ljava/util/List<Ljava/lang/Integer;>;>;"));
+        assertEquals("(Ljava/util/Map;)Ljava/util/Map;",
+                Breakpoint.toNoneGeneric(
+                        "(Ljava/util/Map<Ljava/util/List<Ljava/lang/Integer;>;Ljava/util/List<Ljava/lang/Integer;>;>;)Ljava/util/Map<Ljava/util/List<Ljava/lang/Integer;>;Ljava/util/List<Ljava/lang/Integer;>;>;"));
+    }
+}