Skip to content
Snippets Groups Projects

[graphql] Convert to using the new query interpreter runtime

Merged Brett Walker requested to merge bw-graphql-interpreter into master
25 files
+ 99
95
Compare changes
  • Side-by-side
  • Inline
Files
25
  • a5dab4c9
    Update to graphql 1.13.8 and much more · a5dab4c9
    Brett Walker authored
    - Update to graphql 1.13.8
    - Override unauthorized_object(unauthorized_error)
    - Fix arg handlling in GraphQL::Helpers
    - Now some exceptions are being caught much earlier,
      breaking many unit test specs that relied on checking
      for exceptions
    - Must override == in the class itself
    - GraphQL-Ruby's types used to override def ==(other),
      but it doesn't anymore when using the interpreter
    - Procs are no longer supported
    - GraphQL::Pagination::Connections is now a default and
      is no longer needed
    - Check for object in FindClosest
    - Casues a GraphQL::Schema::DuplicateNamesError
    - Add new expect_graphql_error_to_be_created
@@ -39,7 +39,7 @@ def ready?(**args)
true
end
def load_application_object(argument, lookup_as_type, id, context)
def load_application_object(argument, id, context)
::Gitlab::Graphql::Lazy.new { super }.catch(::GraphQL::UnauthorizedError) do |e|
Gitlab::ErrorTracking.track_exception(e)
# The default behaviour is to abort processing and return nil for the
Loading