Options.schema is overwritten

[NickBolles]

I'm submitting a...

[ ] Regression 
[x] Bug report
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request => Please do not submit support request here, instead post your question on Stack Overflow.

Current behavior

The schema passed in via the options.schema property is overwritten by the auto generated schema.

The schema passed in is merged to the executable schema built by the module here:
https://github.com/nestjs/graphql/blob/master/lib/graphql.factory.ts#L57

 let schema = options.schema
        ? mergeSchemas({
            schemas: [options.schema, executableSchema],
          })
        : executableSchema;

but the next 4 statements overwrite the schema

      const autoGeneratedSchemaConfig = autoGeneratedSchema.toConfig();
      const executableSchemaConfig = executableSchema.toConfig();
      const schemaConfig = this.overrideOrExtendResolvers(
        executableSchemaConfig,
        autoGeneratedSchemaConfig,
      );

      schema = new GraphQLSchema(schemaConfig);

Background

Added by PR

https://github.com/nestjs/graphql/pull/50/files

modified by this commit - bugfix() workaround - multiple graphql packages

56816b4

bug actually introduced in this commit, this change

775beca#diff-80fdafbe4271cf8fbb25f5506d9a3e8eR62

possibly related issue?

#76
#307

#98

Expected behavior

correctly merge the schemas

Minimal reproduction of the problem with instructions

What is the motivation / use case for changing the behavior?

allow passing a schema in to merge to the auto generated schema

[kamilmysliwiec]

Fixed in 6.3.1 :)

[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.