rails-graphql 1.0.0.beta → 1.0.0.rc2

data/test/assets/sqlite.gql

@@ -16,22 +16,22 @@ scalar Bigint
 The Binary scalar type represents a Base64 string.
 Normally used to share files and uploads.
 """
-scalar Binary
+scalar Binary @specifiedBy(url: "https://www.rfc-editor.org/rfc/rfc3548")
 
 "The Boolean scalar type represents true or false."
 scalar Boolean
 
 "The Date scalar type represents a ISO 8601 string value."
-scalar Date
+scalar Date @specifiedBy(url: "https://en.wikipedia.org/wiki/ISO_8601")
 
 "The DateTime scalar type represents a ISO 8601 string value."
-scalar DateTime
+scalar DateTime @specifiedBy(url: "https://en.wikipedia.org/wiki/ISO_8601")
 
 """
 The Decimal scalar type represents signed fractional values with extra precision.
 The values are exchange as string.
 """
-scalar Decimal
+scalar Decimal @specifiedBy(url: "https://en.wikipedia.org/wiki/IEEE_754-2008_revision")
 
 "The Float scalar type represents signed double-precision fractional values."
 scalar Float
@@ -47,9 +47,9 @@ scalar Int
 
 """
 The JSON scalar type represents an unstructured JSON data
-with all its available kyes and values.
+with all its available keys and values.
 """
-scalar JSON
+scalar JSON @specifiedBy(url: "https://www.rfc-editor.org/rfc/rfc8259")
 
 """
 The String scalar type represents textual data, represented as UTF-8 character
@@ -59,9 +59,9 @@ scalar String
 
 """
 The Time scalar type that represents a distance in time using hours,
-minutes, seconds, and miliseconds.
+minutes, seconds, and milliseconds.
 """
-scalar Time
+scalar Time @specifiedBy(url: "https://en.wikipedia.org/wiki/ISO_8601")
 
 "The valid locations that a directive may be placed."
 enum __DirectiveLocation {
@@ -154,25 +154,25 @@ enum __TypeKind {
 input LiteBaseInput {
   id: ID
   
+  _delete: Boolean = false
+  
   factionId: ID
   
   name: String!
   
   planet: String
-  
-  _delete: Boolean = false
 }
 
 # Assigned to LiteFaction class
 input LiteFactionInput {
   id: ID
   
-  name: String
-  
   _delete: Boolean = false
   
   basesAttributes: [LiteBaseInput!]
   
+  name: String
+  
   shipsAttributes: [LiteShipInput!]
 }
 
@@ -180,34 +180,34 @@ input LiteFactionInput {
 input LiteShipInput {
   id: ID
   
+  _delete: Boolean = false
+  
   factionId: ID
   
   name: String
-  
-  _delete: Boolean = false
 }
 
 # Assigned to LiteBase class
 type LiteBase {
   id: ID!
   
+  faction: LiteFaction
+  
   factionId: ID
   
   name: String!
   
   planet: String
-  
-  faction: LiteFaction
 }
 
 # Assigned to LiteFaction class
 type LiteFaction {
   id: ID!
   
-  name: String
-  
   bases: [LiteBase!]!
   
+  name: String
+  
   ships: [LiteShip!]!
 }
 
@@ -215,31 +215,31 @@ type LiteFaction {
 type LiteShip {
   id: ID!
   
+  faction: LiteFaction
+  
   factionId: ID
   
   name: String
-  
-  faction: LiteFaction
 }
 
 type _Mutation {
+  createLiteBase(liteBase: LiteBaseInput!): LiteBase!
+  
   createLiteFaction(liteFaction: LiteFactionInput!): LiteFaction!
   
-  updateLiteFaction(id: ID!, liteFaction: LiteFactionInput!): LiteFaction!
+  createLiteShip(liteShip: LiteShipInput!): LiteShip!
+  
+  deleteLiteBase(id: ID!): Boolean!
   
   deleteLiteFaction(id: ID!): Boolean!
   
-  createLiteBase(liteBase: LiteBaseInput!): LiteBase!
+  deleteLiteShip(id: ID!): Boolean!
   
   updateLiteBase(id: ID!, liteBase: LiteBaseInput!): LiteBase!
   
-  deleteLiteBase(id: ID!): Boolean!
-  
-  createLiteShip(liteShip: LiteShipInput!): LiteShip!
+  updateLiteFaction(id: ID!, liteFaction: LiteFactionInput!): LiteFaction!
   
   updateLiteShip(id: ID!, liteShip: LiteShipInput!): LiteShip!
-  
-  deleteLiteShip(id: ID!): Boolean!
 }
 
 type _Query {
@@ -247,17 +247,17 @@ type _Query {
   
   __type(name: String!): __Type
   
-  liteFactions(order: String): [LiteFaction!]!
-  
-  liteFaction(id: ID!): LiteFaction!
+  liteBase(id: ID!): LiteBase!
   
   liteBases(order: String = "desc"): [LiteBase!]!
   
-  liteBase(id: ID!): LiteBase!
+  liteFaction(id: ID!): LiteFaction!
   
-  liteShips: [LiteShip!]!
+  liteFactions(order: String): [LiteFaction!]!
   
   liteShip(id: ID!): LiteShip!
+  
+  liteShips: [LiteShip!]!
 }
 
 """
@@ -271,13 +271,15 @@ additional information to the executor.
 """
 # Assigned to Rails::GraphQL::Directive class
 type __Directive {
-  name: String!
+  args: [__InputValue!]!
   
   description: String
   
+  isRepeatable: Boolean!
+  
   locations: [__DirectiveLocation!]!
   
-  args: [__InputValue!]!
+  name: String!
 }
 
 """
@@ -286,13 +288,13 @@ of values. It's a string representation, not a numeric representation,
 of a value kept as all caps (ie. ONE_VALUE).
 """
 type __EnumValue {
-  name: String!
+  deprecationReason: String
   
   description: String
   
   isDeprecated: Boolean!
   
-  deprecationReason: String
+  name: String!
 }
 
 """
@@ -302,34 +304,33 @@ a value of a specific type.
 """
 # Assigned to Rails::GraphQL::Field class
 type __Field {
-  name: String!
-  
-  description: String
-  
   args: [__InputValue!]!
   
-  type: __Type!
+  deprecationReason: String
+  
+  description: String
   
   isDeprecated: Boolean!
   
-  deprecationReason: String
+  name: String!
+  
+  type: __Type!
 }
 
 """
-Alongside with scalars and enums, input value objects allow the user
-to provide values to arguments on fields and directives. Different
-from those, input values accepts a list of keyed values, instead of
-a single value.
+Arguments provided to Fields or Directives and the input fields of an
+InputObject are represented as Input Values which describe their type
+and optionally a default value.
 """
 # Assigned to Rails::GraphQL::Field::InputField class
 type __InputValue {
-  name: String!
+  defaultValue: String
   
   description: String
   
-  type: __Type!
+  name: String!
   
-  defaultValue: String
+  type: __Type!
 }
 
 """
@@ -341,15 +342,15 @@ determines the place in the type system where those operations begin.
 """
 # Assigned to Rails::GraphQL::Schema class
 type __Schema {
-  types: [__Type!]!
-  
-  queryType: __Type!
+  directives: [__Directive!]!
   
   mutationType: __Type
   
+  queryType: __Type!
+  
   subscriptionType: __Type
   
-  directives: [__Directive!]!
+  types: [__Type!]!
 }
 
 """
@@ -375,29 +376,31 @@ the schema to define exactly what data is expected.
 """
 # Assigned to Rails::GraphQL::Type class
 type __Type {
-  kind: __TypeKind!
-  
-  name: String
-  
   description: String
   
+  "ENUM only"
+  enumValues(includeDeprecated: Boolean = false): [__EnumValue!]
+  
   "OBJECT and INTERFACE only"
   fields(includeDeprecated: Boolean = false): [__Field!]
   
+  "INPUT_OBJECT only"
+  inputFields: [__InputValue!]
+  
   "OBJECT only"
   interfaces: [__Type!]
   
-  "INTERFACE and UNION only"
-  possibleTypes: [__Type!]
-  
-  "ENUM only"
-  enumValues(includeDeprecated: Boolean = false): [__EnumValue!]
+  kind: __TypeKind!
   
-  "INPUT_OBJECT only"
-  inputFields: [__InputValue!]
+  name: String
   
   "NON_NULL and LIST only"
   ofType: __Type
+  
+  "INTERFACE and UNION only"
+  possibleTypes: [__Type!]
+  
+  specifiedByURL: String
 }
 
 """
@@ -430,3 +433,14 @@ directive @skip(
   if: Boolean!
   
 ) on FIELD | FRAGMENT_SPREAD | INLINE_FRAGMENT
+
+"""
+A built-in directive used within the type system definition language to provide
+a scalar specification URL for specifying the behavior of custom scalar types.
+"""
+directive @specifiedBy(
+
+  "Point to a human-readable specification of the data format."
+  url: String!
+  
+) on SCALAR

data/test/assets/translate.gql

@@ -137,8 +137,9 @@ type ObjectDesc {
 }
 
 type _Query {
-  "Field"
-  sampleField: String
+  __schema: __Schema!
+  
+  __type(name: String!): __Type
   
   "A"
   sampleA: String
@@ -149,18 +150,14 @@ type _Query {
   "C"
   sampleC: String
   
-  "D"
   sampleD: String
   
   "E"
   sampleE: String
   
-  "F"
   sampleF: String
   
-  __schema: __Schema!
-  
-  __type(name: String!): __Type
+  sampleField: String
 }
 
 """
@@ -174,13 +171,15 @@ additional information to the executor.
 """
 # Assigned to Rails::GraphQL::Directive class
 type __Directive {
-  name: String!
+  args: [__InputValue!]!
   
   description: String
   
+  isRepeatable: Boolean!
+  
   locations: [__DirectiveLocation!]!
   
-  args: [__InputValue!]!
+  name: String!
 }
 
 """
@@ -189,13 +188,13 @@ of values. It's a string representation, not a numeric representation,
 of a value kept as all caps (ie. ONE_VALUE).
 """
 type __EnumValue {
-  name: String!
+  deprecationReason: String
   
   description: String
   
   isDeprecated: Boolean!
   
-  deprecationReason: String
+  name: String!
 }
 
 """
@@ -205,34 +204,33 @@ a value of a specific type.
 """
 # Assigned to Rails::GraphQL::Field class
 type __Field {
-  name: String!
-  
-  description: String
-  
   args: [__InputValue!]!
   
-  type: __Type!
+  deprecationReason: String
+  
+  description: String
   
   isDeprecated: Boolean!
   
-  deprecationReason: String
+  name: String!
+  
+  type: __Type!
 }
 
 """
-Alongside with scalars and enums, input value objects allow the user
-to provide values to arguments on fields and directives. Different
-from those, input values accepts a list of keyed values, instead of
-a single value.
+Arguments provided to Fields or Directives and the input fields of an
+InputObject are represented as Input Values which describe their type
+and optionally a default value.
 """
 # Assigned to Rails::GraphQL::Field::InputField class
 type __InputValue {
-  name: String!
+  defaultValue: String
   
   description: String
   
-  type: __Type!
+  name: String!
   
-  defaultValue: String
+  type: __Type!
 }
 
 """
@@ -244,15 +242,15 @@ determines the place in the type system where those operations begin.
 """
 # Assigned to Rails::GraphQL::Schema class
 type __Schema {
-  types: [__Type!]!
-  
-  queryType: __Type!
+  directives: [__Directive!]!
   
   mutationType: __Type
   
+  queryType: __Type!
+  
   subscriptionType: __Type
   
-  directives: [__Directive!]!
+  types: [__Type!]!
 }
 
 """
@@ -278,29 +276,31 @@ the schema to define exactly what data is expected.
 """
 # Assigned to Rails::GraphQL::Type class
 type __Type {
-  kind: __TypeKind!
-  
-  name: String
-  
   description: String
   
+  "ENUM only"
+  enumValues(includeDeprecated: Boolean = false): [__EnumValue!]
+  
   "OBJECT and INTERFACE only"
   fields(includeDeprecated: Boolean = false): [__Field!]
   
+  "INPUT_OBJECT only"
+  inputFields: [__InputValue!]
+  
   "OBJECT only"
   interfaces: [__Type!]
   
-  "INTERFACE and UNION only"
-  possibleTypes: [__Type!]
-  
-  "ENUM only"
-  enumValues(includeDeprecated: Boolean = false): [__EnumValue!]
+  kind: __TypeKind!
   
-  "INPUT_OBJECT only"
-  inputFields: [__InputValue!]
+  name: String
   
   "NON_NULL and LIST only"
   ofType: __Type
+  
+  "INTERFACE and UNION only"
+  possibleTypes: [__Type!]
+  
+  specifiedByURL: String
 }
 
 """
@@ -333,3 +333,14 @@ directive @skip(
   if: Boolean!
   
 ) on FIELD | FRAGMENT_SPREAD | INLINE_FRAGMENT
+
+"""
+A built-in directive used within the type system definition language to provide
+a scalar specification URL for specifying the behavior of custom scalar types.
+"""
+directive @specifiedBy(
+
+  "Point to a human-readable specification of the data format."
+  url: String!
+  
+) on SCALAR

data/test/config.rb

@@ -17,10 +17,11 @@ require 'minitest/autorun'
 require 'minitest/reporters'
 require 'active_record'
 require 'rails-graphql'
-require 'debug'
+require 'pry-byebug'
 
 $config = Rails::GraphQL.config
 $config.logger = ActiveSupport::TaggedLogging.new(Logger.new('/dev/null'))
+$config.enable_introspection = true
 Rails::GraphQL::Request::Backtrace.skip_base_class = NilClass
 
 # ActiveRecord::Base.logger = Logger.new(STDOUT)

data/test/graphql/schema_test.rb

@@ -4,8 +4,8 @@ class GraphQL_SchemaTest < GraphQL::TestCase
   DESCRIBED_CLASS = unmapped_class(Rails::GraphQL::Schema)
 
   def test_gql_name
-    assert_equal('schema', DESCRIBED_CLASS.gql_name)
-    assert_equal('schema', DESCRIBED_CLASS.graphql_name)
+    assert_equal('__Schema', DESCRIBED_CLASS.gql_name)
+    assert_equal('__Schema', DESCRIBED_CLASS.graphql_name)
   end
 
   def test_kind
@@ -120,29 +120,6 @@ class GraphQL_SchemaTest < GraphQL::TestCase
     end
   end
 
-  def test_create_type
-    DESCRIBED_CLASS.stub(:create_klass, passallthrough) do
-      other = double(Module.new, base_type: double(name: 'A::Sample'))
-      result = [:some, other, type_const, { suffix: 'Sample' }]
-      assert_equal(result, DESCRIBED_CLASS.send(:create_type, :some, other))
-
-      result = [:some, type_const::Scalar, type_const, { suffix: 'Scalar' }]
-      assert_equal(result, DESCRIBED_CLASS.send(:create_type, :some, :Scalar))
-    end
-  end
-
-  def test_source
-    DESCRIBED_CLASS.stub(:create_klass, passallthrough) do
-      result = [:some, 1, source_const, { suffix: 'Source' }]
-      assert_equal(result, DESCRIBED_CLASS.send(:source, :some, 1))
-
-      source_const.stub(:find_for!, 2) do
-        result = [:some, 2, source_const, { suffix: 'Source' }]
-        assert_equal(result, DESCRIBED_CLASS.send(:source, :some))
-      end
-    end
-  end
-
   def test_sources
     result, passthrough = collect_all_through
     xargs = { build: true }
@@ -160,12 +137,6 @@ class GraphQL_SchemaTest < GraphQL::TestCase
     end
   end
 
-  def test_create_klass
-    skip 'Needs branching testing'
-    new_klass = DESCRIBED_CLASS.send :create_klass, 'Test', unmapped_class(type_const::Enum)
-    assert(new_klass < type_const::Enum)
-  end
-
   protected
 
     def type_const

data/test/graphql/source_test.rb

@@ -25,7 +25,7 @@ class GraphQL_SourceTest < GraphQL::TestCase
 
   def test_base_name
     described_class.stub(:abstract?, false) do
-      assert_equal('DESCRIBED', described_class.base_name)
+      assert_equal('DESCRIBED_CLASS', described_class.base_name)
     end
   end
 
@@ -144,16 +144,6 @@ class GraphQL_SourceTest < GraphQL::TestCase
     end
   end
 
-  def test_gql_module
-    assert_equal(::GraphQL, described_class.send(:gql_module))
-
-    ::GraphQL.stub_const(:Other, Module.new) do
-      sample = unmapped_class(source_const)
-      ::GraphQL::Other.const_set(:Source, sample)
-      assert_equal(::GraphQL::Other, sample.send(:gql_module))
-    end
-  end
-
   def test_build_bang
     skip
   end

data/test/graphql/type/interface_test.rb

@@ -31,20 +31,23 @@ class GraphQL_Type_InterfaceTest < GraphQL::TestCase
   end
 
   def test_implemented
+    skip 'Needs better double of interface'
     DESCRIBED_CLASS.stub(:gql_name, 'test') do
       DESCRIBED_CLASS.stub(:fields, {}) do
-        object = double(gql_name: 'baz', field?: ->(*) { false }, proxy_field: -> {})
+        object = double(gql_name: 'baz', has_field?: ->(*) { false }, proxy_field: -> {})
         assert_equal(Set[object], implemented_types(object))
       end
 
-      DESCRIBED_CLASS.stub(:fields, { 'a' => 'b' }) do
-        object = double(gql_name: 'baz', field?: ->(*) { true }, fields: { 'a' => /b/ })
+      field = double(name: 'a', :"=~" => ->(*) { true })
+      DESCRIBED_CLASS.stub(:fields, { 'a' => field }) do
+        object = double(gql_name: 'baz', has_field?: ->(*) { true }, fields: { 'a' => field })
         assert_equal(Set[object], implemented_types(object))
       end
 
-      fields = { 'a' => double(gql_name: 'a') }
+      field = double(name: 'a', :"=~" => ->(*) { false })
+      fields = { 'a' => field }
       DESCRIBED_CLASS.stub(:fields, fields) do
-        object = double(gql_name: 'baz', field?: ->(*) { true }, fields: { 'a' => 'b' })
+        object = double(gql_name: 'baz', has_field?: ->(*) { true }, fields: { 'a' => field })
         assert_raises(StandardError) { DESCRIBED_CLASS.implemented(object) }
       end
     end

data/test/graphql/type/object_test.rb

@@ -5,14 +5,17 @@ class GraphQL_Type_ObjectTest < GraphQL::TestCase
 
   def test_valid_member_ask
     fields = {
-      name: double(method_name: :name),
-      age:  double(method_name: :age),
+      name: double(method_name: :name, null?: true, gql_name: 'n'),
+      age:  double(method_name: :age, null?: false, gql_name: 'a'),
     }
 
     DESCRIBED_CLASS.stub(:fields, fields) do
       value = { name: '' }
       refute(DESCRIBED_CLASS.valid_member?(value))
 
+      value = { age: 0 }
+      assert(DESCRIBED_CLASS.valid_member?(value))
+
       value = { name: '', age: 0 }
       assert(DESCRIBED_CLASS.valid_member?(value))
 
@@ -20,6 +23,9 @@ class GraphQL_Type_ObjectTest < GraphQL::TestCase
       assert(DESCRIBED_CLASS.valid_member?(value))
 
       value = { 'name' => '', 'age' => 0 }
+      refute(DESCRIBED_CLASS.valid_member?(value))
+
+      value = { 'n' => '', 'a' => 0 }
       assert(DESCRIBED_CLASS.valid_member?(value))
 
       value = double(name: '',  age: 0)