fixed a bug that could cause directives to be built with synthetic fields

Author: osher-sade

src/main/java/graphql/annotations/processor/directives/DirectiveArgumentCreator.java

@@ -35,7 +35,6 @@ public DirectiveArgumentCreator(CommonPropertiesCreator commonPropertiesCreator,
         this.container = container;
     }
 
-
     public GraphQLArgument getArgument(Field field, Class<?> containingClass) {
         GraphQLArgument.Builder builder = newArgument()
                 .name(commonPropertiesCreator.getName(field))
@@ -60,6 +59,4 @@ private GraphQLInputType getType(Field field) {
         return (GraphQLInputType) typeFunction.buildType(true, field.getType(),
                 field.getAnnotatedType(), container);
     }
-
-
 }

src/main/java/graphql/annotations/processor/directives/DirectiveCreator.java

@@ -36,8 +36,8 @@ public DirectiveCreator(DirectiveArgumentCreator directiveArgumentCreator, Commo
 
     public GraphQLDirective getDirective(Class<?> annotatedClass) {
         GraphQLDirective.Builder builder = newDirective()
-        .name(commonPropertiesCreator.getName(annotatedClass))
-        .description(commonPropertiesCreator.getDescription(annotatedClass));
+                .name(commonPropertiesCreator.getName(annotatedClass))
+                .description(commonPropertiesCreator.getDescription(annotatedClass));
         Introspection.DirectiveLocation[] validLocations = getValidLocations(annotatedClass);
         if (validLocations == null || validLocations.length == 0) {
             throw new GraphQLAnnotationsException("No valid locations defined on directive", null);
@@ -49,17 +49,18 @@ public GraphQLDirective getDirective(Class<?> annotatedClass) {
     }
 
     private void buildArguments(GraphQLDirective.Builder builder, Class<?> annotatedClass) {
-        Arrays.stream(annotatedClass.getDeclaredFields()).forEach(x ->
-                builder.argument(directiveArgumentCreator.getArgument(x, annotatedClass)));
+        Arrays.stream(annotatedClass.getDeclaredFields()).forEach(x -> {
+            if (!x.isSynthetic()) {
+                builder.argument(directiveArgumentCreator.getArgument(x, annotatedClass));
+            }
+        });
     }
 
-
     private Introspection.DirectiveLocation[] getValidLocations(Class<?> annotatedClass) {
         DirectiveLocations directiveLocationsAnnotation = annotatedClass.getAnnotation(DirectiveLocations.class);
         if (directiveLocationsAnnotation != null) {
             return directiveLocationsAnnotation.value();
         }
         return null;
     }
-
 }