Merge pull request #145 from vinayknl/master

Removed duplicate ConnectionDataFetcher

Author: guy120494

src/main/java/graphql/annotations/connection/ConnectionDataFetcher.java

@@ -52,6 +52,9 @@ public Connection<T> get(DataFetchingEnvironment environment) {
     }
 
     private PageInfo getPageInfo(List<Edge<T>> edges, PaginatedData<T> paginatedData) {
+        if (edges.isEmpty()) {
+            return new DefaultPageInfo(null,null,false,false);
+        }
         ConnectionCursor firstCursor = edges.get(0).getCursor();
         ConnectionCursor lastCursor = edges.get(edges.size() - 1).getCursor();
         return new DefaultPageInfo(

src/main/java/graphql/annotations/connection/PaginatedDataConnectionFetcher.java

@@ -36,6 +36,7 @@
 
 import static graphql.annotations.processor.util.RelayKit.EMPTY_CONNECTION;
 import static graphql.schema.GraphQLSchema.newSchema;
+import static java.util.Collections.emptyList;
 import static org.testng.Assert.assertEquals;
 import static org.testng.Assert.assertTrue;
 import static org.testng.Assert.fail;
@@ -308,6 +309,22 @@ public void methodNull() {
         assertEquals(data.get("nullObj").get("edges").size(), 0);
     }
 
+    @Test
+    public void emptyListData() {
+        GraphQLObjectType object = GraphQLAnnotations.object(TestConnections.class);
+        GraphQLSchema schema = newSchema().query(object).build();
+
+        GraphQL graphQL = GraphQL.newGraphQL(schema).build();
+        ExecutionResult result = graphQL.execute("{ objStreamWithParam(first: 1, filter:\"hel\") { edges { cursor node { id, val } } } }",
+                new TestConnections(emptyList()));
+        assertTrue(result.getErrors().isEmpty());
+
+        Map<String, Map<String, List<Map<String, Map<String, Object>>>>> data = result.getData();
+        List<Map<String, Map<String, Object>>> edges = data.get("objStreamWithParam").get("edges");
+
+        assertEquals(edges.size(), 0);
+    }
+
     @Test
     public void methodListWithParam() {
         GraphQLObjectType object = GraphQLAnnotations.object(TestConnections.class);