Merge pull request rmosolgo#658 from jamesdphillips/falsey-default-values-bug

Apply default values that are 'false'

Author: rmosolgo

lib/graphql/input_object_type.rb

@@ -95,12 +95,10 @@ def coerce_non_null_input(value)
 
         if value.key?(input_key)
           coerced_value = input_field_defn.type.coerce_input(field_value)
-        else
-          coerced_value = input_field_defn.default_value
-        end
-
-        if coerced_value || value.key?(input_key)
           input_values[input_key] = coerced_value
+        elsif input_field_defn.default_value?
+          default_value = input_field_defn.default_value
+          input_values[input_key] = default_value
         end
       end
 

spec/graphql/input_object_type_spec.rb

@@ -239,6 +239,7 @@
         a: String
         b: Int!
         c: String = "Default"
+        d: Boolean = false
       }
     |) }
     let(:input_type) { schema.types['ExampleInputObject'] }
@@ -274,6 +275,13 @@
       assert_equal 1, result['b']
       assert_equal 'Test', result['c']
     end
+
+    it "false default values are returned" do
+      input = MinimumInputObject.new({"b" => 1})
+      result = input_type.coerce_input(input)
+
+      assert_equal false, result['d']
+    end
   end
 
   describe "when sent into a query" do