Fix some minor code style issues (reported by checkstyle)

Author: kobylynskyi

src/main/java/com/kobylynskyi/graphql/codegen/generators/impl/FieldResolversGenerator.java

@@ -53,7 +53,8 @@ private List<File> generate(List<ExtendedFieldDefinition> fieldDefinitions,
         }
         Set<String> fieldNamesWithResolvers = mappingContext.getFieldNamesWithResolvers();
         List<ExtendedFieldDefinition> fieldDefsWithResolvers = fieldDefinitions.stream()
-                .filter(fieldDef -> fieldNamesWithResolvers.contains(parentDefinition.getName() + "." + fieldDef.getName()))
+                .filter(fieldDef ->
+                        fieldNamesWithResolvers.contains(parentDefinition.getName() + "." + fieldDef.getName()))
                 .collect(toList());
 
         List<File> generatedFiles = new ArrayList<>();

src/main/java/com/kobylynskyi/graphql/codegen/model/MappingConfig.java

@@ -184,7 +184,8 @@ public void combine(MappingConfig source) {
         customAnnotationsMapping = combineMap(customAnnotationsMapping, source.customAnnotationsMapping);
         directiveAnnotationsMapping = combineMap(directiveAnnotationsMapping, source.directiveAnnotationsMapping);
         resolverArgumentAnnotations = combineSet(resolverArgumentAnnotations, source.resolverArgumentAnnotations);
-        parametrizedResolverAnnotations = combineSet(parametrizedResolverAnnotations, source.parametrizedResolverAnnotations);
+        parametrizedResolverAnnotations = combineSet(parametrizedResolverAnnotations,
+                source.parametrizedResolverAnnotations);
         generateAllMethodInProjection = getValueOrDefaultToThis(source,
                 GraphQLCodegenConfiguration::getGenerateAllMethodInProjection);
         responseProjectionMaxDepth = getValueOrDefaultToThis(source,

src/main/java/com/kobylynskyi/graphql/codegen/model/MappingContext.java

@@ -400,20 +400,28 @@ public Map<String, Set<String>> getParentInterfaceProperties() {
         return parentInterfaceProperties;
     }
 
+    /**
+     * Get names of all the fields that require separate resolver class.
+     * This includes fields from type definitions and interface definitions.
+     *
+     * @return a flat set of all field names that require resolver.
+     */
     public Set<String> getFieldNamesWithResolvers() {
         if (fieldNamesWithResolvers == null) {
             fieldNamesWithResolvers = new HashSet<>();
             for (ExtendedObjectTypeDefinition definition : document.getTypeDefinitions()) {
                 definition.getFieldDefinitions().stream()
                         .filter(fieldDef -> FieldDefinitionToParameterMapper.generateResolversForField(
                                 this, fieldDef, definition))
-                        .forEach(fieldDef -> fieldNamesWithResolvers.add(definition.getName() + "." + fieldDef.getName()));
+                        .forEach(fieldDef ->
+                                fieldNamesWithResolvers.add(definition.getName() + "." + fieldDef.getName()));
             }
             for (ExtendedInterfaceTypeDefinition definition : document.getInterfaceDefinitions()) {
                 definition.getFieldDefinitions().stream()
                         .filter(fieldDef -> FieldDefinitionToParameterMapper.generateResolversForField(
                                 this, fieldDef, definition))
-                        .forEach(fieldDef -> fieldNamesWithResolvers.add(definition.getName() + "." + fieldDef.getName()));
+                        .forEach(fieldDef ->
+                                fieldNamesWithResolvers.add(definition.getName() + "." + fieldDef.getName()));
             }
         }
         return fieldNamesWithResolvers;

src/test/java/com/kobylynskyi/graphql/codegen/GraphQLCodegenAnnotationsTest.java

@@ -213,8 +213,8 @@ void generate_ResolverArgumentAnnotations() throws Exception {
                 outputBuildDir, mappingConfig, TestUtils.getStaticGeneratedInfo()).generate();
 
         File[] files = Objects.requireNonNull(outputJavaClassesDir.listFiles());
-        assertSameTrimmedContent(
-                new File("src/test/resources/expected-classes/annotation/CreateEventMutationResolver_ArgumentAnnotations.java.txt"),
+        assertSameTrimmedContent(new File("src/test/resources/expected-classes/annotation/" +
+                        "CreateEventMutationResolver_ArgumentAnnotations.java.txt"),
                 getFileByName(files, "CreateEventMutationResolver.java"));
         assertSameTrimmedContent(
                 new File("src/test/resources/expected-classes/annotation/QueryResolver_ArgumentAnnotations.java.txt"),
@@ -232,8 +232,8 @@ void generate_ParametrizedResolverAnnotations() throws Exception {
                 outputBuildDir, mappingConfig, TestUtils.getStaticGeneratedInfo()).generate();
 
         File[] files = Objects.requireNonNull(outputJavaClassesDir.listFiles());
-        assertSameTrimmedContent(
-                new File("src/test/resources/expected-classes/annotation/EventPropertyResolver_ParametrizedResolverAnnotations.java.txt"),
+        assertSameTrimmedContent(new File("src/test/resources/expected-classes/annotation/" +
+                        "EventPropertyResolver_ParametrizedResolverAnnotations.java.txt"),
                 getFileByName(files, "EventPropertyResolver.java"));
     }
 

src/test/java/com/kobylynskyi/graphql/codegen/model/MappingConfigTest.java

@@ -168,8 +168,10 @@ private static void compareMappingConfigs(MappingConfig mappingConfig, MappingCo
                 mappingConfig.getUseOptionalForNullableReturnTypes());
         assertEquals(expectedMappingConfig.getRelayConfig(), mappingConfig.getRelayConfig());
         assertEquals(expectedMappingConfig.getTypesAsInterfaces(), mappingConfig.getTypesAsInterfaces());
-        assertEquals(expectedMappingConfig.getResolverArgumentAnnotations(), mappingConfig.getResolverArgumentAnnotations());
-        assertEquals(expectedMappingConfig.getParametrizedResolverAnnotations(), mappingConfig.getParametrizedResolverAnnotations());
+        assertEquals(expectedMappingConfig.getResolverArgumentAnnotations(),
+                mappingConfig.getResolverArgumentAnnotations());
+        assertEquals(expectedMappingConfig.getParametrizedResolverAnnotations(),
+                mappingConfig.getParametrizedResolverAnnotations());
     }
 
     @Test
@@ -327,7 +329,8 @@ void combineCustomWithCustom() {
         assertEquals("for", mappingConfig.getRelayConfig().getDirectiveArgumentName());
         assertEquals(new HashSet<>(Arrays.asList("User", "User2")), mappingConfig.getTypesAsInterfaces());
         assertEquals(new HashSet<>(Arrays.asList("Ann1", "Ann2")), mappingConfig.getResolverArgumentAnnotations());
-        assertEquals(new HashSet<>(Arrays.asList("PAnn1", "PAnn2")), mappingConfig.getParametrizedResolverAnnotations());
+        assertEquals(new HashSet<>(Arrays.asList("PAnn1", "PAnn2")),
+                mappingConfig.getParametrizedResolverAnnotations());
     }
 
 }