Merge pull request #51 from Shopify/implement-use

Make GraphQL::Batch a plugin by adding `use`

README.md

@@ -46,14 +46,23 @@ class RecordLoader < GraphQL::Batch::Loader
 end
 ```
 
-Use lazy execution and instrumentation by `GraphQL::Batch` in your schema
+Use `GraphQL::Batch` as a plugin in your schema (for graphql >= `1.5.0`).
 
 ```ruby
 MySchema = GraphQL::Schema.define do
   query MyQueryType
 
-  lazy_resolve(Promise, :sync)
-  instrument(:query, GraphQL::Batch::Setup)
+  use GraphQL::Batch
+end
+```
+
+For pre `1.5.0` versions:
+
+```ruby
+MySchema = GraphQL::Schema.define do
+  query MyQueryType
+
+  GraphQL::Batch.use(self)
 end
 ```
 

lib/graphql/batch.rb

@@ -19,6 +19,11 @@ def self.batch
       end
     end
 
+    def self.use(schema_defn)
+      schema_defn.instrument(:query, GraphQL::Batch::Setup)
+      schema_defn.lazy_resolve(::Promise, :sync)
+    end
+
     autoload :ExecutionStrategy, 'graphql/batch/execution_strategy'
     autoload :MutationExecutionStrategy, 'graphql/batch/mutation_execution_strategy'
   end

lib/graphql/batch/execution_strategy.rb

@@ -1,4 +1,4 @@
-warn "GraphQL::Batch::ExecutionStrategy is deprecated, instead use `lazy_resolve(Promise, :sync)` and `instrument(:query, GraphQL::Batch::Setup)` in GraphQL::Schema.define"
+warn "GraphQL::Batch::ExecutionStrategy is deprecated, instead add `use GraphQL::Batch` in GraphQL::Schema.define"
 
 module GraphQL::Batch
   class ExecutionStrategy < GraphQL::Query::SerialExecution

test/support/schema.rb

@@ -152,6 +152,5 @@
   query QueryType
   mutation MutationType
 
-  lazy_resolve(Promise, :sync)
-  instrument(:query, GraphQL::Batch::Setup)
+  use GraphQL::Batch
 end