revert cancellation (#4535)

The existing version of cancellation yields partial results, which is
really resolver cancellation, something that can be done in user space.

This was (probably?) a mistake.

The plan is to re-implement cancellation as an abrupt termination, i.e.
execute will throw when abortSignal triggers.

This PR first reverts the current work to demonstrate the feature more
cleanly in the follow-on PR.

Author: yaacovCR

src/execution/AbortSignalListener.ts

@@ -26,7 +26,6 @@ export class ResolveInfo implements GraphQLResolveInfo {
   private _fieldDetailsList: FieldDetailsList;
   private _parentType: GraphQLObjectType;
   private _path: Path;
-  private _abortSignal: AbortSignal | undefined;
 
   private _fieldName: string | undefined;
   private _fieldNodes: ReadonlyArray<FieldNode> | undefined;
@@ -38,21 +37,18 @@ export class ResolveInfo implements GraphQLResolveInfo {
   private _operation: OperationDefinitionNode | undefined;
   private _variableValues: VariableValues | undefined;
 
-  // eslint-disable-next-line max-params
   constructor(
     validatedExecutionArgs: ValidatedExecutionArgs,
     fieldDef: GraphQLField<unknown, unknown>,
     fieldDetailsList: FieldDetailsList,
     parentType: GraphQLObjectType,
     path: Path,
-    abortSignal: AbortSignal | undefined,
   ) {
     this._validatedExecutionArgs = validatedExecutionArgs;
     this._fieldDef = fieldDef;
     this._fieldDetailsList = fieldDetailsList;
     this._parentType = parentType;
     this._path = path;
-    this._abortSignal = abortSignal;
   }
 
   get fieldName(): string {
@@ -107,8 +103,4 @@ export class ResolveInfo implements GraphQLResolveInfo {
     this._variableValues ??= this._validatedExecutionArgs.variableValues;
     return this._variableValues;
   }
-
-  get abortSignal(): AbortSignal | undefined {
-    return this._abortSignal;
-  }
 }

src/execution/ResolveInfo.ts

@@ -19,14 +19,10 @@ describe('ResolveInfo', () => {
     fields: { test: { type: GraphQLString } },
   });
 
-  const abortController = new AbortController();
-  const abortSignal = abortController.signal;
-
   const validatedExecutionArgs = validateExecutionArgs({
     schema: new GraphQLSchema({ query }),
     document: parse(`{ test }`),
     rootValue: { test: 'root' },
-    abortSignal,
   });
 
   assert('schema' in validatedExecutionArgs);
@@ -54,7 +50,6 @@ describe('ResolveInfo', () => {
     fieldDetailsList,
     query,
     path,
-    abortSignal,
   );
 
   it('exposes fieldName', () => {
@@ -104,8 +99,4 @@ describe('ResolveInfo', () => {
       validatedExecutionArgs.variableValues,
     );
   });
-
-  it('exposes abortSignal', () => {
-    expect(resolveInfo.abortSignal).to.equal(abortSignal);
-  });
 });