graphql 1.7.7 → 1.7.8

data/spec/graphql/query/arguments_spec.rb

@@ -16,14 +16,22 @@ describe GraphQL::Query::Arguments do
       argument :c, !test_input_1, as: :inputObject
     end
 
-    GraphQL::Query::Arguments.new({
-      a: 1,
-      b: 2,
-      c: GraphQL::Query::Arguments.new({
-        d: 3,
-        e: 4,
-      }, argument_definitions: test_input_1.arguments),
-    }, argument_definitions: test_input_2.arguments)
+    GraphQL::Query::Arguments.new(
+      {
+        a: 1,
+        b: 2,
+        c: GraphQL::Query::Arguments.new(
+          {
+            d: 3,
+            e: 4,
+          },
+          argument_definitions: test_input_1.arguments,
+          defaults_used: Set.new
+        ),
+      },
+      argument_definitions: test_input_2.arguments,
+      defaults_used: Set.new
+    )
   }
 
   it "returns keys as strings, with aliases" do
@@ -73,7 +81,11 @@ describe GraphQL::Query::Arguments do
       )
     end
 
-    new_arguments = GraphQL::Query::Arguments.new(transformed_args, argument_definitions: types)
+    new_arguments = GraphQL::Query::Arguments.new(
+      transformed_args,
+      argument_definitions: types,
+      defaults_used: Set.new
+    )
     expected_hash = {
       "A" => 1,
       "B" => 2,
@@ -96,7 +108,8 @@ describe GraphQL::Query::Arguments do
     it "wraps input objects, but not other hashes" do
       args = GraphQL::Query::Arguments.new(
         {a: 1, b: {a: 2}, c: {a: 3}},
-        argument_definitions: input_type.arguments
+        argument_definitions: input_type.arguments,
+        defaults_used: Set.new
       )
       assert_kind_of GraphQL::Query::Arguments, args["b"]
       assert_instance_of Hash, args["c"]
@@ -198,6 +211,15 @@ describe GraphQL::Query::Arguments do
       assert_equal({"a" => 1, "b" => 2}, last_args.to_h)
     end
 
+    it "indicates when default argument values were applied" do
+      schema.execute("{ argTest(a: 1) }")
+
+      last_args = arg_values.last
+
+      assert_equal false, last_args.default_used?('a')
+      assert_equal true, last_args.default_used?('b')
+    end
+
     it "works from variables" do
       variables = { "arg" => { "a" => 1, "d" => nil } }
       schema.execute("query ArgTest($arg: TestInput){ argTest(d: $arg) }", variables: variables)
@@ -306,7 +328,7 @@ describe GraphQL::Query::Arguments do
       assert_nil input_object.arguments_class
 
       GraphQL::Query::Arguments.construct_arguments_class(input_object)
-      args = input_object.arguments_class.new({foo: 3, bar: -90})
+      args = input_object.arguments_class.new({foo: 3, bar: -90}, Set.new)
 
       assert_equal 3, args.foo
       assert_equal -90, args.bar

data/spec/graphql/schema/build_from_definition_spec.rb

@@ -308,7 +308,7 @@ schema {
 }
 
 type Hello {
-  str(int: Int, bool: Boolean): String
+  str(bool: Boolean, int: Int): String
 }
       SCHEMA
 
@@ -505,7 +505,7 @@ type HelloScalars {
 }
 
 type Mutation {
-  addHelloScalars(str: String, int: Int, bool: Boolean): HelloScalars
+  addHelloScalars(bool: Boolean, int: Int, str: String): HelloScalars
 }
       SCHEMA
 
@@ -520,7 +520,7 @@ enum Color {
 }
 
 type Mutation {
-  hello(str: String, int: Int, color: Color = RED, nullDefault: Int = null): String
+  hello(color: Color = RED, int: Int, nullDefault: Int = null, str: String): String
 }
 
 type Query {
@@ -545,7 +545,7 @@ type HelloScalars {
 }
 
 type Subscription {
-  subscribeHelloScalars(str: String, int: Int, bool: Boolean): HelloScalars
+  subscribeHelloScalars(bool: Boolean, int: Int, str: String): HelloScalars
 }
       SCHEMA
 
@@ -603,6 +603,15 @@ type Query {
 
       build_schema_and_compare_output(schema.chop)
     end
+
+    it 'supports empty types' do
+      schema = <<-SCHEMA
+type Query {
+}
+      SCHEMA
+
+      build_schema_and_compare_output(schema.chop)
+    end
   end
 
   describe 'Failures' do

data/spec/graphql/schema/printer_spec.rb

@@ -136,6 +136,14 @@ schema {
   query: Root
 }
 
+# Marks an element of a GraphQL schema as no longer supported.
+directive @deprecated(
+  # Explains why this element was deprecated, usually also including a suggestion
+  # for how to access supported similar data. Formatted in
+  # [Markdown](https://daringfireball.net/projects/markdown/).
+  reason: String = "No longer supported"
+) on FIELD_DEFINITION | ENUM_VALUE
+
 # Directs the executor to include this field or fragment only when the `if` argument is true.
 directive @include(
   # Included when true.
@@ -148,14 +156,6 @@ directive @skip(
   if: Boolean!
 ) on FIELD | FRAGMENT_SPREAD | INLINE_FRAGMENT
 
-# Marks an element of a GraphQL schema as no longer supported.
-directive @deprecated(
-  # Explains why this element was deprecated, usually also including a suggestion
-  # for how to access supported similar data. Formatted in
-  # [Markdown](https://daringfireball.net/projects/markdown/).
-  reason: String = "No longer supported"
-) on FIELD_DEFINITION | ENUM_VALUE
-
 # A Directive provides a way to describe alternate runtime execution and type validation behavior in a GraphQL document.
 #
 # In some cases, you need to provide options to alter GraphQL's execution behavior
@@ -345,7 +345,6 @@ schema {
   subscription: Subscription
 }
 SCHEMA
-
       assert_match expected, GraphQL::Schema::Printer.print_schema(custom_schema)
     end
 
@@ -390,7 +389,7 @@ type Audio {
 
 enum Choice {
   BAR
-  BAZ @deprecated(reason: "Use \\\"BAR\\\".")
+  BAZ @deprecated(reason: "Use "BAR".")
   FOO
   WOZ @deprecated
 }
@@ -439,7 +438,7 @@ interface Node {
 type Post {
   body: String!
   comments: [Comment!]
-  comments_count: Int! @deprecated(reason: "Use \\\"comments\\\".")
+  comments_count: Int! @deprecated(reason: "Use "comments".")
   id: ID!
   title: String!
 }
@@ -489,7 +488,6 @@ enum Choice {
 }
 
 type Subscription {
-
 }
 
 input Varied {
@@ -509,7 +507,9 @@ SCHEMA
       when GraphQL::Argument
         member.name != "id"
       else
-        member.deprecation_reason.nil?
+        if member.respond_to?(:deprecation_reason)
+          member.deprecation_reason.nil?
+        end
       end
     }
 
@@ -600,7 +600,7 @@ SCHEMA
 type Post {
   body: String!
   comments: [Comment!]
-  comments_count: Int! @deprecated(reason: \"Use \\\"comments\\\".\")
+  comments_count: Int! @deprecated(reason: "Use "comments".")
   id: ID!
   title: String!
 }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: graphql
 version: !ruby/object:Gem::Version
-  version: 1.7.7
+  version: 1.7.8
 platform: ruby
 authors:
 - Robert Mosolgo
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-11-29 00:00:00.000000000 Z
+date: 2018-01-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: benchmark-ips
@@ -426,6 +426,7 @@ files:
 - lib/graphql/language/nodes.rb
 - lib/graphql/language/parser.rb
 - lib/graphql/language/parser.y
+- lib/graphql/language/printer.rb
 - lib/graphql/language/token.rb
 - lib/graphql/language/visitor.rb
 - lib/graphql/list_type.rb
@@ -905,6 +906,7 @@ files:
 - spec/graphql/language/lexer_spec.rb
 - spec/graphql/language/nodes_spec.rb
 - spec/graphql/language/parser_spec.rb
+- spec/graphql/language/printer_spec.rb
 - spec/graphql/language/visitor_spec.rb
 - spec/graphql/list_type_spec.rb
 - spec/graphql/non_null_type_spec.rb
@@ -1375,6 +1377,7 @@ test_files:
 - spec/graphql/language/lexer_spec.rb
 - spec/graphql/language/nodes_spec.rb
 - spec/graphql/language/parser_spec.rb
+- spec/graphql/language/printer_spec.rb
 - spec/graphql/language/visitor_spec.rb
 - spec/graphql/list_type_spec.rb
 - spec/graphql/non_null_type_spec.rb