From 70babdda9800e2fa39e507b1d59507f791e9948b Mon Sep 17 00:00:00 2001 From: Evan Huus Date: Fri, 30 Apr 2021 16:02:50 +0000 Subject: [PATCH] Move error classes higher in the load order Fixes a weird require/load-order issue that showed up while I was trying to validate default values. See the explanation at https://github.com/rmosolgo/graphql-ruby/pull/3448#discussion_r623964305 --- lib/graphql.rb | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/lib/graphql.rb b/lib/graphql.rb index 4648961929..d5326baefd 100644 --- a/lib/graphql.rb +++ b/lib/graphql.rb @@ -81,10 +81,19 @@ def match?(pattern) # Order matters for these: require "graphql/execution_error" +require "graphql/runtime_type_error" +require "graphql/unresolved_type_error" +require "graphql/invalid_null_error" +require "graphql/analysis_error" +require "graphql/coercion_error" +require "graphql/invalid_name_error" +require "graphql/integer_decoding_error" +require "graphql/integer_encoding_error" +require "graphql/string_encoding_error" + require "graphql/define" require "graphql/base_type" require "graphql/object_type" - require "graphql/enum_type" require "graphql/input_object_type" require "graphql/interface_type" @@ -109,9 +118,6 @@ def match?(pattern) require "graphql/tracing" require "graphql/dig" require "graphql/execution" -require "graphql/runtime_type_error" -require "graphql/unresolved_type_error" -require "graphql/invalid_null_error" require "graphql/pagination" require "graphql/schema" require "graphql/query" @@ -133,12 +139,6 @@ def match?(pattern) require "graphql/dataloader" require "graphql/introspection" -require "graphql/analysis_error" -require "graphql/coercion_error" -require "graphql/invalid_name_error" -require "graphql/integer_decoding_error" -require "graphql/integer_encoding_error" -require "graphql/string_encoding_error" require "graphql/version" require "graphql/compatibility" require "graphql/function"