Rename AST nodes to *Node to disambiguate types.

Author: leebyron

src/error/GraphQLError.js

@@ -9,7 +9,7 @@
  */
 
 import { getLocation } from '../language';
-import type { Node } from '../language/ast';
+import type { ASTNode } from '../language/ast';
 import type { Source } from '../language/source';
 
 /**
@@ -50,7 +50,7 @@ declare class GraphQLError extends Error {
   /**
    * An array of GraphQL AST Nodes corresponding to this error.
    */
-  nodes: Array<Node> | void;
+  nodes: Array<ASTNode> | void;
 
   /**
    * The source GraphQL document corresponding to this error.

src/error/__tests__/GraphQLError-test.js

@@ -64,9 +64,9 @@ describe('GraphQLError', () => {
       field
     }`);
     const ast = parse(source);
-    const fieldAST = ast.definitions[0].selectionSet.selections[0];
-    const e = new GraphQLError('msg', [ fieldAST ]);
-    expect(e.nodes).to.deep.equal([ fieldAST ]);
+    const fieldNode = ast.definitions[0].selectionSet.selections[0];
+    const e = new GraphQLError('msg', [ fieldNode ]);
+    expect(e.nodes).to.deep.equal([ fieldNode ]);
     expect(e.source).to.equal(source);
     expect(e.positions).to.deep.equal([ 8 ]);
     expect(e.locations).to.deep.equal([ { line: 2, column: 7 } ]);
@@ -77,9 +77,9 @@ describe('GraphQLError', () => {
       field
     }`);
     const ast = parse(source);
-    const operationAST = ast.definitions[0];
-    const e = new GraphQLError('msg', [ operationAST ]);
-    expect(e.nodes).to.deep.equal([ operationAST ]);
+    const operationNode = ast.definitions[0];
+    const e = new GraphQLError('msg', [ operationNode ]);
+    expect(e.nodes).to.deep.equal([ operationNode ]);
     expect(e.source).to.equal(source);
     expect(e.positions).to.deep.equal([ 0 ]);
     expect(e.locations).to.deep.equal([ { line: 1, column: 1 } ]);

src/execution/__tests__/executor-test.js

@@ -208,7 +208,7 @@ describe('Execute: Handles basic execution tasks', () => {
 
     expect(Object.keys(info)).to.deep.equal([
       'fieldName',
-      'fieldASTs',
+      'fieldNodes',
       'returnType',
       'parentType',
       'path',
@@ -219,8 +219,8 @@ describe('Execute: Handles basic execution tasks', () => {
       'variableValues',
     ]);
     expect(info.fieldName).to.equal('test');
-    expect(info.fieldASTs).to.have.lengthOf(1);
-    expect(info.fieldASTs[0]).to.equal(
+    expect(info.fieldNodes).to.have.lengthOf(1);
+    expect(info.fieldNodes[0]).to.equal(
       ast.definitions[0].selectionSet.selections[0]
     );
     expect(info.returnType).to.equal(GraphQLString);