Bug: printer can print non-parsable value

This fixes a bug where an empty "block" list could be skipped by the printer.

Author: leebyron

src/language/__tests__/schema-kitchen-sink.graphql

@@ -42,6 +42,8 @@ extend type Foo {
   seven(argument: [String]): Type
 }
 
+type NoFields {}
+
 directive @skip(if: Boolean!) on FIELD | FRAGMENT_SPREAD | INLINE_FRAGMENT
 
 directive @include(if: Boolean!)

src/language/__tests__/schema-printer-test.js

@@ -88,6 +88,8 @@ extend type Foo {
   seven(argument: [String]): Type
 }
 
+type NoFields {}
+
 directive @skip(if: Boolean!) on FIELD | FRAGMENT_SPREAD | INLINE_FRAGMENT
 
 directive @include(if: Boolean!) on FIELD | FRAGMENT_SPREAD | INLINE_FRAGMENT

src/language/printer.js

@@ -144,13 +144,13 @@ function join(maybeArray, separator) {
 }
 
 /**
- * Given maybeArray, print an empty string if it is null or empty, otherwise
- * print each item on its own line, wrapped in an indented "{ }" block.
+ * Given array, print each item on its own line, wrapped in an
+ * indented "{ }" block.
  */
-function block(maybeArray) {
-  return length(maybeArray) ?
-    indent('{\n' + join(maybeArray, '\n')) + '\n}' :
-    '';
+function block(array) {
+  return array && array.length !== 0 ?
+    indent('{\n' + join(array, '\n')) + '\n}' :
+    '{}';
 }
 
 /**
@@ -166,7 +166,3 @@ function wrap(start, maybeString, end) {
 function indent(maybeString) {
   return maybeString && maybeString.replace(/\n/g, '\n  ');
 }
-
-function length(maybeArray) {
-  return maybeArray ? maybeArray.length : 0;
-}