fix(Relay::Connection) properly skip connections when ctx.skip

Author: rmosolgo

lib/graphql/compatibility/execution_specification.rb

@@ -418,6 +418,16 @@ def test_it_treats_failed_type_resolution_like_nil
             assert_equal false, res.key?("errors")
             assert_equal [SpecificationSchema::BOGUS_NODE, SpecificationSchema::BOGUS_NODE], log
           end
+
+          def test_it_skips_connections
+            query_type = GraphQL::ObjectType.define do
+              name "Query"
+              connection :skipped, types[query_type], resolve: ->(o,a,c) { c.skip }
+            end
+            schema = GraphQL::Schema.define(query: query_type)
+            res = schema.execute("{ skipped { __typename } }")
+            assert_equal({"data" => nil}, res)
+          end
         end
       end
     end

lib/graphql/relay/connection_resolve.rb

@@ -27,6 +27,8 @@ def call(obj, args, ctx)
           else
             nodes
           end
+        elsif nodes.is_a?(GraphQL::Execution::Execute::Skip)
+          nodes
         else
           build_connection(nodes, args, parent, ctx)
         end