Merge pull request #2408 from koyedele/ko-fix-duplicate-resolver-bug

fix(@nestjs/graphql): set unique ids on resolvers to fix cache errors

Author: kamilmysliwiec

packages/apollo/tests/code-first-duplicate-resolvers/app.module.ts

@@ -0,0 +1,21 @@
+import { ApolloDriver } from '@nestjs/apollo';
+import { Module } from '@nestjs/common';
+import { GraphQLModule } from '@nestjs/graphql';
+import { ModuleAModule } from './module-a/module-a.module';
+import { ModuleBModule } from './module-b/module-b.module';
+import { QueryResolver } from './query.resolver';
+
+@Module({
+  imports: [
+    GraphQLModule.forRoot({
+      driver: ApolloDriver,
+      autoSchemaFile: true,
+    }),
+    ModuleAModule,
+    ModuleBModule,
+    QueryResolver,
+  ],
+  controllers: [],
+  providers: [],
+})
+export class AppModule {}

packages/apollo/tests/code-first-duplicate-resolvers/module-a/module-a.module.ts

@@ -0,0 +1,9 @@
+import { Module } from '@nestjs/common';
+import { UserResolver } from './user.resolver';
+
+@Module({
+  imports: [],
+  controllers: [],
+  providers: [UserResolver],
+})
+export class ModuleAModule {}

packages/apollo/tests/code-first-duplicate-resolvers/module-a/user.resolver.ts

@@ -0,0 +1,9 @@
+import { Args, Mutation, Resolver } from '@nestjs/graphql';
+
+@Resolver()
+export class UserResolver {
+  @Mutation(() => String, { name: 'moduleALogin' })
+  login(@Args('code') code: string) {
+    return code;
+  }
+}

packages/apollo/tests/code-first-duplicate-resolvers/module-b/module-b.module.ts

@@ -0,0 +1,9 @@
+import { Module } from '@nestjs/common';
+import { UserResolver } from './user.resolver';
+
+@Module({
+  imports: [],
+  controllers: [],
+  providers: [UserResolver],
+})
+export class ModuleBModule {}

packages/apollo/tests/code-first-duplicate-resolvers/module-b/user.resolver.ts

@@ -0,0 +1,9 @@
+import { Args, Mutation, Resolver } from '@nestjs/graphql';
+
+@Resolver()
+export class UserResolver {
+  @Mutation(() => String, { name: 'moduleBLogin' })
+  login(@Args('username') username: string) {
+    return username;
+  }
+}

packages/apollo/tests/code-first-duplicate-resolvers/query.resolver.ts

@@ -0,0 +1,9 @@
+import { Args, Mutation, Query, Resolver } from '@nestjs/graphql';
+
+@Resolver()
+export class QueryResolver {
+  @Query(() => Boolean, { name: '_' })
+  test() {
+    return true;
+  }
+}

packages/apollo/tests/duplicate-resolvers/app.module.ts

@@ -0,0 +1,20 @@
+import { ApolloDriver } from '@nestjs/apollo';
+import { Module } from '@nestjs/common';
+import { GraphQLModule } from '@nestjs/graphql';
+import { join } from 'path';
+import { ModuleAModule } from './module-a/module-a.module';
+import { ModuleBModule } from './module-b/module-b.module';
+
+@Module({
+  imports: [
+    GraphQLModule.forRoot({
+      driver: ApolloDriver,
+      typePaths: [join(__dirname, '*.graphql')],
+    }),
+    ModuleAModule,
+    ModuleBModule,
+  ],
+  controllers: [],
+  providers: [],
+})
+export class AppModule {}

packages/apollo/tests/duplicate-resolvers/duplicate-resolver.graphql

@@ -0,0 +1,8 @@
+type Query {
+    _: Boolean
+}
+
+type Mutation {
+    moduleALogin(code: String): String
+    moduleBLogin(username: String): String
+}

packages/apollo/tests/duplicate-resolvers/module-a/module-a.module.ts

@@ -0,0 +1,9 @@
+import { Module } from '@nestjs/common';
+import { UserResolver } from './user.resolver';
+
+@Module({
+  imports: [],
+  controllers: [],
+  providers: [UserResolver],
+})
+export class ModuleAModule {}

packages/apollo/tests/duplicate-resolvers/module-a/user.resolver.ts

@@ -0,0 +1,9 @@
+import { Args, Mutation, Resolver } from '@nestjs/graphql';
+
+@Resolver()
+export class UserResolver {
+  @Mutation('moduleALogin')
+  login(@Args('code') code: string) {
+    return code;
+  }
+}