Moved UppercaseDirective class into groovy test case

Author: oliemansm

src/test/groovy/com/coxautodev/graphql/tools/RelayConnectionSpec.groovy

@@ -4,8 +4,13 @@ import graphql.GraphQL
 import graphql.execution.AsyncExecutionStrategy
 import graphql.relay.Connection
 import graphql.relay.SimpleListConnection
+import graphql.schema.DataFetcher
+import graphql.schema.DataFetcherFactories
 import graphql.schema.DataFetchingEnvironment
+import graphql.schema.GraphQLFieldDefinition
 import graphql.schema.GraphQLSchema
+import graphql.schema.idl.SchemaDirectiveWiring
+import graphql.schema.idl.SchemaDirectiveWiringEnvironment
 import spock.lang.Specification
 
 class RelayConnectionSpec extends Specification {
@@ -51,7 +56,7 @@ class RelayConnectionSpec extends Specification {
                         }
                     ''')
                     .resolvers(new QueryResolver())
-                    .directive("uppercase", new RelayConnectionTest.UppercaseDirective())
+                    .directive("uppercase", new UppercaseDirective())
                     .build()
                     .makeExecutableSchema()
             GraphQL gql = GraphQL.newGraphQL(schema)
@@ -116,5 +121,21 @@ class RelayConnectionSpec extends Specification {
         }
     }
 
+    static class UppercaseDirective implements SchemaDirectiveWiring {
+
+        @Override
+        GraphQLFieldDefinition onField(SchemaDirectiveWiringEnvironment<GraphQLFieldDefinition> env) {
+            GraphQLFieldDefinition field = env.getElement();
+            DataFetcher dataFetcher = DataFetcherFactories.wrapDataFetcher(field.getDataFetcher(), {
+                dataFetchingEnvironment, value ->
+                    if (value == null) {
+                        return null
+                    }
+                    return  ((String) value).toUpperCase()
+            })
+            return field.transform({ builder -> builder.dataFetcher(dataFetcher) });
+        }
+    }
+
 
 }

src/test/java/com/coxautodev/graphql/tools/ReactiveTest.java

@@ -0,0 +1,46 @@
+package com.coxautodev.graphql.tools;
+
+import graphql.GraphQL;
+import graphql.execution.AsyncExecutionStrategy;
+import graphql.schema.GraphQLSchema;
+import groovy.lang.Closure;
+import org.junit.Test;
+
+import java.util.HashMap;
+import java.util.concurrent.Future;
+
+public class ReactiveTest {
+
+    @Test
+    public void futureSucceeds() {
+        GraphQLSchema schema = SchemaParser.newParser().file("Reactive.graphqls")
+                .resolvers(new Query())
+                .build()
+                .makeExecutableSchema();
+
+        GraphQL gql = GraphQL.newGraphQL(schema)
+                .queryExecutionStrategy(new AsyncExecutionStrategy())
+                .build();
+        Utils.assertNoGraphQlErrors(gql, new HashMap<>(), new Object(), new Closure<String>(null) {
+            @Override
+            public String call() {
+                return "query { organization(organizationId: 1) { user { id } } }";
+            }
+        });
+    }
+
+    static class Query implements GraphQLQueryResolver {
+        Future<Organization> organization(int organizationid) {
+            return null;
+        }
+    }
+
+    static class Organization {
+        private User user;
+    }
+
+    static class User {
+        private Long id;
+        private String name;
+    }
+}

src/test/java/com/coxautodev/graphql/tools/RelayConnectionTest.java

@@ -22,7 +22,6 @@ public void compiles() {
                 .resolvers(new QueryResolver())
                 .dictionary(User.class)
                 .directive("connection", new RelayConnection())
-                .directive("uppercase", new UppercaseDirective())
                 .build()
                 .makeExecutableSchema();
     }
@@ -50,19 +49,5 @@ public GraphQLFieldDefinition onField(SchemaDirectiveWiringEnvironment<GraphQLFi
 
     }
 
-    static class UppercaseDirective implements SchemaDirectiveWiring {
 
-        @Override
-        public GraphQLFieldDefinition onField(SchemaDirectiveWiringEnvironment<GraphQLFieldDefinition> env) {
-            GraphQLFieldDefinition field = env.getElement();
-            DataFetcher dataFetcher = DataFetcherFactories.wrapDataFetcher(field.getDataFetcher(), ((dataFetchingEnvironment, value) -> {
-                if (value == null) {
-                    return null;
-                }
-                String uppercase = ((String) value).toUpperCase();
-                return uppercase;
-            }));
-            return field.transform(builder -> builder.dataFetcher(dataFetcher));
-        }
-    }
 }

src/test/resources/Reactive.graphqls

@@ -0,0 +1,12 @@
+type Query {
+    organization(organizationId: ID): Organization
+}
+
+type Organization {
+    user: User
+}
+
+type User {
+    id: ID
+    name: String
+}

src/test/resources/RelayConnection.graphqls

@@ -1,5 +1,4 @@
 directive @connection on FIELD_DEFINITION
-directive @uppercase on FIELD_DEFINITION
 
 type Query {
     users(first: Int, after: String): UserConnection @connection
@@ -17,7 +16,7 @@ type UserEdge {
 
 type User {
     id: ID!
-    name: String @uppercase
+    name: String
 }
 
 type PageInfo {