enhance(wrap): avoid calling MapLeafValues.transformRequest if nothin…

…g is transformed during transformSchema
ardatan · Oct 31, 2022 · c1d01f3 · c1d01f3
1 parent c83f048
commit c1d01f3
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/.changeset/thick-panthers-carry.md b/.changeset/thick-panthers-carry.md
@@ -0,0 +1,5 @@
+---
+'@graphql-tools/wrap': patch
+---
+
+Do not transform request if there is no value transformed during the schema transformation
diff --git a/packages/wrap/src/transforms/TransformEnumValues.ts b/packages/wrap/src/transforms/TransformEnumValues.ts
@@ -18,6 +18,7 @@ export default class TransformEnumValues<TContext = Record<string, any>>
   private transformedSchema: GraphQLSchema | undefined;
   private mapping: Record<string, Record<string, string>>;
   private reverseMapping: Record<string, Record<string, string>>;
+  private noTransformation = true;
 
   constructor(
     enumValueTransformer: EnumValueTransformer,
@@ -50,6 +51,9 @@ export default class TransformEnumValues<TContext = Record<string, any>>
     delegationContext: DelegationContext<TContext>,
     transformationContext: TransformEnumValuesTransformationContext
   ): ExecutionRequest {
+    if (this.noTransformation) {
+      return originalRequest;
+    }
     return this.transformer.transformRequest(originalRequest, delegationContext, transformationContext);
   }
 
@@ -58,6 +62,9 @@ export default class TransformEnumValues<TContext = Record<string, any>>
     delegationContext: DelegationContext<TContext>,
     transformationContext: TransformEnumValuesTransformationContext
   ) {
+    if (this.noTransformation) {
+      return originalResult;
+    }
     return this.transformer.transformResult(originalResult, delegationContext, transformationContext);
   }
 
@@ -77,6 +84,7 @@ export default class TransformEnumValues<TContext = Record<string, any>>
         }
         this.mapping[typeName][externalValue] = newExternalValue;
         this.reverseMapping[typeName][newExternalValue] = externalValue;
+        this.noTransformation = false;
       }
     }
     return transformedEnumValue;