Corrected the float specification as per spec and Lee Byrons direction

bbakerman · bbakerman · commit 9dfacb2e7556 · 2017-12-17T13:39:49.000+11:00
diff --git a/src/main/antlr/Graphql.g4 b/src/main/antlr/Graphql.g4
@@ -199,7 +199,7 @@ NAME: [_A-Za-z][_0-9A-Za-z]*;
 
 IntValue : Sign? IntegerPart;
 
-FloatValue : Sign? IntegerPart ('.' Digit*)? ExponentPart?;
+FloatValue : Sign? IntegerPart ('.' Digit+)? ExponentPart?;
 
 Sign : '-';
 
diff --git a/src/test/groovy/graphql/parser/ParserTest.groovy b/src/test/groovy/graphql/parser/ParserTest.groovy
@@ -313,14 +313,14 @@ class ParserTest extends Specification {
         given:
         def input = """
             query myQuery {
-              hello(arg: {intKey:1, floatKey: 4., stringKey: \"world\", subObject: {subKey:true} } ) }
+              hello(arg: {intKey:1, floatKey: 4.1, stringKey: \"world\", subObject: {subKey:true} } ) }
             """
 
         and: "expected query"
 
         def objectValue = new ObjectValue()
         objectValue.getObjectFields().add(new ObjectField("intKey", new IntValue(1)))
-        objectValue.getObjectFields().add(new ObjectField("floatKey", new FloatValue(4)))
+        objectValue.getObjectFields().add(new ObjectField("floatKey", new FloatValue(4.1)))
         objectValue.getObjectFields().add(new ObjectField("stringKey", new StringValue("world")))
         def subObject = new ObjectValue()
         subObject.getObjectFields().add(new ObjectField("subKey", new BooleanValue(true)))
@@ -379,6 +379,18 @@ class ParserTest extends Specification {
 
     }
 
+    def "#848 floats must have digits"() {
+        given:
+        def input = """
+            { hello(arg: 4.) }
+            """
+        when:
+        def document = new Parser().parseDocument(input)
+
+        then:
+        thrown(ParseCancellationException)
+    }
+
     def "extraneous input is an exception"() {
         given:
         def input = """
