Remove DepthExecutor prototype

README.md

@@ -2,7 +2,7 @@
 
 **An (experimental) breadth-first GraphQL executor for Ruby**
 
-Depth-first execution resolves every object field descending down a response tree, while breadth-first visits every _selection position_ once with an aggregated set of objects. The breadth-first approach tends to be much faster due to fewer resolver calls and intermediary promises.
+Depth-first execution resolves every object field descending down a response tree, while breadth-first visits every _selection position_ once with an aggregated set of objects. The breadth-first approach is much faster due to fewer resolver calls and intermediary promises.
 
 ```shell
 graphql-ruby: 140002 resolvers
@@ -43,5 +43,5 @@ Breadth-first then runs a single resolver per document selection, and coalesces
 
 While bigger responses will always take longer to process, the workload is your own business logic with very little GraphQL execution overhead. Other advantages:
 
-* Eliminates the need for DataLoader promises, because resolvers are inherently batched.
-* Executes via flat queuing without deep recursion and huge call stacks.
+* Eliminates boilerplate need for DataLoader promises, because resolvers are inherently batched.
+* Executes via flat queuing without deep recursion and large call stacks.

lib/graphql/cardinal.rb

@@ -15,5 +15,4 @@ module Cardinal
 require_relative "cardinal/loader"
 require_relative "cardinal/field_resolvers"
 require_relative "cardinal/executor"
-require_relative "cardinal/depth_executor"
 require_relative "cardinal/version"

lib/graphql/cardinal/executor.rb

@@ -294,7 +294,5 @@ def report_exception(message, path: @path.dup)
         # todo: hook up some kind of error reporting...
       end
     end
-
-    class BreadthExecutor < Executor; end
   end
 end

test/graphql/cardinal/executor/loaders_test.rb

@@ -95,7 +95,7 @@ def test_splits_loaders_by_group_across_fields
       }
     }
 
-    executor = GraphQL::Cardinal::BreadthExecutor.new(LOADER_SCHEMA, LOADER_RESOLVERS, document, source)
+    executor = GraphQL::Cardinal::Executor.new(LOADER_SCHEMA, LOADER_RESOLVERS, document, source)
     assert_equal expected, executor.perform
     assert_equal [["Apple", "Banana"], ["Coconut"]], FancyLoader.perform_keys
   end
@@ -129,7 +129,7 @@ def test_maintains_ordered_selections_around_object_fields
       }
     }
 
-    executor = GraphQL::Cardinal::BreadthExecutor.new(LOADER_SCHEMA, LOADER_RESOLVERS, document, source)
+    executor = GraphQL::Cardinal::Executor.new(LOADER_SCHEMA, LOADER_RESOLVERS, document, source)
     result = executor.perform
     assert_equal expected, result
     assert_equal result.dig("data", "widget").keys, expected.dig("data", "widget").keys
@@ -164,7 +164,7 @@ def test_maintains_ordered_selections_around_leaf_fields
       }
     }
 
-    executor = GraphQL::Cardinal::BreadthExecutor.new(LOADER_SCHEMA, LOADER_RESOLVERS, document, source)
+    executor = GraphQL::Cardinal::Executor.new(LOADER_SCHEMA, LOADER_RESOLVERS, document, source)
     result = executor.perform
     assert_equal expected, result
     assert_equal result.dig("data", "widget").keys, expected.dig("data", "widget").keys

test/test_helper.rb

@@ -19,11 +19,6 @@
 require_relative './fixtures'
 
 def breadth_exec(query, source, variables: {}, context: {})
-  executor = GraphQL::Cardinal::BreadthExecutor.new(SCHEMA, BREADTH_RESOLVERS, GraphQL.parse(query), source)
-  executor.perform
-end
-
-def depth_exec(query, source, variables: {}, context: {})
-  executor = GraphQL::Cardinal::DepthExecutor.new(SCHEMA, DEPTH_RESOLVERS, GraphQL.parse(query), source)
+  executor = GraphQL::Cardinal::Executor.new(SCHEMA, BREADTH_RESOLVERS, GraphQL.parse(query), source)
   executor.perform
 end