Fixed a problem where during the scanning of InputTypes it included s…

…ynthetic methods, added a filter + test

common/schema-builder/src/main/java/io/smallrye/graphql/schema/creator/type/InputTypeCreator.java

@@ -153,7 +153,7 @@ private void addFields(InputType inputType, ClassInfo classInfo, Reference refer
         // Find all methods and properties up the tree
         for (ClassInfo c = classInfo; c != null; c = ScanningContext.getIndex().getClassByName(c.superName())) {
             if (!c.toString().startsWith(JAVA_DOT)) { // Not java objects
-                allMethods.addAll(c.methods());
+                c.methods().stream().filter(methodInfo -> !methodInfo.isSynthetic()).forEach(allMethods::add);
                 for (final FieldInfo fieldInfo : c.fields()) {
                     allFields.putIfAbsent(fieldInfo.name(), fieldInfo);
                 }

common/schema-builder/src/test/java/io/smallrye/graphql/index/SchemaBuilderTest.java

@@ -199,8 +199,8 @@ public void testGenericSchemaBuilding() {
         Set<Operation> mutations = schema.getMutations();
         Map<String, Type> outputTypes = schema.getTypes();
 
-        assertEquals(queries.size(), 2);
-        assertEquals(mutations.size(), 4);
+        assertEquals(queries.size(), 3);
+        assertEquals(mutations.size(), 5);
 
         Operation firstQuery = queries.stream()
                 .filter(q -> q.getName().equals("heroes"))
@@ -221,6 +221,12 @@ public void testGenericSchemaBuilding() {
         Type greetingType = outputTypes.get("Greet");
         assertNotNull(greetingType);
 
+        Operation thirdQuery = queries.stream()
+                .filter(q -> q.getName().equals("saySome")).findFirst().orElseThrow(AssertionError::new);
+        assertEquals(thirdQuery.getArguments().size(), 1);
+        assertEquals(thirdQuery.getArguments().get(0).getReference().getName(), "SomeInput");
+        assertEquals(thirdQuery.getReference().getName(), "Some");
+
         // ------------------------------------------------------------------
         // MUTATIONS
         Operation firstMutation = mutations.stream()
@@ -281,6 +287,17 @@ public void testGenericSchemaBuilding() {
         // return type
         assertEquals(fourthMutation.getReference().getName(), "Hero");
 
+        Operation fifthMutation = mutations.stream()
+                .filter(q -> q.getName().equals("updateSome"))
+                .findFirst()
+                .orElseThrow(AssertionError::new);
+
+        // arguments
+        assertEquals(fifthMutation.getArguments().size(), 1);
+        assertEquals(fifthMutation.getArguments().get(0).getReference().getName(), "SomeInput");
+        // return type
+        assertEquals(fifthMutation.getReference().getName(), "Some");
+
     }
 
     @Test

common/schema-builder/src/test/java/io/smallrye/graphql/index/generic/Attribute.java

@@ -0,0 +1,7 @@
+package io.smallrye.graphql.index.generic;
+
+public interface Attribute<T> {
+    T getValue();
+
+    void setValue(T value);
+}

common/schema-builder/src/test/java/io/smallrye/graphql/index/generic/HeroResource.java

@@ -48,4 +48,13 @@ public ResponseComposite sayHello(String name) {
         return null;
     }
 
+    @Query
+    public Some saySome(Some value) {
+        return null;
+    }
+
+    @Mutation
+    public Some updateSome(Some some) {
+        return null;
+    }
 }

common/schema-builder/src/test/java/io/smallrye/graphql/index/generic/Some.java

@@ -0,0 +1,22 @@
+package io.smallrye.graphql.index.generic;
+
+public class Some implements Attribute<Long> {
+    Long some;
+
+    public Some() {
+    }
+
+    public Some(Long some) {
+        this.some = some;
+    }
+
+    @Override
+    public Long getValue() {
+        return some;
+    }
+
+    @Override
+    public void setValue(Long value) {
+        this.some = value;
+    }
+}