Skip to content

Commit 9330c78

Browse files
guy120494guy120494
committed
Modified tests
1 parent fc8912b commit 9330c78

File tree

10 files changed

+85
-95
lines changed

10 files changed

+85
-95
lines changed

‎src/test/java/graphql/annotations/GraphQLDataFetcherTest.java‎

Lines changed: 7 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -20,20 +20,14 @@
2020
import graphql.annotations.annotationTypes.GraphQLField;
2121
import graphql.annotations.annotationTypes.GraphQLName;
2222
import graphql.annotations.processor.GraphQLAnnotations;
23-
import graphql.schema.DataFetcher;
24-
import graphql.schema.DataFetchingEnvironment;
25-
import graphql.schema.GraphQLObjectType;
26-
import graphql.schema.GraphQLSchema;
27-
import graphql.schema.PropertyDataFetcher;
23+
import graphql.schema.*;
2824
import org.testng.annotations.BeforeMethod;
2925
import org.testng.annotations.Test;
3026

3127
import java.util.HashMap;
3228

3329
import static graphql.schema.GraphQLSchema.newSchema;
34-
import static org.testng.Assert.assertFalse;
35-
import static org.testng.Assert.assertNotNull;
36-
import static org.testng.Assert.assertTrue;
30+
import static org.testng.Assert.*;
3731

3832
public class GraphQLDataFetcherTest {
3933

@@ -53,7 +47,7 @@ public void shouldUsePreferredConstructor() {
5347
final ExecutionResult result = graphql.execute("{sample {isGreat isBad}}");
5448

5549
// Then
56-
final HashMap<String, Object> data = (HashMap) result.getData();
50+
final HashMap<String, Object> data = result.getData();
5751
assertNotNull(data);
5852
assertTrue(((HashMap<String, Boolean>) data.get("sample")).get("isGreat"));
5953
assertTrue(((HashMap<String, Boolean>) data.get("sample")).get("isBad"));
@@ -70,7 +64,7 @@ public void shouldUseProvidedSoloArgumentForDataFetcherDeclaredInMethod() {
7064
final ExecutionResult result = graphql.execute("{great}");
7165

7266
// Then
73-
final HashMap<String, Object> data = (HashMap) result.getData();
67+
final HashMap<String, Object> data = result.getData();
7468
assertNotNull(data);
7569
assertFalse((Boolean)data.get("great"));
7670
}
@@ -83,12 +77,12 @@ public void shouldUseTargetAndArgumentsForDataFetcherDeclaredInMethod() {
8377
final GraphQL graphql = GraphQL.newGraphQL(schema).build();
8478

8579
// When
86-
final ExecutionResult result = graphql.execute("{sample {bad}}");
80+
final ExecutionResult result = graphql.execute("{sample {isBad}}");
8781

8882
// Then
89-
final HashMap<String, Object> data = (HashMap) result.getData();
83+
final HashMap<String, Object> data = result.getData();
9084
assertNotNull(data);
91-
assertTrue(((HashMap<String,Boolean>)data.get("sample")).get("bad"));
85+
assertTrue(((HashMap<String,Boolean>)data.get("sample")).get("isBad"));
9286
}
9387

9488
@GraphQLName("Query")

‎src/test/java/graphql/annotations/GraphQLEnumTest.java‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
import graphql.annotations.annotationTypes.GraphQLField;
2020
import graphql.annotations.annotationTypes.GraphQLName;
2121
import graphql.annotations.processor.GraphQLAnnotations;
22-
import graphql.schema.*;
22+
import graphql.schema.GraphQLObjectType;
2323
import org.testng.annotations.BeforeMethod;
2424
import org.testng.annotations.Test;
2525

@@ -72,17 +72,17 @@ public void test() throws IllegalAccessException, NoSuchMethodException, Instant
7272
GraphQLObjectType queryObject = GraphQLAnnotations.object(Query.class);
7373
GraphQL graphql = GraphQL.newGraphQL(newSchema().query(queryObject).build()).build();
7474

75-
ExecutionResult result = graphql.execute("{ defaultUser{ name } }");
76-
assertEquals(result.getData().toString(), "{defaultUser={name=ONE}}");
75+
ExecutionResult result = graphql.execute("{ defaultUser{ getName } }");
76+
assertEquals(result.getData().toString(), "{defaultUser={getName=ONE}}");
7777
}
7878

7979
@Test
8080
public void testAsInput() throws IllegalAccessException, NoSuchMethodException, InstantiationException {
8181
GraphQLObjectType queryObject = GraphQLAnnotations.object(Query.class);
8282
GraphQL graphql = GraphQL.newGraphQL(newSchema().query(queryObject).build()).build();
8383

84-
ExecutionResult result = graphql.execute("{ user(param:TWO){ name } }");
85-
assertEquals(result.getData().toString(), "{user={name=TWO}}");
84+
ExecutionResult result = graphql.execute("{ user(param:TWO){ getName } }");
85+
assertEquals(result.getData().toString(), "{user={getName=TWO}}");
8686
}
8787

8888

‎src/test/java/graphql/annotations/GraphQLExtensionsTest.java‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
import graphql.schema.*;
2424
import org.testng.annotations.Test;
2525

26+
import java.util.Comparator;
2627
import java.util.List;
2728
import java.util.Map;
2829

@@ -80,7 +81,7 @@ public TestObjectExtensionInvalid(TestObject obj) {
8081
}
8182

8283
@GraphQLField
83-
public String getField() {
84+
public String field() {
8485
return "invalid";
8586
}
8687
}
@@ -102,7 +103,7 @@ public void fields() {
102103
List<GraphQLFieldDefinition> fields = object.getFieldDefinitions();
103104
assertEquals(fields.size(), 5);
104105

105-
fields.sort((o1, o2) -> o1.getName().compareTo(o2.getName()));
106+
fields.sort(Comparator.comparing(GraphQLFieldDefinition::getName));
106107

107108
assertEquals(fields.get(0).getName(), "field");
108109
assertEquals(fields.get(1).getName(), "field2");
@@ -122,7 +123,7 @@ public void values() {
122123
GraphQLSchema schemaInherited = newSchema().query(object).build();
123124

124125
ExecutionResult result = GraphQL.newGraphQL(schema).build().execute("{field field2 field3 field4 field5}", new GraphQLExtensionsTest.TestObject());
125-
Map<String, Object> data = (Map<String, Object>) result.getData();
126+
Map<String, Object> data = result.getData();
126127
assertEquals(data.get("field"), "test");
127128
assertEquals(data.get("field2"), "test test2");
128129
assertEquals(data.get("field3"), "test test3");

‎src/test/java/graphql/annotations/GraphQLFragmentTest.java‎

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -28,12 +28,7 @@
2828
import org.testng.annotations.BeforeMethod;
2929
import org.testng.annotations.Test;
3030

31-
import java.util.Arrays;
32-
import java.util.HashMap;
33-
import java.util.HashSet;
34-
import java.util.List;
35-
import java.util.Map;
36-
import java.util.Set;
31+
import java.util.*;
3732

3833
import static org.testng.AssertJUnit.assertEquals;
3934

@@ -76,7 +71,7 @@ public void testInterfaceInlineFragment() throws Exception {
7671
GraphQL graphQL2 = GraphQL.newGraphQL(schema).build();
7772

7873
// When
79-
ExecutionResult graphQLResult = graphQL2.execute("{items { ... on MyObject {a, my {b}} ... on MyObject2 {a, b} }}", new RootObject());
74+
ExecutionResult graphQLResult = graphQL2.execute("{getItems { ... on MyObject {getA, getMy {getB}} ... on MyObject2 {getA, getB} }}", new RootObject());
8075
Set resultMap = ((Map) graphQLResult.getData()).entrySet();
8176

8277
// Then

‎src/test/java/graphql/annotations/GraphQLInputTest.java‎

Lines changed: 20 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,17 @@
2121
import graphql.annotations.annotationTypes.GraphQLName;
2222
import graphql.annotations.annotationTypes.GraphQLTypeResolver;
2323
import graphql.annotations.processor.GraphQLAnnotations;
24-
import graphql.annotations.processor.ProcessingElementsContainer;
2524
import graphql.annotations.processor.exceptions.GraphQLAnnotationsException;
26-
import graphql.schema.*;
25+
import graphql.schema.GraphQLInputObjectType;
26+
import graphql.schema.GraphQLObjectType;
27+
import graphql.schema.GraphQLSchema;
28+
import graphql.schema.TypeResolver;
2729
import org.testng.annotations.Test;
2830

29-
import java.util.*;
30-
import java.util.stream.Collectors;
31+
import java.util.Collections;
32+
import java.util.HashMap;
33+
import java.util.List;
34+
import java.util.Map;
3135

3236
import static graphql.schema.GraphQLSchema.newSchema;
3337
import static org.testng.Assert.assertEquals;
@@ -284,9 +288,9 @@ public void testInputAndOutputWithSameName() {
284288
// arrange + act
285289
GraphQLSchema schema = newSchema().query(GraphQLAnnotations.object(QueryInputAndOutput.class)).build();
286290
// assert
287-
assertEquals(schema.getQueryType().getFieldDefinition("hero").getType().getName(), "hero");
288-
assertEquals(schema.getQueryType().getFieldDefinition("string").getArgument("input").getType().getName(), "Inputhero");
289-
assertEquals(((GraphQLInputObjectType) schema.getQueryType().getFieldDefinition("string")
291+
assertEquals(schema.getQueryType().getFieldDefinition("getHero").getType().getName(), "hero");
292+
assertEquals(schema.getQueryType().getFieldDefinition("getString").getArgument("input").getType().getName(), "Inputhero");
293+
assertEquals(((GraphQLInputObjectType) schema.getQueryType().getFieldDefinition("getString")
290294
.getArgument("input").getType()).getField("skill").getType().getName(), "InputSkill");
291295
}
292296

@@ -295,8 +299,8 @@ public void testInputAndOutputSameClass() {
295299
// arrange + act
296300
GraphQLSchema schema = newSchema().query(GraphQLAnnotations.object(QueryInputAndOutput2.class)).build();
297301
// assert
298-
assertEquals(schema.getQueryType().getFieldDefinition("skill").getType().getName(), "Skill");
299-
assertEquals(schema.getQueryType().getFieldDefinition("a").getArgument("skill").getType().getName(), "InputSkill");
302+
assertEquals(schema.getQueryType().getFieldDefinition("getSkill").getType().getName(), "Skill");
303+
assertEquals(schema.getQueryType().getFieldDefinition("getA").getArgument("skill").getType().getName(), "InputSkill");
300304
}
301305

302306
@GraphQLName("A")
@@ -345,13 +349,13 @@ public void testInputAndOutputWithSameNameWithChildrenWithSameName(){
345349
// arrange + act
346350
GraphQLSchema schema = newSchema().query(GraphQLAnnotations.object(QuerySameNameWithChildren.class)).build();
347351
// assert
348-
assertEquals(schema.getQueryType().getFieldDefinition("aout").getType().getName(), "A");
349-
assertEquals(schema.getQueryType().getFieldDefinition("aout").getType().getClass(),GraphQLObjectType.class);
350-
assertEquals(schema.getQueryType().getFieldDefinition("bout").getType().getName(), "B");
351-
assertEquals(schema.getQueryType().getFieldDefinition("bout").getType().getClass(),GraphQLObjectType.class);
352-
assertEquals(schema.getQueryType().getFieldDefinition("a").getArgument("input").getType().getName(), "InputA");
353-
assertEquals(schema.getQueryType().getFieldDefinition("a").getArgument("input").getType().getClass(), GraphQLInputObjectType.class);
354-
assertEquals(schema.getQueryType().getFieldDefinition("b").getArgument("input").getType().getClass(), GraphQLInputObjectType.class);
352+
assertEquals(schema.getQueryType().getFieldDefinition("getAout").getType().getName(), "A");
353+
assertEquals(schema.getQueryType().getFieldDefinition("getAout").getType().getClass(),GraphQLObjectType.class);
354+
assertEquals(schema.getQueryType().getFieldDefinition("getBout").getType().getName(), "B");
355+
assertEquals(schema.getQueryType().getFieldDefinition("getBout").getType().getClass(),GraphQLObjectType.class);
356+
assertEquals(schema.getQueryType().getFieldDefinition("getA").getArgument("input").getType().getName(), "InputA");
357+
assertEquals(schema.getQueryType().getFieldDefinition("getA").getArgument("input").getType().getClass(), GraphQLInputObjectType.class);
358+
assertEquals(schema.getQueryType().getFieldDefinition("getB").getArgument("input").getType().getClass(), GraphQLInputObjectType.class);
355359

356360
}
357361
}

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

Lines changed: 9 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -22,14 +22,12 @@
2222
import graphql.annotations.processor.GraphQLAnnotations;
2323
import graphql.annotations.processor.ProcessingElementsContainer;
2424
import graphql.annotations.processor.retrievers.GraphQLFieldRetriever;
25-
import graphql.annotations.processor.retrievers.GraphQLObjectHandler;
2625
import graphql.annotations.processor.retrievers.GraphQLObjectInfoRetriever;
2726
import graphql.annotations.processor.searchAlgorithms.BreadthFirstSearch;
2827
import graphql.annotations.processor.searchAlgorithms.ParentalSearch;
2928
import graphql.annotations.processor.typeBuilders.InputObjectBuilder;
3029
import graphql.annotations.processor.typeFunctions.TypeFunction;
3130
import graphql.schema.*;
32-
3331
import graphql.schema.GraphQLType;
3432
import graphql.schema.idl.SchemaParser;
3533
import graphql.schema.idl.SchemaPrinter;
@@ -44,10 +42,7 @@
4442
import static graphql.Scalars.GraphQLString;
4543
import static graphql.schema.GraphQLFieldDefinition.newFieldDefinition;
4644
import static graphql.schema.GraphQLSchema.newSchema;
47-
import static org.testng.Assert.assertEquals;
48-
import static org.testng.Assert.assertNotNull;
49-
import static org.testng.Assert.assertNull;
50-
import static org.testng.Assert.assertTrue;
45+
import static org.testng.Assert.*;
5146

5247
@SuppressWarnings("unchecked")
5348
public class GraphQLObjectTest {
@@ -358,10 +353,10 @@ public void accessors() {
358353
GraphQLObjectType object = GraphQLAnnotations.object(TestAccessors.class);
359354
List<GraphQLFieldDefinition> fields = object.getFieldDefinitions();
360355
assertEquals(fields.size(), 2);
361-
fields.sort((o1, o2) -> o1.getName().compareTo(o2.getName()));
356+
fields.sort(Comparator.comparing(GraphQLFieldDefinition::getName));
362357

363-
assertEquals(fields.get(1).getName(), "value");
364-
assertEquals(fields.get(0).getName(), "anotherValue");
358+
assertEquals(fields.get(0).getName(), "getValue");
359+
assertEquals(fields.get(1).getName(), "setAnotherValue");
365360
}
366361

367362

@@ -438,7 +433,7 @@ public void onMethod() {
438433
GraphQLObjectType object = GraphQLAnnotations.object(OnMethodTest.class);
439434
List<GraphQLFieldDefinition> fields = object.getFieldDefinitions();
440435
assertEquals(fields.size(), 1);
441-
assertEquals(fields.get(0).getName(), "value");
436+
assertEquals(fields.get(0).getName(), "getValue");
442437
}
443438

444439
public static class TestFetcher implements DataFetcher {
@@ -618,7 +613,7 @@ public TestComplexInputArgument(@GraphQLName("inputs") Collection<TestInputArgum
618613
}
619614

620615
@GraphQLField
621-
public Collection<TestInputArgument> getInputs() {
616+
public Collection<TestInputArgument> inputs() {
622617
return inputs;
623618
}
624619

@@ -670,7 +665,7 @@ public void complexInputObjectArgument() {
670665
GraphQLSchema schema = newSchema().query(object).build();
671666
ExecutionResult result = GraphQL.newGraphQL(schema).build().execute("{ test2(arg: {inputs:[{ a:\"ok\", b:2 }]}, other:0) }", new TestObjectInput());
672667
assertTrue(result.getErrors().isEmpty());
673-
Map<String, Object> v = (Map<String, Object>) result.getData();
668+
Map<String, Object> v = result.getData();
674669
assertEquals(v.get("test2"), "ok");
675670
}
676671

@@ -817,12 +812,12 @@ public static class InheritGraphQLFieldTest {
817812
@GraphQLField(false)
818813
public String forcedOff;
819814

820-
public String getOn() {
815+
public String on() {
821816
return "on";
822817
}
823818

824819
@GraphQLField(false)
825-
public String getOff() {
820+
public String off() {
826821
return "off";
827822
}
828823
}

‎src/test/java/graphql/annotations/GraphQLUnionTest.java‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -106,21 +106,21 @@ public void unionQueryWithCustomTypeResolver_askForDog_getDog() {
106106
GraphQLSchema schema = newSchema().query(GraphQLAnnotations.object(Query.class)).build();
107107

108108
GraphQL graphQL = GraphQL.newGraphQL(schema).build();
109-
String query = "{ pet(kindOfPet:\"dog\"){ ... on Cat {mew}, ... on Dog{waf}} }";
109+
String query = "{ getPet(kindOfPet:\"dog\"){ ... on Cat {mew}, ... on Dog{waf}} }";
110110
ExecutionResult result = graphQL.execute(query);
111111
assertTrue(result.getErrors().isEmpty());
112-
assertEquals(((Map<String, Map<String, String>>) result.getData()).get("pet").get("waf"), "waf");
112+
assertEquals(((Map<String, Map<String, String>>) result.getData()).get("getPet").get("waf"), "waf");
113113
}
114114

115115
@Test
116116
public void unionQueryWithCustomTypeResolver_askForCat_getCat() {
117117
GraphQLSchema schema = newSchema().query(GraphQLAnnotations.object(Query.class)).build();
118118

119119
GraphQL graphQL = GraphQL.newGraphQL(schema).build();
120-
String query = "{ pet(kindOfPet:\"cat\"){ ... on Cat {mew}, ... on Dog{waf}} }";
120+
String query = "{ getPet(kindOfPet:\"cat\"){ ... on Cat {mew}, ... on Dog{waf}} }";
121121
ExecutionResult result = graphQL.execute(query);
122122
assertTrue(result.getErrors().isEmpty());
123-
assertEquals(((Map<String, Map<String, String>>) result.getData()).get("pet").get("mew"), "mew");
123+
assertEquals(((Map<String, Map<String, String>>) result.getData()).get("getPet").get("mew"), "mew");
124124
}
125125

126126
static class Screen implements Hardware {

0 commit comments

Comments
 (0)