Add dots at the end of error messages (#2307)

Author: IvanGoncharov

src/execution/__tests__/executor-test.js

@@ -32,7 +32,7 @@ describe('Execute: Handles basic execution tasks', () => {
     });
 
     // $DisableFlowOnNegativeTest
-    expect(() => execute({ schema })).to.throw('Must provide document');
+    expect(() => execute({ schema })).to.throw('Must provide document.');
   });
 
   it('throws if no schema is provided', () => {

src/execution/execute.js

@@ -251,7 +251,7 @@ export function assertValidExecutionArguments(
   document: DocumentNode,
   rawVariableValues: ?{ +[variable: string]: mixed, ... },
 ): void {
-  devAssert(document, 'Must provide document');
+  devAssert(document, 'Must provide document.');
 
   // If the schema used for execution is invalid, throw an error.
   assertValidSchema(schema);

src/jsutils/invariant.js

@@ -4,7 +4,7 @@ export default function invariant(condition: mixed, message?: string): void {
   const booleanCondition = Boolean(condition);
   if (!booleanCondition) {
     throw new Error(
-      message != null ? message : 'Unexpected invariant triggered',
+      message != null ? message : 'Unexpected invariant triggered.',
     );
   }
 }

src/language/__tests__/parser-test.js

@@ -23,12 +23,12 @@ function expectSyntaxError(text) {
 describe('Parser', () => {
   it('asserts that a source to parse was provided', () => {
     // $DisableFlowOnNegativeTest
-    expect(() => parse()).to.throw('Must provide Source. Received: undefined');
+    expect(() => parse()).to.throw('Must provide Source. Received: undefined.');
   });
 
   it('asserts that an invalid source to parse was provided', () => {
     // $DisableFlowOnNegativeTest
-    expect(() => parse({})).to.throw('Must provide Source. Received: {}');
+    expect(() => parse({})).to.throw('Must provide Source. Received: {}.');
   });
 
   it('parse provides useful errors', () => {

src/language/__tests__/printer-test.js

@@ -27,7 +27,7 @@ describe('Printer: Query document', () => {
     const badAST = { random: 'Data' };
     // $DisableFlowOnNegativeTest
     expect(() => print(badAST)).to.throw(
-      'Invalid AST Node: { random: "Data" }',
+      'Invalid AST Node: { random: "Data" }.',
     );
   });
 

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

@@ -23,7 +23,7 @@ describe('Printer: SDL document', () => {
     const badAST = { random: 'Data' };
     // $DisableFlowOnNegativeTest
     expect(() => print(badAST)).to.throw(
-      'Invalid AST Node: { random: "Data" }',
+      'Invalid AST Node: { random: "Data" }.',
     );
   });
 

src/language/parser.js

@@ -171,7 +171,7 @@ class Parser {
     const sourceObj = typeof source === 'string' ? new Source(source) : source;
     devAssert(
       sourceObj instanceof Source,
-      `Must provide Source. Received: ${inspect(sourceObj)}`,
+      `Must provide Source. Received: ${inspect(sourceObj)}.`,
     );
 
     this._lexer = new Lexer(sourceObj);

src/language/source.js

@@ -27,11 +27,11 @@ export class Source {
     this.locationOffset = locationOffset || { line: 1, column: 1 };
     devAssert(
       this.locationOffset.line > 0,
-      'line in locationOffset is 1-indexed and must be positive',
+      'line in locationOffset is 1-indexed and must be positive.',
     );
     devAssert(
       this.locationOffset.column > 0,
-      'column in locationOffset is 1-indexed and must be positive',
+      'column in locationOffset is 1-indexed and must be positive.',
     );
   }
 }

src/language/visitor.js

@@ -294,7 +294,7 @@ export function visit(
     let result;
     if (!Array.isArray(node)) {
       if (!isNode(node)) {
-        throw new Error('Invalid AST Node: ' + inspect(node));
+        throw new Error(`Invalid AST Node: ${inspect(node)}.`);
       }
       const visitFn = getVisitFn(visitor, node.kind, isLeaving);
       if (visitFn) {

src/subscription/__tests__/subscribe-test.js

@@ -337,13 +337,13 @@ describe('Subscription Initialization Phase', () => {
     await expectPromiseToThrow(
       // $DisableFlowOnNegativeTest
       () => subscribe(emailSchema, null),
-      'Must provide document',
+      'Must provide document.',
     );
 
     await expectPromiseToThrow(
       // $DisableFlowOnNegativeTest
       () => subscribe({ schema: emailSchema }),
-      'Must provide document',
+      'Must provide document.',
     );
   });
 
@@ -386,7 +386,7 @@ describe('Subscription Initialization Phase', () => {
 
     await expectPromiseToThrow(
       () => createSubscription(pubsub, invalidEmailSchema),
-      'Subscription field must return Async Iterable. Received: "test"',
+      'Subscription field must return Async Iterable. Received: "test".',
     );
   });