fix(graphql, #505): Less restrictive readResolverOpts for auto crud

doug-martin · Sep 3, 2020 · 529d6f8 · 529d6f8
1 parent c59f43a
commit 529d6f8
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/packages/query-graphql/__tests__/module.spec.ts b/packages/query-graphql/__tests__/module.spec.ts
@@ -24,4 +24,21 @@ describe('NestjsQueryGraphQLModule', () => {
     expect(graphqlModule.providers).toHaveLength(3);
     expect(graphqlModule.exports).toHaveLength(4);
   });
+
+  it('should allow a defaultFilter for read options', () => {
+    const graphqlModule = NestjsQueryGraphQLModule.forFeature({
+      imports: [],
+      resolvers: [
+        {
+          DTOClass: TestDTO,
+          EntityClass: TestDTO,
+          read: { defaultFilter: { name: { eq: 'foo' } } },
+        },
+      ],
+    });
+    expect(graphqlModule.imports).toHaveLength(1);
+    expect(graphqlModule.module).toBe(NestjsQueryGraphQLModule);
+    expect(graphqlModule.providers).toHaveLength(3);
+    expect(graphqlModule.exports).toHaveLength(4);
+  });
 });
diff --git a/packages/query-graphql/src/module.ts b/packages/query-graphql/src/module.ts
@@ -12,7 +12,7 @@ export interface NestjsQueryGraphqlModuleOpts {
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   assemblers?: Class<Assembler<any, any>>[];
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  resolvers: AutoResolverOpts<any, any, unknown, unknown, ReadResolverOpts<unknown>, PagingStrategies>[];
+  resolvers: AutoResolverOpts<any, any, unknown, unknown, ReadResolverOpts<any>, PagingStrategies>[];
   pubSub?: Provider<GraphQLPubSub>;
 }