fix

ardatan · Jul 12, 2021 · 31ed61a · 31ed61a
1 parent 9309291
commit 31ed61a
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 10 deletions.
diff --git a/packages/batch-delegate/tests/withTransforms.test.ts b/packages/batch-delegate/tests/withTransforms.test.ts
@@ -73,7 +73,7 @@ describe('works with complex transforms', () => {
         ]
       }),
       resultTransformer: (results, delegationContext) => {
-        const userIds = delegationContext.args['userIds'];
+        const userIds = delegationContext.args?.['userIds'];
         const booksByUserIds = results.reduce(
           (acc: any, { userId, books }: { userId: string, books: any[] }) => {
             acc[userId] = books

diff --git a/packages/delegate/src/transforms/AddArgumentsAsVariables.ts b/packages/delegate/src/transforms/AddArgumentsAsVariables.ts
@@ -51,7 +51,7 @@ function addVariablesToRootField(
   variables: Record<string, any>;
 } {
   const document = originalRequest.document;
-  const variableValues = originalRequest.variables;
+  const variableValues = originalRequest.variables ?? {};
 
   const operations: Array<OperationDefinitionNode> = document.definitions.filter(
     def => def.kind === Kind.OPERATION_DEFINITION

diff --git a/packages/delegate/src/transforms/FilterToSchema.ts b/packages/delegate/src/transforms/FilterToSchema.ts
@@ -41,7 +41,7 @@ export default class FilterToSchema implements Transform {
 function filterToSchema(
   targetSchema: GraphQLSchema,
   document: DocumentNode,
-  variables: Record<string, any>
+  variables?: Record<string, any>
 ): { document: DocumentNode; variables: Record<string, any> } {
   const operations: Array<OperationDefinitionNode> = document.definitions.filter(
     def => def.kind === Kind.OPERATION_DEFINITION
@@ -105,13 +105,15 @@ function filterToSchema(
     });
   }
 
-  const newVariables = usedVariables.reduce((acc, variableName) => {
-    const variableValue = variables[variableName];
-    if (variableValue !== undefined) {
-      acc[variableName] = variableValue;
-    }
-    return acc;
-  }, {});
+  const newVariables = variables
+    ? usedVariables.reduce((acc, variableName) => {
+        const variableValue = variables[variableName];
+        if (variableValue !== undefined) {
+          acc[variableName] = variableValue;
+        }
+        return acc;
+      }, {})
+    : {};
 
   return {
     document: {