Use empty arrays instead of undefined properties

Author: IvanGoncharov

src/type/definition.js

@@ -643,7 +643,7 @@ export class GraphQLObjectType {
   name: string;
   description: ?string;
   astNode: ?ObjectTypeDefinitionNode;
-  extensionASTNodes: ?$ReadOnlyArray<ObjectTypeExtensionNode>;
+  extensionASTNodes: $ReadOnlyArray<ObjectTypeExtensionNode>;
   isTypeOf: ?GraphQLIsTypeOfFn<*, *>;
 
   _typeConfig: GraphQLObjectTypeConfig<*, *>;
@@ -654,7 +654,7 @@ export class GraphQLObjectType {
     this.name = config.name;
     this.description = config.description;
     this.astNode = config.astNode;
-    this.extensionASTNodes = config.extensionASTNodes;
+    this.extensionASTNodes = config.extensionASTNodes || [];
     this.isTypeOf = config.isTypeOf;
     this._typeConfig = config;
     invariant(typeof config.name === 'string', 'Must provide name.');
@@ -900,7 +900,7 @@ export class GraphQLInterfaceType {
   name: string;
   description: ?string;
   astNode: ?InterfaceTypeDefinitionNode;
-  extensionASTNodes: ?$ReadOnlyArray<InterfaceTypeExtensionNode>;
+  extensionASTNodes: $ReadOnlyArray<InterfaceTypeExtensionNode>;
   resolveType: ?GraphQLTypeResolver<*, *>;
 
   _typeConfig: GraphQLInterfaceTypeConfig<*, *>;
@@ -910,7 +910,7 @@ export class GraphQLInterfaceType {
     this.name = config.name;
     this.description = config.description;
     this.astNode = config.astNode;
-    this.extensionASTNodes = config.extensionASTNodes;
+    this.extensionASTNodes = config.extensionASTNodes || [];
     this.resolveType = config.resolveType;
     this._typeConfig = config;
     invariant(typeof config.name === 'string', 'Must provide name.');

src/type/schema.js

@@ -76,7 +76,7 @@ export function isSchema(schema) {
  */
 export class GraphQLSchema {
   astNode: ?SchemaDefinitionNode;
-  extensionASTNodes: ?$ReadOnlyArray<SchemaExtensionNode>;
+  extensionASTNodes: $ReadOnlyArray<SchemaExtensionNode>;
   _queryType: ?GraphQLObjectType;
   _mutationType: ?GraphQLObjectType;
   _subscriptionType: ?GraphQLObjectType;
@@ -87,7 +87,7 @@ export class GraphQLSchema {
   // Used as a cache for validateSchema().
   __validationErrors: ?$ReadOnlyArray<GraphQLError>;
   // Referenced by validateSchema().
-  __allowedLegacyNames: ?$ReadOnlyArray<string>;
+  __allowedLegacyNames: $ReadOnlyArray<string>;
 
   constructor(config: GraphQLSchemaConfig): void {
     // If this schema was built from a source known to be valid, then it may be
@@ -117,14 +117,14 @@ export class GraphQLSchema {
       );
     }
 
-    this.__allowedLegacyNames = config.allowedLegacyNames;
+    this.__allowedLegacyNames = config.allowedLegacyNames || [];
     this._queryType = config.query;
     this._mutationType = config.mutation;
     this._subscriptionType = config.subscription;
     // Provide specified directives (e.g. @include and @skip) by default.
     this._directives = config.directives || specifiedDirectives;
     this.astNode = config.astNode;
-    this.extensionASTNodes = config.extensionASTNodes;
+    this.extensionASTNodes = config.extensionASTNodes || [];
 
     // Build type map now to detect any errors within this schema.
     let initialTypes: Array<?GraphQLNamedType> = [

src/type/validate.js

@@ -218,10 +218,7 @@ function validateName(
 ): void {
   // If a schema explicitly allows some legacy name which is no longer valid,
   // allow it to be assumed valid.
-  if (
-    context.schema.__allowedLegacyNames &&
-    context.schema.__allowedLegacyNames.indexOf(node.name) !== -1
-  ) {
+  if (context.schema.__allowedLegacyNames.indexOf(node.name) !== -1) {
     return;
   }
   // Ensure names are valid, however introspection types opt out.
@@ -580,14 +577,10 @@ function validateInputFields(
 
 function getAllNodes<T: ASTNode, K: ASTNode>(object: {
   +astNode: ?T,
-  +extensionASTNodes?: ?$ReadOnlyArray<K>,
+  +extensionASTNodes: $ReadOnlyArray<K>,
 }): $ReadOnlyArray<T | K> {
   const { astNode, extensionASTNodes } = object;
-  return astNode
-    ? extensionASTNodes
-      ? [astNode].concat(extensionASTNodes)
-      : [astNode]
-    : extensionASTNodes || [];
+  return astNode ? [astNode].concat(extensionASTNodes) : extensionASTNodes;
 }
 
 function getImplementsInterfaceNode(

src/utilities/__tests__/extendSchema-test.js

@@ -245,7 +245,7 @@ describe('extendSchema', () => {
     const testDirective = extendedTwiceSchema.getDirective('test');
 
     expect(query.extensionASTNodes).to.have.lengthOf(2);
-    expect(testType.extensionASTNodes).to.equal(undefined);
+    expect(testType.extensionASTNodes).to.have.lengthOf(0);
 
     const restoredExtensionAST = {
       kind: Kind.DOCUMENT,

src/utilities/extendSchema.js

@@ -223,9 +223,7 @@ export function extendSchema(
   });
 
   const schemaExtensionASTNodes = schemaExtensions
-    ? schema.extensionASTNodes
-      ? schema.extensionASTNodes.concat(schemaExtensions)
-      : schemaExtensions
+    ? schema.extensionASTNodes.concat(schemaExtensions)
     : schema.extensionASTNodes;
 
   const types = [
@@ -239,10 +237,9 @@ export function extendSchema(
   // Support both original legacy names and extended legacy names.
   const schemaAllowedLegacyNames = schema.__allowedLegacyNames;
   const extendAllowedLegacyNames = options && options.allowedLegacyNames;
-  const allowedLegacyNames =
-    schemaAllowedLegacyNames && extendAllowedLegacyNames
-      ? schemaAllowedLegacyNames.concat(extendAllowedLegacyNames)
-      : schemaAllowedLegacyNames || extendAllowedLegacyNames;
+  const allowedLegacyNames = extendAllowedLegacyNames
+    ? schemaAllowedLegacyNames.concat(extendAllowedLegacyNames)
+    : schemaAllowedLegacyNames;
 
   // Then produce and return a Schema with these types.
   return new GraphQLSchema({
@@ -301,9 +298,7 @@ export function extendSchema(
   function extendObjectType(type: GraphQLObjectType): GraphQLObjectType {
     const name = type.name;
     const extensionASTNodes = typeExtensionsMap[name]
-      ? type.extensionASTNodes
-        ? type.extensionASTNodes.concat(typeExtensionsMap[name])
-        : typeExtensionsMap[name]
+      ? type.extensionASTNodes.concat(typeExtensionsMap[name])
       : type.extensionASTNodes;
     return new GraphQLObjectType({
       name,
@@ -321,9 +316,7 @@ export function extendSchema(
   ): GraphQLInterfaceType {
     const name = type.name;
     const extensionASTNodes = typeExtensionsMap[name]
-      ? type.extensionASTNodes
-        ? type.extensionASTNodes.concat(typeExtensionsMap[name])
-        : typeExtensionsMap[name]
+      ? type.extensionASTNodes.concat(typeExtensionsMap[name])
       : type.extensionASTNodes;
     return new GraphQLInterfaceType({
       name: type.name,