Custom query resolver is always null

[4F2E4A2E]

Ho can the query method be passed to the right Query type parent?
Or how can the resolver class be annotated correctly in order to resolve?

.graphql

type CustomQuery{
    foo: String
}
type Query {
    mw: CustomQuery
}

CustomResolver

import { Query, Resolver } from '@nestjs/graphql';

@Resolver()
export class CustomResolver {
  constructor() {}

  @Query()
  foo(): string {
    return 'bar';
  }
}

create schema

const typeDefs = this.graphQLFactory.mergeTypesByPaths( './**/*.graphql');
const schema = this.graphQLFactory.createSchema({ typeDefs });

result

{
  "data": {
    "mw": {
      "foo": null
    }
  },
  "extensions": {
    "tracing": {
      "version": 1,
      "startTime": "2018-02-20T17:01:09.202Z",
      "endTime": "2018-02-20T17:01:09.202Z",
      "duration": 242964,
      "execution": {
        "resolvers": [
          {
            "path": [
              "mw"
            ],
            "parentType": "Query",
            "fieldName": "mw",
            "returnType": "CustomQuery",
            "startOffset": 77828,
            "duration": 96395
          },
          {
            "path": [
              "mw",
              "foo"
            ],
            "parentType": "CustomQuery",
            "fieldName": "foo",
            "returnType": "String",
            "startOffset": 214124,
            "duration": 8691
          }
        ]
      }
    }
  }
}

maybe relevant dependencies

{
  "dependencies": {
    "@nestjs/common": "4.6.4",
    "@nestjs/core": "4.6.4",
    "@nestjs/graphql": "2.0.0",
    "@types/graphql": "0.12.4",
    "graphql": "0.13.1",
    "graphql-tools": "2.21.0"
  } 
}

A) What am I doing wrong?
B) Can anyone confirm that custom query types are not supported at the moment?
C) Would a PR supporting this via annotation be welcomed?

[rgolea]

@4F2E4A2E Okay... So you're doing wrong the entire resolver.
For the schema:

type CustomQuery{
    foo: String
}
type Query {
    mw: CustomQuery
}

You should do this:

import { Query, Resolver, ResolveProperty } from '@nestjs/graphql';

@Resolver('CustomQuery')
export class CustomResolver {
  constructor() {}
  //resolving the property
  @ResolveProperty()
  foo(): string {
    return 'bar';
  }
  //resolving via query (do one either/or)
  @Query()
  mw(){
     return {foo: 'bar'}
  }
}

[4F2E4A2E]

I did ghave tried this before, it dit not work. Did you get this working for sure?

[rgolea]

Yes. The Query decorator appends the resolver to the Query type and the (as so the Mutation or the Subscription decorator). The ResolveProperty gives format to that property and the Resolver decorator NEEDS a type name.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.