remove not usable public constructor

andimarek · andimarek · commit fc2ec3eb0f31 · 2019-07-11T07:12:53.000+10:00
diff --git a/src/main/java/graphql/ExecutionInput.java b/src/main/java/graphql/ExecutionInput.java
@@ -26,17 +26,14 @@ public class ExecutionInput {
     private final ExecutionId executionId;
 
 
-    public ExecutionInput(String query, String operationName, Object context, Object root, Map<String, Object> variables) {
-        this(query, operationName, context, root, variables, new DataLoaderRegistry(), null, null);
-    }
-
     @Internal
     private ExecutionInput(String query, String operationName, Object context, Object root, Map<String, Object> variables, DataLoaderRegistry dataLoaderRegistry, CacheControl cacheControl, ExecutionId executionId) {
         this.query = assertNotNull(query, "query can't be null");
         this.operationName = operationName;
         this.context = context;
         this.root = root;
-        this.variables = assertNotNull(variables, "variables map can't be null");
+        this.variables = variables;
+//        this.variables = assertNotNull(variables, "variables map can't be null");
         this.dataLoaderRegistry = dataLoaderRegistry;
         this.cacheControl = cacheControl;
         this.executionId = executionId;
diff --git a/src/test/groovy/graphql/GraphQLTest.groovy b/src/test/groovy/graphql/GraphQLTest.groovy
@@ -987,19 +987,6 @@ many lines''']
         assEx.message.contains("variables map can't be null")
 
 
-    }
-
-    def "variables map can't be null via ExecutionInput constructor"() {
-        given:
-
-        when:
-        new ExecutionInput("query", "opName", null, null, null);
-
-        then:
-        def assEx = thrown(AssertException)
-        assEx.message.contains("variables map can't be null")
-
-
     }
 
     def "query can't be null via ExecutionInput"() {
@@ -1028,16 +1015,4 @@ many lines''']
 
     }
 
-    def "query can't be null via ExecutionInput constructor"() {
-        given:
-
-        when:
-        new ExecutionInput(null, "opName", null, null, null);
-
-        then:
-        def assEx = thrown(AssertException)
-        assEx.message.contains("query can't be null")
-
-
-    }
 }
