Reflect defaultErrorBehavior in buildClientSchema

Author: benjie

src/utilities/__tests__/buildClientSchema-test.ts

@@ -18,6 +18,7 @@ import {
   GraphQLString,
 } from '../../type/scalars';
 import { GraphQLSchema } from '../../type/schema';
+import { validateSchema } from '../../type/validate';
 
 import { graphqlSync } from '../../graphql';
 
@@ -158,6 +159,26 @@ describe('Type System: build schema from introspection', () => {
     expect(clientSchema.getType('ID')).to.equal(undefined);
   });
 
+  it('reflects defaultErrorBehavior', () => {
+    const schema = buildSchema(`
+      schema @behavior(onError: NO_PROPAGATE) {
+        query: Query
+      }
+      type Query {
+        foo: String
+      }
+    `);
+    const introspection = introspectionFromSchema(schema, {
+      errorBehavior: true,
+    });
+    const clientSchema = buildClientSchema(introspection);
+
+    expect(clientSchema.defaultErrorBehavior).to.equal('NO_PROPAGATE');
+
+    const errors = validateSchema(clientSchema);
+    expect(errors).to.have.length(0);
+  });
+
   it('builds a schema with a recursive type reference', () => {
     const sdl = dedent`
       schema {

src/utilities/buildClientSchema.ts

@@ -108,6 +108,8 @@ export function buildClientSchema(
     ? schemaIntrospection.directives.map(buildDirective)
     : [];
 
+  const defaultErrorBehavior = schemaIntrospection.defaultErrorBehavior;
+
   // Then produce and return a Schema with these types.
   return new GraphQLSchema({
     description: schemaIntrospection.description,
@@ -116,6 +118,7 @@ export function buildClientSchema(
     subscription: subscriptionType,
     types: Object.values(typeMap),
     directives,
+    defaultErrorBehavior,
     assumeValid: options?.assumeValid,
   });
 

src/utilities/getIntrospectionQuery.ts

@@ -1,5 +1,7 @@
 import type { Maybe } from '../jsutils/Maybe';
 
+import type { GraphQLErrorBehavior } from '../error/ErrorBehavior';
+
 import type { DirectiveLocation } from '../language/directiveLocation';
 
 export interface IntrospectionOptions {
@@ -206,6 +208,7 @@ export interface IntrospectionSchema {
   >;
   readonly types: ReadonlyArray<IntrospectionType>;
   readonly directives: ReadonlyArray<IntrospectionDirective>;
+  readonly defaultErrorBehavior?: Maybe<GraphQLErrorBehavior>;
 }
 
 export type IntrospectionType =