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

The resolver metadata cache currently uses the resolver name for the
key but this causes issues if we have two resolvers in different modules
but with the same name and identical method signatures. For controllers,
each controller gets a unique ID that is used as the key in the cache.
Add code to set up unique IDs for resolvers as well so that cache
retrieval is correct.

Fixes #1907

Author: koyedele

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;
+  }
+}