Fix flow and unit test

graphql · Apr 12, 2016 · 10997c5 · 10997c5
1 parent d460b74
commit 10997c5
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/src/validation/__tests__/KnownArgumentNames-test.js b/src/validation/__tests__/KnownArgumentNames-test.js
@@ -109,7 +109,7 @@ describe('Validate: Known argument names', () => {
         dog @skip(unless: true)
       }
     `, [
-      unknownDirectiveArg('unless', 'skip', [], 3, 19),
+      unknownDirectiveArg('unless', 'skip', [ 'if' ], 3, 19),
     ]);
   });
 

diff --git a/src/validation/rules/KnownArgumentNames.js b/src/validation/rules/KnownArgumentNames.js
@@ -53,6 +53,7 @@ export function unknownDirectiveArgMessage(
   } else {
     message += ' There is no known argument for this directive.';
   }
+  return message;
 }
 
 /**

diff --git a/src/validation/rules/KnownTypeNames.js b/src/validation/rules/KnownTypeNames.js
@@ -15,7 +15,7 @@ import { suggestionList } from '../../utilities/suggestionList';
 
 export function unknownTypeMessage(
   type: GraphQLType,
-  suggestedTypes: Arrray<string>
+  suggestedTypes: Array<string>
 ): string {
   let message = `Unknown type "${type}".`;
   const MAX_LENGTH = 5;