Add coordinate field to schema element definitions

* Changes `GraphQLSchemaElement` interface into a base class which defines a `.coordinate` property and `toString`/`toJSON` methods.
* Uses the base class to types, fields, arguments, input fields, enum values, and directives.
* Uses this in validation error printing string templates.
* Exports the now finalized GraphQLSchemaElement, GraphQLField, GraphQLArgument, GraphQLInputField, GraphQLEnumValue classes.

This differs from the original #3145 in that GraphQLField, GraphQLArgument, GraphQLInputField, GraphQLEnumValue constructors take the parent type/field as arguments instead of the parent coordinate. This obviates the need to validate the parent coordinate.

Author: leebyron

src/index.ts

@@ -34,24 +34,23 @@ export type { GraphQLArgs } from './graphql.js';
 export { graphql, graphqlSync } from './graphql.js';
 
 // Create and operate on GraphQL type definitions and schema.
-export type {
-  GraphQLField,
-  GraphQLArgument,
-  GraphQLEnumValue,
-  GraphQLInputField,
-} from './type/index.js';
 export {
   resolveObjMapThunk,
   resolveReadonlyArrayThunk,
   // Definitions
   GraphQLSchema,
   GraphQLDirective,
+  GraphQLSchemaElement,
   GraphQLScalarType,
   GraphQLObjectType,
   GraphQLInterfaceType,
   GraphQLUnionType,
   GraphQLEnumType,
   GraphQLInputObjectType,
+  GraphQLField,
+  GraphQLArgument,
+  GraphQLEnumValue,
+  GraphQLInputField,
   GraphQLList,
   GraphQLNonNull,
   // Standard GraphQL Scalars

src/type/__tests__/definition-test.ts

@@ -329,6 +329,7 @@ describe('Type System: Objects', () => {
       }),
     });
     expect(objType.getFields().f).to.deep.include({
+      coordinate: 'SomeObject.f',
       parentType: objType,
       name: 'f',
       description: undefined,
@@ -358,6 +359,7 @@ describe('Type System: Objects', () => {
     const f = objType.getFields().f;
 
     expect(f).to.deep.include({
+      coordinate: 'SomeObject.f',
       parentType: objType,
       name: 'f',
       description: undefined,
@@ -372,6 +374,7 @@ describe('Type System: Objects', () => {
     expect(f.args).to.have.lengthOf(1);
 
     expect(f.args[0]).to.deep.include({
+      coordinate: 'SomeObject.f(arg:)',
       parent: f,
       name: 'arg',
       description: undefined,
@@ -736,6 +739,8 @@ describe('Type System: Enums', () => {
     expect(values).to.have.lengthOf(3);
 
     expect(values[0]).to.deep.include({
+      coordinate: 'EnumWithNullishValue.NULL',
+      parentEnum: EnumTypeWithNullishValue,
       name: 'NULL',
       description: undefined,
       value: null,
@@ -745,6 +750,8 @@ describe('Type System: Enums', () => {
     });
 
     expect(values[1]).to.deep.include({
+      coordinate: 'EnumWithNullishValue.NAN',
+      parentEnum: EnumTypeWithNullishValue,
       name: 'NAN',
       description: undefined,
       value: NaN,
@@ -754,6 +761,8 @@ describe('Type System: Enums', () => {
     });
 
     expect(values[2]).to.deep.include({
+      coordinate: 'EnumWithNullishValue.NO_CUSTOM_VALUE',
+      parentEnum: EnumTypeWithNullishValue,
       name: 'NO_CUSTOM_VALUE',
       description: undefined,
       value: 'NO_CUSTOM_VALUE',
@@ -856,6 +865,7 @@ describe('Type System: Input Objects', () => {
         },
       });
       expect(inputObjType.getFields().f).to.deep.include({
+        coordinate: 'SomeInputObject.f',
         parentType: inputObjType,
         name: 'f',
         description: undefined,
@@ -876,6 +886,7 @@ describe('Type System: Input Objects', () => {
         }),
       });
       expect(inputObjType.getFields().f).to.deep.include({
+        coordinate: 'SomeInputObject.f',
         parentType: inputObjType,
         name: 'f',
         description: undefined,

src/type/__tests__/directive-test.ts

@@ -40,6 +40,7 @@ describe('Type System: Directive', () => {
     expect(directive.args).to.have.lengthOf(2);
 
     expect(directive.args[0]).to.deep.include({
+      coordinate: '@Foo(foo:)',
       parent: directive,
       name: 'foo',
       description: undefined,
@@ -52,6 +53,7 @@ describe('Type System: Directive', () => {
     });
 
     expect(directive.args[1]).to.deep.include({
+      coordinate: '@Foo(bar:)',
       parent: directive,
       name: 'bar',
       description: undefined,

src/type/__tests__/enumType-test.ts

@@ -398,6 +398,7 @@ describe('Type System: Enum Values', () => {
 
     expect(values[0]).to.deep.include({
       parentEnum: ComplexEnum,
+      coordinate: 'Complex.ONE',
       name: 'ONE',
       description: undefined,
       value: Complex1,
@@ -408,6 +409,7 @@ describe('Type System: Enum Values', () => {
 
     expect(values[1]).to.deep.include({
       parentEnum: ComplexEnum,
+      coordinate: 'Complex.TWO',
       name: 'TWO',
       description: undefined,
       value: Complex2,