Only emit ternary if there are no args

src/codegen.ts

@@ -262,30 +262,36 @@ class Codegen {
         F.createIdentifier("source"),
         F.createIdentifier(name),
       );
-      const callExpression = F.createCallExpression(
+      let valueExpression: ts.Expression = F.createCallExpression(
         prop,
         undefined,
         RESOLVER_ARGS.map((name) => {
           return F.createIdentifier(name);
         }),
       );
 
-      const isFunc = F.createStrictEquality(
-        F.createTypeOfExpression(prop),
-        F.createStringLiteral("function"),
-      );
+      // If there are no args, this might be a property not a method. In that case, we first need to
+      // check if the property is a function. If it is, we call it, otherwise we just read the prop.
+      if (field.args.length === 0) {
+        const isFunc = F.createStrictEquality(
+          F.createTypeOfExpression(prop),
+          F.createStringLiteral("function"),
+        );
+
+        const ternary = F.createConditionalExpression(
+          isFunc,
+          undefined,
+          valueExpression,
+          undefined,
+          prop,
+        );
+        valueExpression = ternary;
+      }
 
-      const ternary = F.createConditionalExpression(
-        isFunc,
-        undefined,
-        callExpression,
-        undefined,
-        prop,
-      );
       return this.method(
         methodName,
         RESOLVER_ARGS.map((name) => this.param(name)),
-        [F.createReturnStatement(ternary)],
+        [F.createReturnStatement(valueExpression)],
       );
     }
 

src/tests/fixtures/field_definitions/RenamedFieldWithArgs.ts

@@ -0,0 +1,7 @@
+/** @gqlType */
+export default class SomeType {
+  /** @gqlField greetz */
+  hello(args: { greeting: string }): string {
+    return `${args.greeting} world!`;
+  }
+}

src/tests/fixtures/field_definitions/RenamedFieldWithArgs.ts.expected

@@ -0,0 +1,45 @@
+-----------------
+INPUT
+----------------- 
+/** @gqlType */
+export default class SomeType {
+  /** @gqlField greetz */
+  hello(args: { greeting: string }): string {
+    return `${args.greeting} world!`;
+  }
+}
+
+-----------------
+OUTPUT
+-----------------
+-- SDL --
+type SomeType {
+  greetz(greeting: String!): String @propertyName(name: "hello")
+}
+-- TypeScript --
+import { GraphQLSchema, GraphQLObjectType, GraphQLString, GraphQLNonNull } from "graphql";
+export function getSchema(): GraphQLSchema {
+    const SomeTypeType: GraphQLObjectType = new GraphQLObjectType({
+        name: "SomeType",
+        fields() {
+            return {
+                greetz: {
+                    name: "greetz",
+                    type: GraphQLString,
+                    args: {
+                        greeting: {
+                            name: "greeting",
+                            type: new GraphQLNonNull(GraphQLString)
+                        }
+                    },
+                    resolve(source, args, context, info) {
+                        return source.hello(source, args, context, info);
+                    }
+                }
+            };
+        }
+    });
+    return new GraphQLSchema({
+        types: [SomeTypeType]
+    });
+}