fix(resolvers): resolve #1142 name issue and add relevant tests

Author: AvKat

src/utils/createResolversMap.ts

@@ -15,7 +15,7 @@ import { ResolversMap, EnumResolver, ResolverObject } from "../interfaces";
 export function createResolversMap(schema: GraphQLSchema): ResolversMap {
   const typeMap = schema.getTypeMap();
   return Object.keys(typeMap)
-    .filter(typeName => !typeName.includes("__"))
+    .filter(typeName => !typeName.startsWith("__"))
     .reduce<ResolversMap>((resolversMap, typeName) => {
       const type = typeMap[typeName];
       if (type instanceof GraphQLObjectType) {

tests/functional/typedefs-resolvers.ts

@@ -45,6 +45,7 @@ import {
   ResolversMap,
   ResolverObject,
   ResolverOptions,
+  FieldResolver,
 } from "../../src";
 
 describe("typeDefs and resolvers", () => {
@@ -104,6 +105,14 @@ describe("typeDefs and resolvers", () => {
         sampleType3StringField: string;
       }
 
+      @ObjectType("SampleType__4")
+      class SampleType4 {
+        @Field()
+        sampleInterfaceStringField: string;
+        @Field()
+        sampleType4StringField: string;
+      }
+
       @InputType()
       class SampleInput {
         @Field()
@@ -236,9 +245,26 @@ describe("typeDefs and resolvers", () => {
         }
       }
 
+      @Service()
+      @Resolver(of => SampleType4)
+      class SampleObjectTypeWithDoubleUnderscoreInNameResolver {
+        @FieldResolver(returns => String)
+        sampleResolvedField(): string {
+          return "sampleResolvedField";
+        }
+
+        @Query(returns => SampleType4)
+        async sampleQueryOnObjectTypeWithDoubleUnderScore(): Promise<SampleType4> {
+          const type4 = new SampleType4();
+          type4.sampleInterfaceStringField = "sampleInterfaceStringField";
+          type4.sampleType4StringField = "sampleType4StringField";
+          return type4;
+        }
+      }
+
       pubSub = new PubSub();
       ({ typeDefs, resolvers } = await buildTypeDefsAndResolvers({
-        resolvers: [SampleResolver],
+        resolvers: [SampleResolver, SampleObjectTypeWithDoubleUnderscoreInNameResolver],
         authChecker: () => false,
         pubSub,
         container: Container,
@@ -281,6 +307,10 @@ describe("typeDefs and resolvers", () => {
         const sampleType2 = schemaIntrospection.types.find(
           it => it.name === "SampleType2",
         ) as IntrospectionObjectType;
+        const sampleType4 = schemaIntrospection.types.find(
+          it => it.name === "SampleType__4",
+        ) as IntrospectionObjectType;
+
         const sampleType1StringField = sampleType1.fields.find(
           it => it.name === "sampleType1StringField",
         )!;
@@ -294,6 +324,7 @@ describe("typeDefs and resolvers", () => {
         expect(sampleType1.interfaces).toHaveLength(1);
         expect(sampleType1.interfaces[0].name).toBe("SampleInterface");
         expect(sampleType2StringField.deprecationReason).toBe("sampleType2StringFieldDeprecation");
+        expect(sampleType4.fields).toHaveLength(3);
       });
 
       it("should generate input type", async () => {
@@ -303,7 +334,8 @@ describe("typeDefs and resolvers", () => {
         const sampleInputDefaultStringField = sampleInput.inputFields.find(
           it => it.name === "sampleInputDefaultStringField",
         )!;
-        const sampleInputDefaultStringFieldType = sampleInputDefaultStringField.type as IntrospectionNamedTypeRef;
+        const sampleInputDefaultStringFieldType =
+          sampleInputDefaultStringField.type as IntrospectionNamedTypeRef;
 
         expect(sampleInput.kind).toBe(TypeKind.INPUT_OBJECT);
         expect(sampleInput.inputFields).toHaveLength(2);
@@ -350,7 +382,7 @@ describe("typeDefs and resolvers", () => {
           it => it.name === schemaIntrospection.queryType.name,
         ) as IntrospectionObjectType;
 
-        expect(queryType.fields).toHaveLength(8);
+        expect(queryType.fields).toHaveLength(9);
       });
 
       it("should generate mutations", async () => {
@@ -568,6 +600,26 @@ describe("typeDefs and resolvers", () => {
         expect(enumValue).toBe("OptionTwoString");
       });
 
+      it("should properly execute field resolver for object type with two underscores NOT in the beginning", async () => {
+        const document = gql`
+          query {
+            sampleQueryOnObjectTypeWithDoubleUnderScore {
+              sampleResolvedField
+              sampleInterfaceStringField
+              sampleType4StringField
+            }
+          }
+        `;
+
+        const { data } = await execute(schema, document);
+
+        expect(data!.sampleQueryOnObjectTypeWithDoubleUnderScore).toEqual({
+          sampleResolvedField: "sampleResolvedField",
+          sampleInterfaceStringField: "sampleInterfaceStringField",
+          sampleType4StringField: "sampleType4StringField",
+        });
+      });
+
       it("should properly run subscriptions", async () => {
         const document = gql`
           subscription {