Add \n at the end of .graphql files (#2285)

We need to use the same hack as CSS unfortunately.

Fixes #2275

src/multiparser.js

@@ -80,7 +80,10 @@ function fromBabylonFlowOrTypeScript(path) {
         return {
           options: { parser: "graphql" },
           transformDoc: doc =>
-            concat([indent(concat([softline, doc])), softline]),
+            concat([
+              indent(concat([softline, stripTrailingHardline(doc)])),
+              softline
+            ]),
           text: parent.quasis[0].value.raw
         };
       }

src/printer-graphql.js

@@ -22,7 +22,10 @@ function genericPrint(path, options, print) {
 
   switch (n.kind) {
     case "Document": {
-      return join(concat([hardline, hardline]), path.map(print, "definitions"));
+      return concat([
+        join(concat([hardline, hardline]), path.map(print, "definitions")),
+        hardline
+      ]);
     }
     case "OperationDefinition": {
       return concat([

tests/graphql_arguments/__snapshots__/jsfmt.spec.js.snap

@@ -16,4 +16,5 @@ exports[`hello.graphql 1`] = `
     string: "hello world this is a very long string!"
   )
 }
+
 `;

tests/graphql_bracket_spacing/__snapshots__/jsfmt.spec.js.snap

@@ -12,6 +12,7 @@ fragment Visit on HighlightedVisit
   @argumentDefinitions(count: { type: "Int", defaultValue: 20 }) {
   name
 }
+
 `;
 
 exports[`bracket_spacing.graphql 2`] = `
@@ -26,4 +27,5 @@ fragment Visit on HighlightedVisit
   @argumentDefinitions(count: {type: "Int", defaultValue: 20}) {
   name
 }
+
 `;

tests/graphql_comments/__snapshots__/jsfmt.spec.js.snap

@@ -15,6 +15,7 @@ query(
 ) {
   someField
 }
+
 `;
 
 exports[`fields.graphql 1`] = `
@@ -27,4 +28,5 @@ query {
 query {
   someField # Trailing comment
 }
+
 `;

tests/graphql_definitions/__snapshots__/jsfmt.spec.js.snap

@@ -49,4 +49,5 @@ query {
 {
   noOperationType
 }
+
 `;

tests/graphql_directives/__snapshots__/jsfmt.spec.js.snap

@@ -47,4 +47,5 @@ fragment YouCanHaveDirectivesHereToo on SomeType @yesReally(what: "yes") {
 query QueryWVars($x: String) @directive {
   hey
 }
+
 `;

tests/graphql_fields/__snapshots__/jsfmt.spec.js.snap

@@ -20,4 +20,5 @@ exports[`fields.graphql 1`] = `
     }
   }
 }
+
 `;

tests/graphql_fragments/__snapshots__/jsfmt.spec.js.snap

@@ -32,4 +32,5 @@ fragment XYZ on ABC {
   field
   ...AFragment
 }
+
 `;

tests/graphql_hello/__snapshots__/jsfmt.spec.js.snap

@@ -12,4 +12,5 @@ exports[`hello.graphql 1`] = `
     tagline
   }
 }
+
 `;

tests/graphql_kitchen_sink/__snapshots__/jsfmt.spec.js.snap

@@ -110,4 +110,5 @@ fragment frag on Friend {
 query {
   field
 }
+
 `;

tests/graphql_lists/__snapshots__/jsfmt.spec.js.snap

@@ -16,4 +16,5 @@ exports[`lists.graphql 1`] = `
     ]
   )
 }
+
 `;

tests/graphql_objects/__snapshots__/jsfmt.spec.js.snap

@@ -22,4 +22,5 @@ exports[`objects.graphql 1`] = `
   )
   emptyObj(arg: {})
 }
+
 `;

tests/graphql_trailing_comma/__snapshots__/jsfmt.spec.js.snap

@@ -49,6 +49,7 @@ query CityGuidesSocialUnitRendererQuery(
     }
   }
 }
+
 `;
 
 exports[`trailing.graphql 2`] = `
@@ -100,6 +101,7 @@ query CityGuidesSocialUnitRendererQuery(
     }
   }
 }
+
 `;
 
 exports[`trailing.graphql 3`] = `
@@ -151,4 +153,5 @@ query CityGuidesSocialUnitRendererQuery(
     }
   }
 }
+
 `;

tests/graphql_union_types/__snapshots__/jsfmt.spec.js.snap

@@ -55,4 +55,5 @@ union longUnion =
 # comment
 # comment2
 union union = B | C | D
+
 `;

tests/graphql_variable_definitions/__snapshots__/jsfmt.spec.js.snap

@@ -45,4 +45,5 @@ query listslong(
 ) {
   ok
 }
+
 `;