refactor(Execution) put back stuff I didn't use

Author: rmosolgo

lib/graphql/execution.rb

@@ -1,4 +1,3 @@
-require "graphql/execution/context"
 require "graphql/execution/deferred_execution"
 require "graphql/execution/selection_on_type"
 require "graphql/execution/resolve_type"

lib/graphql/execution/context.rb

@@ -1,3 +1,9 @@
+require "graphql/query/serial_execution/execution_context"
+require "graphql/query/serial_execution/value_resolution"
+require "graphql/query/serial_execution/field_resolution"
+require "graphql/query/serial_execution/operation_resolution"
+require "graphql/query/serial_execution/selection_resolution"
+
 module GraphQL
   class Query
     class SerialExecution
@@ -13,7 +19,7 @@ def execute(ast_operation, root_type, query_obj)
         operation_resolution.new(
           ast_operation,
           root_type,
-          GraphQL::Execution::Context.new(query_obj, self)
+          ExecutionContext.new(query_obj, self)
         ).result
       end
 
@@ -31,8 +37,3 @@ def selection_resolution
     end
   end
 end
-
-require "graphql/query/serial_execution/value_resolution"
-require "graphql/query/serial_execution/field_resolution"
-require "graphql/query/serial_execution/operation_resolution"
-require "graphql/query/serial_execution/selection_resolution"

lib/graphql/query/serial_execution.rb

@@ -0,0 +1,31 @@
+module GraphQL
+  class Query
+    class SerialExecution
+      class ExecutionContext
+        attr_reader :query, :schema, :strategy
+
+        def initialize(query, strategy)
+          @query = query
+          @schema = query.schema
+          @strategy = strategy
+        end
+
+        def get_type(type)
+          @schema.types[type]
+        end
+
+        def get_fragment(name)
+          @query.fragments[name]
+        end
+
+        def get_field(type, name)
+          @schema.get_field(type, name)
+        end
+
+        def add_error(err)
+          @query.context.errors << err
+        end
+      end
+    end
+  end
+end

lib/graphql/query/serial_execution/execution_context.rb

@@ -25,7 +25,6 @@
       id, flavor
     }
     |}
-
     it "the error is inserted into the errors key and the rest of the query is fulfilled" do
       expected_result = {
         "data"=>{

spec/graphql/execution_error_spec.rb

@@ -26,7 +26,6 @@
     {"name"=>"GOAT",  "isDeprecated"=> false },
     {"name"=>"SHEEP", "isDeprecated"=> false },
   ]}
-
   it "exposes metadata about types" do
     expected = {"data"=> {
       "cheeseType" => {

spec/graphql/introspection/type_type_spec.rb

@@ -1,6 +1,6 @@
 require "spec_helper"
 
-describe GraphQL::Execution::Context do
+describe GraphQL::Query::SerialExecution::ExecutionContext do
   let(:query_string) { %|
     query getFlavor($cheeseId: Int!) {
       brie: cheese(id: 1)   { ...cheeseFields, taste: flavor }
@@ -17,7 +17,7 @@
     operation_name: operation_name,
   )}
   let(:execution_context) {
-    GraphQL::Execution::Context.new(query, nil)
+    GraphQL::Query::SerialExecution::ExecutionContext.new(query, nil)
   }
 
   describe "add_error" do

spec/graphql/execution/context_spec.rb renamed to spec/graphql/query/serial_execution/execution_context_spec.rb

@@ -1,8 +1,5 @@
 require "spec_helper"
-#
-# TODO port tests to new module
-#
-#
+
 describe GraphQL::Query::SerialExecution::ValueResolution do
   let(:query_root) {
     day_of_week_enum = GraphQL::EnumType.define do