Throw error if provided incorrect type for resolvers' arguments

Author: nodkz

src/resolvers/count.js

@@ -3,12 +3,11 @@
 
 import type {
   MongooseModelT,
-  GraphQLObjectType,
   ExtendedResolveParams,
   genResolverOpts,
 } from '../definition';
 import Resolver from '../../../graphql-compose/src/resolver/resolver';
-import { GraphQLInt } from 'graphql';
+import { GraphQLInt, GraphQLObjectType } from 'graphql';
 
 import { filterHelperArgs, filterHelper } from './helpers/filter';
 
@@ -18,6 +17,16 @@ export default function count(
   gqType: GraphQLObjectType,
   opts?: genResolverOpts,
 ): Resolver {
+  if (!model || !model.modelName || !model.schema) {
+    throw new Error(
+      'First arg for Resolver count() should be instance of Mongoose Model.'
+    );
+  }
+
+  if (!(gqType instanceof GraphQLObjectType)) {
+    throw new Error('Second arg for Resolver count() should be instance of GraphQLObjectType.');
+  }
+
   return new Resolver({
     outputType: GraphQLInt,
     name: 'count',

src/resolvers/createOne.js

@@ -17,6 +17,16 @@ export default function createOne(
   gqType: GraphQLObjectType,
   opts?: genResolverOpts,
 ): Resolver {
+  if (!model || !model.modelName || !model.schema) {
+    throw new Error(
+      'First arg for Resolver createOne() should be instance of Mongoose Model.'
+    );
+  }
+
+  if (!(gqType instanceof GraphQLObjectType)) {
+    throw new Error('Second arg for Resolver createOne() should be instance of GraphQLObjectType.');
+  }
+
   const resolver = new Resolver({
     name: 'createOne',
     kind: 'mutation',

src/resolvers/findById.js

@@ -2,14 +2,14 @@
 
 import type {
   MongooseModelT,
-  GraphQLObjectType,
   ExtendedResolveParams,
   genResolverOpts,
 } from '../definition';
 import Resolver from '../../../graphql-compose/src/resolver/resolver';
 
 import {
   GraphQLNonNull,
+  GraphQLObjectType,
 } from 'graphql';
 import GraphQLMongoID from '../types/mongoid';
 
@@ -20,6 +20,16 @@ export default function findById(
   gqType: GraphQLObjectType,
   opts?: genResolverOpts // eslint-disable-line no-unused-vars
 ): Resolver {
+  if (!model || !model.modelName || !model.schema) {
+    throw new Error(
+      'First arg for Resolver findById() should be instance of Mongoose Model.'
+    );
+  }
+
+  if (!(gqType instanceof GraphQLObjectType)) {
+    throw new Error('Second arg for Resolver findById() should be instance of GraphQLObjectType.');
+  }
+
   return new Resolver({
     outputType: gqType,
     name: 'findById',

src/resolvers/findByIds.js

@@ -2,7 +2,6 @@
 
 import type {
   MongooseModelT,
-  GraphQLObjectType,
   ExtendedResolveParams,
   genResolverOpts,
 } from '../definition';
@@ -12,6 +11,7 @@ import mongoose from 'mongoose';
 import {
   GraphQLNonNull,
   GraphQLList,
+  GraphQLObjectType,
 } from 'graphql';
 import GraphQLMongoID from '../types/mongoid';
 
@@ -24,6 +24,16 @@ export default function findByIds(
   gqType: GraphQLObjectType,
   opts?: genResolverOpts
 ): Resolver {
+  if (!model || !model.modelName || !model.schema) {
+    throw new Error(
+      'First arg for Resolver findByIds() should be instance of Mongoose Model.'
+    );
+  }
+
+  if (!(gqType instanceof GraphQLObjectType)) {
+    throw new Error('Second arg for Resolver findByIds() should be instance of GraphQLObjectType.');
+  }
+
   return new Resolver({
     outputType: new GraphQLList(gqType),
     name: 'findByIds',

src/resolvers/findMany.js

@@ -3,13 +3,13 @@
 
 import type {
   MongooseModelT,
-  GraphQLObjectType,
   ExtendedResolveParams,
   genResolverOpts,
 } from '../definition';
 import Resolver from '../../../graphql-compose/src/resolver/resolver';
 import {
   GraphQLList,
+  GraphQLObjectType,
 } from 'graphql';
 
 import { skipHelperArgs, skipHelper } from './helpers/skip';
@@ -24,6 +24,16 @@ export default function findMany(
   gqType: GraphQLObjectType,
   opts?: genResolverOpts,
 ): Resolver {
+  if (!model || !model.modelName || !model.schema) {
+    throw new Error(
+      'First arg for Resolver findMany() should be instance of Mongoose Model.'
+    );
+  }
+
+  if (!(gqType instanceof GraphQLObjectType)) {
+    throw new Error('Second arg for Resolver findMany() should be instance of GraphQLObjectType.');
+  }
+
   return new Resolver({
     outputType: new GraphQLList(gqType),
     name: 'findMany',

src/resolvers/findOne.js

@@ -7,10 +7,10 @@ import { skipHelperArgs, skipHelper } from './helpers/skip';
 import { filterHelperArgs, filterHelper } from './helpers/filter';
 import { sortHelperArgs, sortHelper } from './helpers/sort';
 import { projectionHelper } from './helpers/projection';
+import { GraphQLObjectType } from 'graphql';
 
 import type {
   MongooseModelT,
-  GraphQLObjectType,
   ExtendedResolveParams,
   genResolverOpts,
 } from '../definition';
@@ -21,6 +21,16 @@ export default function findOne(
   gqType: GraphQLObjectType,
   opts?: genResolverOpts,
 ): Resolver {
+  if (!model || !model.modelName || !model.schema) {
+    throw new Error(
+      'First arg for Resolver findOne() should be instance of Mongoose Model.'
+    );
+  }
+
+  if (!(gqType instanceof GraphQLObjectType)) {
+    throw new Error('Second arg for Resolver findOne() should be instance of GraphQLObjectType.');
+  }
+
   return new Resolver({
     outputType: gqType,
     name: 'findOne',

src/resolvers/removeById.js

@@ -20,6 +20,18 @@ export default function removeById(
   gqType: GraphQLObjectType,
   opts?: genResolverOpts, // eslint-disable-line no-unused-vars
 ): Resolver {
+  if (!model || !model.modelName || !model.schema) {
+    throw new Error(
+      'First arg for Resolver removeById() should be instance of Mongoose Model.'
+    );
+  }
+
+  if (!(gqType instanceof GraphQLObjectType)) {
+    throw new Error(
+      'Second arg for Resolver removeById() should be instance of GraphQLObjectType.'
+    );
+  }
+
   const resolver = new Resolver({
     name: 'removeById',
     kind: 'mutation',

src/resolvers/removeMany.js

@@ -16,6 +16,18 @@ export default function removeMany(
   gqType: GraphQLObjectType,
   opts?: genResolverOpts,
 ): Resolver {
+  if (!model || !model.modelName || !model.schema) {
+    throw new Error(
+      'First arg for Resolver removeMany() should be instance of Mongoose Model.'
+    );
+  }
+
+  if (!(gqType instanceof GraphQLObjectType)) {
+    throw new Error(
+      'Second arg for Resolver removeMany() should be instance of GraphQLObjectType.'
+    );
+  }
+
   const resolver = new Resolver({
     name: 'removeMany',
     kind: 'mutation',

src/resolvers/removeOne.js

@@ -21,6 +21,18 @@ export default function removeOne(
   gqType: GraphQLObjectType,
   opts?: genResolverOpts,
 ): Resolver {
+  if (!model || !model.modelName || !model.schema) {
+    throw new Error(
+      'First arg for Resolver removeOne() should be instance of Mongoose Model.'
+    );
+  }
+
+  if (!(gqType instanceof GraphQLObjectType)) {
+    throw new Error(
+      'Second arg for Resolver removeOne() should be instance of GraphQLObjectType.'
+    );
+  }
+
   const resolver = new Resolver({
     name: 'removeOne',
     kind: 'mutation',

src/resolvers/updateById.js

@@ -18,6 +18,18 @@ export default function updateById(
   gqType: GraphQLObjectType,
   opts?: genResolverOpts,
 ): Resolver {
+  if (!model || !model.modelName || !model.schema) {
+    throw new Error(
+      'First arg for Resolver updateById() should be instance of Mongoose Model.'
+    );
+  }
+
+  if (!(gqType instanceof GraphQLObjectType)) {
+    throw new Error(
+      'Second arg for Resolver updateById() should be instance of GraphQLObjectType.'
+    );
+  }
+
   const findByIdResolver = findById(model, gqType);
 
   const resolver = new Resolver({