fix: support new graphql context

fixes #393

Author: oliemansm

build.gradle

@@ -92,9 +92,9 @@ subprojects {
 
     jacocoTestReport {
         reports {
-            xml.enabled = true
-            html.enabled = false
-            csv.enabled = false
+            xml.required = true
+            html.required = false
+            csv.required = false
         }
     }
 

gradle.properties

@@ -1,4 +1,4 @@
-version=12.0.1-SNAPSHOT
+version=13.0.0-SNAPSHOT
 group=com.graphql-java-kickstart
 PROJECT_NAME=graphql-java-servlet
 PROJECT_DESC=GraphQL Java Kickstart

graphql-java-kickstart/src/main/java/graphql/kickstart/execution/GraphQLInvokerSubjectProxy.java

@@ -47,7 +47,7 @@ public enum ContextSetting {
   public GraphQLBatchedInvocationInput getBatch(
       List<GraphQLRequest> requests,
       GraphQLSchema schema,
-      Supplier<GraphQLContext> contextSupplier,
+      Supplier<GraphQLKickstartContext> contextSupplier,
       Object root) {
     switch (this) {
       case PER_QUERY_WITH_INSTRUMENTATION:

graphql-java-kickstart/src/main/java/graphql/kickstart/execution/context/ContextSetting.java

@@ -1,37 +1,36 @@
 package graphql.kickstart.execution.context;
 
+import java.util.HashMap;
+import java.util.Map;
 import java.util.Objects;
-import java.util.Optional;
-import javax.security.auth.Subject;
 import org.dataloader.DataLoaderRegistry;
 
 /**
  * An object for the DefaultGraphQLContextBuilder to return. Can be extended to include more
  * context.
  */
-public class DefaultGraphQLContext implements GraphQLContext {
-
-  private final Subject subject;
+public class DefaultGraphQLContext implements GraphQLKickstartContext {
 
   private final DataLoaderRegistry dataLoaderRegistry;
 
-  public DefaultGraphQLContext(DataLoaderRegistry dataLoaderRegistry, Subject subject) {
+  public DefaultGraphQLContext(DataLoaderRegistry dataLoaderRegistry) {
     this.dataLoaderRegistry =
         Objects.requireNonNull(dataLoaderRegistry, "dataLoaderRegistry is required");
-    this.subject = subject;
   }
 
   public DefaultGraphQLContext() {
-    this(new DataLoaderRegistry(), null);
+    this(new DataLoaderRegistry());
   }
 
   @Override
-  public Optional<Subject> getSubject() {
-    return Optional.ofNullable(subject);
+  public DataLoaderRegistry getDataLoaderRegistry() {
+    return dataLoaderRegistry;
   }
 
   @Override
-  public DataLoaderRegistry getDataLoaderRegistry() {
-    return dataLoaderRegistry;
+  public Map<Object, Object> getMapOfContext() {
+    Map<Object, Object> map = new HashMap<>();
+    map.put(DataLoaderRegistry.class, dataLoaderRegistry);
+    return map;
   }
 }

graphql-java-kickstart/src/main/java/graphql/kickstart/execution/context/DefaultGraphQLContext.java

@@ -4,7 +4,7 @@
 public class DefaultGraphQLContextBuilder implements GraphQLContextBuilder {
 
   @Override
-  public GraphQLContext build() {
+  public GraphQLKickstartContext build() {
     return new DefaultGraphQLContext();
   }
 }

graphql-java-kickstart/src/main/java/graphql/kickstart/execution/context/DefaultGraphQLContextBuilder.java

@@ -3,5 +3,5 @@
 public interface GraphQLContextBuilder {
 
   /** @return the graphql context */
-  GraphQLContext build();
+  GraphQLKickstartContext build();
 }

graphql-java-kickstart/src/main/java/graphql/kickstart/execution/context/GraphQLContextBuilder.java

@@ -1,17 +1,15 @@
 package graphql.kickstart.execution.context;
 
-import java.util.Optional;
-import javax.security.auth.Subject;
+import java.util.Map;
 import lombok.NonNull;
 import org.dataloader.DataLoaderRegistry;
 
 /** Represents the context required by the servlet to execute a GraphQL request. */
-public interface GraphQLContext {
-
-  /** @return the subject to execute the query as. */
-  Optional<Subject> getSubject();
+public interface GraphQLKickstartContext {
 
   /** @return the Dataloader registry to use for the execution. Must not return <code>null</code> */
   @NonNull
   DataLoaderRegistry getDataLoaderRegistry();
+
+  Map<Object, Object> getMapOfContext();
 }

graphql-java-kickstart/src/main/java/graphql/kickstart/execution/context/GraphQLContext.java renamed to graphql-java-kickstart/src/main/java/graphql/kickstart/execution/context/GraphQLKickstartContext.java

@@ -5,12 +5,9 @@
 import graphql.ExecutionInput;
 import graphql.execution.ExecutionId;
 import graphql.kickstart.execution.GraphQLRequest;
-import graphql.kickstart.execution.context.GraphQLContext;
+import graphql.kickstart.execution.context.GraphQLKickstartContext;
 import graphql.schema.GraphQLSchema;
-import java.util.Collections;
 import java.util.List;
-import java.util.Optional;
-import javax.security.auth.Subject;
 
 /** Represents a single GraphQL execution. */
 public class GraphQLSingleInvocationInput implements GraphQLInvocationInput {
@@ -19,32 +16,24 @@ public class GraphQLSingleInvocationInput implements GraphQLInvocationInput {
 
   private final ExecutionInput executionInput;
 
-  private final Subject subject;
-
   public GraphQLSingleInvocationInput(
-      GraphQLRequest request, GraphQLSchema schema, GraphQLContext context, Object root) {
+      GraphQLRequest request, GraphQLSchema schema, GraphQLKickstartContext context, Object root) {
     this.schema = schema;
     this.executionInput = createExecutionInput(request, context, root);
-    subject = context.getSubject().orElse(null);
   }
 
   /** @return the schema to use to execute this query. */
   public GraphQLSchema getSchema() {
     return schema;
   }
 
-  /** @return a subject to execute the query as. */
-  public Optional<Subject> getSubject() {
-    return Optional.ofNullable(subject);
-  }
-
   private ExecutionInput createExecutionInput(
-      GraphQLRequest graphQLRequest, GraphQLContext context, Object root) {
+      GraphQLRequest graphQLRequest, GraphQLKickstartContext context, Object root) {
     return ExecutionInput.newExecutionInput()
         .query(graphQLRequest.getQuery())
         .operationName(graphQLRequest.getOperationName())
         .context(context)
-        .graphQLContext(Collections.singletonMap(context.getClass(), context))
+        .graphQLContext(context.getMapOfContext())
         .root(root)
         .variables(graphQLRequest.getVariables())
         .extensions(graphQLRequest.getExtensions())
@@ -61,5 +50,4 @@ public ExecutionInput getExecutionInput() {
   public List<String> getQueries() {
     return singletonList(executionInput.getQuery());
   }
-
 }

graphql-java-kickstart/src/main/java/graphql/kickstart/execution/input/GraphQLSingleInvocationInput.java

@@ -4,7 +4,7 @@
 
 import graphql.kickstart.execution.GraphQLRequest;
 import graphql.kickstart.execution.context.ContextSetting;
-import graphql.kickstart.execution.context.GraphQLContext;
+import graphql.kickstart.execution.context.GraphQLKickstartContext;
 import graphql.schema.GraphQLSchema;
 import java.util.List;
 import java.util.function.Supplier;
@@ -21,7 +21,7 @@ public class PerQueryBatchedInvocationInput implements GraphQLBatchedInvocationI
   public PerQueryBatchedInvocationInput(
       List<GraphQLRequest> requests,
       GraphQLSchema schema,
-      Supplier<GraphQLContext> contextSupplier,
+      Supplier<GraphQLKickstartContext> contextSupplier,
       Object root,
       ContextSetting contextSetting) {
     invocationInputs =