(sbg): don't include abstract super class's methods to avoid compilation issues when identical signatures are used

Author: petekanev

android-static-binding-generator/project/staticbindinggenerator/src/main/java/org/nativescript/staticbindinggenerator/Generator.java

@@ -191,37 +191,47 @@ private String getNormalizedName(String filename) {
     private Map<String, List<Method>> getPublicApi(JavaClass clazz) {
         Map<String, List<Method>> api = new HashMap<String, List<Method>>();
         JavaClass currentClass = clazz;
+        String clazzName = clazz.getClassName();
         while (true) {
             String currentClassname = currentClass.getClassName();
-            List<Method> methods = new ArrayList<Method>();
-            for (Method m : currentClass.getMethods()) {
-                methods.add(m);
-            }
 
-//            System.out.println("SBG: getPublicApi:collectInterfaceMethods classname: " + currentClassname);
+            boolean shouldCollectMethods = !(!clazzName.equals(currentClassname) && currentClass.isAbstract());
+
+            if (shouldCollectMethods || currentClass.isInterface()) {
+                // Don't include abstract parent class's methods to avoid compilation issues
+                // where a child class has 2 methods, of the same type, with just a
+                // return type/parameter type that differs by being of a superclass of the class being extended.
+                // see Test testCanCompileBindingClassExtendingAnExtendedClassWithMethodsWithTheSameSignature
+                List<Method> methods = new ArrayList<Method>();
+                for (Method m : currentClass.getMethods()) {
+                    methods.add(m);
+                }
+
+//                System.out.println("SBG: getPublicApi:collectInterfaceMethods classname: " + currentClassname);
 
-            collectInterfaceMethods(clazz, methods);
-            for (Method m : methods) {
-                if (!m.isSynthetic() && (m.isPublic() || m.isProtected()) && !m.isStatic()) {
-                    String name = m.getName();
+                collectInterfaceMethods(clazz, methods);
+                for (Method m : methods) {
+                    if (!m.isSynthetic() && (m.isPublic() || m.isProtected()) && !m.isStatic()) {
+                        String name = m.getName();
 
-                    List<Method> methodGroup;
-                    if (api.containsKey(name)) {
-                        methodGroup = api.get(name);
-                    } else {
-                        methodGroup = new ArrayList<Method>();
-                        api.put(name, methodGroup);
-                    }
-                    boolean found = false;
-                    String methodSig = m.getSignature();
-                    for (Method m1 : methodGroup) {
-                        found = methodSig.equals(m1.getSignature());
-                        if (found) {
-                            break;
+                        List<Method> methodGroup;
+                        if (api.containsKey(name)) {
+                            methodGroup = api.get(name);
+                        } else {
+                            methodGroup = new ArrayList<Method>();
+                            api.put(name, methodGroup);
+                        }
+                        boolean found = false;
+                        String methodSig = m.getSignature();
+                        for (Method m1 : methodGroup) {
+                            found = methodSig.equals(m1.getSignature());
+                            if (found) {
+                                break;
+                            }
+                        }
+                        if (!found) {
+                            methodGroup.add(m);
                         }
-                    }
-                    if (!found) {
-                        methodGroup.add(m);
                     }
                 }
             }
@@ -650,12 +660,6 @@ private void collectInterfaceMethods(JavaClass clazz, List<Method> methods) {
         JavaClass currentClass = clazz;
 
         while (true) {
-            if (currentClass == null) {
-                System.out.println("Contains android.support.v7.widget.RecyclerView$Adapter: " + classes.keySet().contains("android.support.v7.widget.RecyclerView$Adapter"));
-
-                System.out.println("Contains android.support.v7.widget.RecyclerView.Adapter: " + classes.keySet().contains("android.support.v7.widget.RecyclerView.Adapter"));
-            }
-
             String currentClassname = currentClass.getClassName();
 
             Queue<String> queue = new ArrayDeque<String>();

android-static-binding-generator/project/staticbindinggenerator/src/test/java/com/example/ListView.java

@@ -0,0 +1,11 @@
+package com.example;
+
+/**
+ * Created by pkanev on 4/27/2017.
+ */
+
+public class ListView extends View {
+    public ListView createView() {
+        return new ListView();
+    }
+}

android-static-binding-generator/project/staticbindinggenerator/src/test/java/com/example/View.java

@@ -0,0 +1,11 @@
+package com.example;
+
+/**
+ * Created by pkanev on 4/27/2017.
+ */
+
+public abstract class View {
+    public View createView() {
+        return null;
+    }
+}

android-static-binding-generator/project/staticbindinggenerator/src/test/java/org/nativescript/staticbindinggenerator/test/GeneratorTest.java

@@ -1,5 +1,6 @@
 package org.nativescript.staticbindinggenerator.test;
 
+import com.example.ListView;
 import com.example.MyInterface;
 
 import org.apache.commons.io.IOUtils;
@@ -103,4 +104,29 @@ public void testCanCompileBindingClassImplementingMultipleInterfaces() throws Ex
         Assert.assertNotNull(ComplexClass);
         Assert.assertEquals(5, ComplexClass.getInterfaces().length); // 4 + 1 (hashcodeprovider)
     }
+
+    @Test
+    public void testCanCompileBindingClassExtendingAnExtendedClassWithMethodsWithTheSameSignature() throws Exception {
+        URL u = ListView.class.getResource('/' + ListView.class.getName().replace('.', '/') + ".class");
+        File f = new File(u.toURI()).getParentFile().getParentFile().getParentFile();
+
+        String dataRowString = "com.example.ListView*_fapp_l9_c29__*createView*com.example.MyListView**";
+        DataRow dataRow = new DataRow(dataRowString);
+
+        System.out.println(dataRowString);
+
+        String outputDir = null;
+        String[] libs = {runtimePath, f.getAbsolutePath()};
+        Generator generator = new Generator(outputDir, libs);
+        Binding binding = generator.generateBinding(dataRow);
+
+        StringBuffer sourceCode = new StringBuffer();
+        sourceCode.append(binding.getContent());
+
+        Iterable<String> options = new ArrayList<String>(Arrays.asList("-cp", dependenciesDir));
+        Class<?> ComplexClass = InMemoryJavaCompiler.compile(binding.getClassname(), sourceCode.toString(), options);
+
+        Assert.assertNotNull(ComplexClass);
+        Assert.assertEquals(4, ComplexClass.getDeclaredMethods().length); // 1 + constructor + (equals + hashcode)
+    }
 }

android-static-binding-generator/project/staticbindinggenerator/src/test/resources/org/nativescript/staticbindinggenerator/test/datarow-extend-class-with-same-method-signature.txt

@@ -0,0 +1 @@
+"com.example.ListView*_fapp_l9_c29__*createView*com.example.MyListView**"