fix git status

Author: twof

src/execution/__tests__/semantic-nullability-test.ts

@@ -1,12 +1,16 @@
 import { expect } from 'chai';
 import { describe, it } from 'mocha';
+
+import { GraphQLError } from '../../error/GraphQLError';
+
+import type { ExecutableDefinitionNode, FieldNode } from '../../language/ast';
 import { parse } from '../../language/parser';
+
 import { GraphQLNonNull, GraphQLObjectType, GraphQLSemanticNonNull, GraphQLSemanticNullable } from '../../type/definition';
-import { execute } from '../execute';
-import { GraphQLSchema } from '../../type/schema';
 import { GraphQLString } from '../../type/scalars';
-import { ExecutableDefinitionNode, FieldNode } from '../../language';
-import { GraphQLError } from '../../error/GraphQLError';
+import { GraphQLSchema } from '../../type/schema';
+
+import { execute } from '../execute';
 
 describe('Execute: Handles Semantic Nullability', () => {
     const DeepDataType = new GraphQLObjectType({
@@ -45,8 +49,8 @@ describe('Execute: Handles Semantic Nullability', () => {
         rootValue: data,
       });
 
-      let executable = document.definitions?.values().next().value as ExecutableDefinitionNode;
-      let selectionSet = executable.selectionSet.selections.values().next().value;
+      const executable = document.definitions?.values().next().value as ExecutableDefinitionNode;
+      const selectionSet = executable.selectionSet.selections.values().next().value;
 
       expect(result).to.deep.equal({
         data: {
@@ -68,7 +72,7 @@ describe('Execute: Handles Semantic Nullability', () => {
       const data = {
         a: () => 'Apple',
         b: () => { throw new Error(
-          `Something went wrong`,
+          'Something went wrong',
         ); },
         c: () => 'Cookie'
       };
@@ -79,8 +83,8 @@ describe('Execute: Handles Semantic Nullability', () => {
         }
       `);
 
-      let executable = document.definitions?.values().next().value as ExecutableDefinitionNode;
-      let selectionSet = executable.selectionSet.selections.values().next().value;
+      const executable = document.definitions?.values().next().value as ExecutableDefinitionNode;
+      const selectionSet = executable.selectionSet.selections.values().next().value;
 
       const result = await execute({
         schema: new GraphQLSchema({ query: DataType }),
@@ -131,9 +135,9 @@ describe('Execute: Handles Semantic Nullability', () => {
         rootValue: data,
       });
 
-      let executable = document.definitions?.values().next().value as ExecutableDefinitionNode;
-      let dSelectionSet = executable.selectionSet.selections.values().next().value as FieldNode;
-      let fSelectionSet = dSelectionSet.selectionSet?.selections.values().next().value;
+      const executable = document.definitions?.values().next().value as ExecutableDefinitionNode;
+      const dSelectionSet = executable.selectionSet.selections.values().next().value as FieldNode;
+      const fSelectionSet = dSelectionSet.selectionSet?.selections.values().next().value;
 
       expect(result).to.deep.equal({
         data: {

src/language/__tests__/parser-test.ts

@@ -11,7 +11,6 @@ import { Kind } from '../kinds';
 import { parse, parseConstValue, parseType, parseValue } from '../parser';
 import { Source } from '../source';
 import { TokenKind } from '../tokenKind';
-import { Console } from 'console';
 
 function expectSyntaxError(text: string) {
   return expectToThrowJSON(() => parse(text));

src/language/parser.ts

@@ -254,9 +254,9 @@ export class Parser {
    */
   parseDefinition(): DefinitionNode {
     // TODO: I don't know what isConst represents. Every other callsite has it false
-    let directives = this.parseDirectives(false);
-    for (let directive of directives) {
-      if (directive.name.value == "SemanticNullability") {
+    const directives = this.parseDirectives(false);
+    for (const directive of directives) {
+      if (directive.name.value === 'SemanticNullability') {
         this._options.useSemanticNullability = true;
       }
     }
@@ -779,22 +779,22 @@ export class Parser {
           kind: Kind.SEMANTIC_NULLABLE_TYPE,
           type,
         });
-      } else {
-        return this.node<SemanticNonNullTypeNode>(start, {
-          kind: Kind.SEMANTIC_NON_NULL_TYPE,
-          type,
-        });
       }
-    } else {
-      if (this.expectOptionalToken(TokenKind.BANG)) {
-        return this.node<NonNullTypeNode>(start, {
-          kind: Kind.NON_NULL_TYPE,
-          type,
-        });
-      }
-
-      return type;
+      
+      return this.node<SemanticNonNullTypeNode>(start, {
+        kind: Kind.SEMANTIC_NON_NULL_TYPE,
+        type,
+      });
+    } 
+    
+    if (this.expectOptionalToken(TokenKind.BANG)) {
+      return this.node<NonNullTypeNode>(start, {
+        kind: Kind.NON_NULL_TYPE,
+        type,
+      });
     }
+
+    return type;
   }
 
   /**

src/language/printer.ts

@@ -17,7 +17,7 @@ export interface PrintOptions {
  * Converts an AST into a string, using one set of reasonable
  * formatting rules.
  */
-export function print(ast: ASTNode, options: PrintOptions = {}): string {
+export function print(ast: ASTNode): string {
   return visit(ast, printDocASTReducer);
 }