Check if new connections before calling method on it

Author: rmosolgo

lib/graphql/schema/field/connection_extension.rb

@@ -44,7 +44,7 @@ def after_resolve(value:, object:, arguments:, context:, memo:)
               if field.has_max_page_size? && !value.has_max_page_size_override?
                 value.max_page_size = field.max_page_size
               end
-              if (custom_t = context.schema.connections.edge_class_for_field(@field))
+              if context.schema.new_connections? && (custom_t = context.schema.connections.edge_class_for_field(@field))
                 value.edge_class = custom_t
               end
               value

spec/graphql/pagination/connections_spec.rb

@@ -112,4 +112,23 @@ def things2
 
     assert_includes err.message, "undefined method `no_such_method' for <BadThing!>"
   end
+
+  class SingleNewConnectionSchema < GraphQL::Schema
+    class Query < GraphQL::Schema::Object
+      field :strings, GraphQL::Types::String.connection_type, null: false
+
+      def strings
+        GraphQL::Pagination::ArrayConnection.new(["a", "b", "c"])
+      end
+    end
+
+    query(Query)
+    use GraphQL::Execution::Interpreter
+    use GraphQL::Analysis::AST
+  end
+
+  it "works when new connections are not installed" do
+    res = SingleNewConnectionSchema.execute("{ strings(first: 2) { edges { node } } }")
+    assert_equal ["a", "b"], res["data"]["strings"]["edges"].map { |e| e["node"] }
+  end
 end