Merge pull request #52 from timsuchanek/fix-51

Fix #51

Author: tailhook

src/query/format.rs

@@ -186,13 +186,15 @@ impl<'a, T: Text<'a>> Displayable for Mutation<'a, T>
         if let Some(ref name) = self.name {
             f.write(" ");
             f.write(name.as_ref());
-            if !self.variable_definitions.is_empty() {
-                f.write("(");
-                for var in &self.variable_definitions {
-                    var.display(f);
-                }
-                f.write(")");
+        }
+        if !self.variable_definitions.is_empty() {
+            f.write("(");
+            self.variable_definitions[0].display(f);
+            for var in &self.variable_definitions[1..] {
+                f.write(", ");
+                var.display(f);
             }
+            f.write(")");
         }
         format_directives(&self.directives, f);
         f.write(" ");

tests/queries/mutation_nameless_vars.graphql

@@ -0,0 +1,4 @@
+mutation($first: Int, $second: Int) {
+  field1(first: $first)
+  field2(second: $second)
+}

tests/query_roundtrips.rs

@@ -44,6 +44,7 @@ fn roundtrip2(filename: &str) {
 #[test] fn query_arguments() { roundtrip("query_arguments"); }
 #[test] fn query_directive() { roundtrip("query_directive"); }
 #[test] fn mutation_directive() { roundtrip("mutation_directive"); }
+#[test] fn mutation_nameless_vars() { roundtrip("mutation_nameless_vars"); }
 #[test] fn subscription_directive() { roundtrip("subscription_directive"); }
 #[test] fn string_literal() { roundtrip("string_literal"); }
 #[test] fn triple_quoted_literal() { roundtrip("triple_quoted_literal"); }