Skip to content

Commit 1ef0496

Browse files
committed
added tests that were removed in previous commit.
added FieldDataFetcher where needed
1 parent 5742dcf commit 1ef0496

File tree

2 files changed

+7
-12
lines changed

2 files changed

+7
-12
lines changed

src/main/java/graphql/annotations/processor/retrievers/fieldBuilders/field/FieldDataFetcherBuilder.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ private DataFetcher handleNullCase(DataFetcher actualDataFetcher) {
8080
}
8181

8282
if (actualDataFetcher == null) {
83-
actualDataFetcher = wrapExtension(new PropertyDataFetcher(field.getName()), field);
83+
actualDataFetcher = wrapExtension(new FieldDataFetcher(field.getName()), field);
8484
}
8585
return actualDataFetcher;
8686
}

src/test/java/graphql/annotations/GraphQLObjectTest.java

Lines changed: 6 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -18,22 +18,14 @@
1818
import graphql.GraphQL;
1919
import graphql.Scalars;
2020
import graphql.annotations.annotationTypes.*;
21-
import graphql.annotations.dataFetchers.ExtensionDataFetcherWrapper;
21+
import graphql.annotations.annotationTypes.GraphQLNonNull;
2222
import graphql.annotations.processor.GraphQLAnnotations;
2323
import graphql.annotations.processor.ProcessingElementsContainer;
2424
import graphql.annotations.processor.retrievers.GraphQLInputObjectRetriever;
2525
import graphql.annotations.processor.retrievers.GraphQLObjectHandler;
2626
import graphql.annotations.processor.typeFunctions.TypeFunction;
27-
import graphql.schema.DataFetcher;
28-
import graphql.schema.DataFetchingEnvironment;
29-
30-
import graphql.schema.GraphQLArgument;
31-
import graphql.schema.GraphQLFieldDefinition;
32-
import graphql.schema.GraphQLInputObjectType;
33-
import graphql.schema.GraphQLInputType;
34-
import graphql.schema.GraphQLList;
35-
import graphql.schema.GraphQLObjectType;
36-
import graphql.schema.GraphQLSchema;
27+
import graphql.schema.*;
28+
3729
import graphql.schema.GraphQLType;
3830
import graphql.schema.idl.SchemaParser;
3931
import graphql.schema.idl.SchemaPrinter;
@@ -214,6 +206,9 @@ public void fields() {
214206
assertEquals(fields.get(5).getName(), "privateTest");
215207
assertEquals(fields.get(6).getName(), "publicTest");
216208

209+
assertEquals(fields.get(5).getDataFetcher().getClass(), PropertyDataFetcher.class);
210+
assertEquals(fields.get(6).getDataFetcher().getClass(), FieldDataFetcher.class);
211+
217212
assertEquals(fields.get(7).getName(), "z_nonOptionalString");
218213
assertTrue(fields.get(7).getType() instanceof graphql.schema.GraphQLNonNull);
219214
}

0 commit comments

Comments
 (0)