Fix some typos and duplicated code

Author: hirsch88

src/api/queries/GetPetsQuery.ts

@@ -1,12 +1,12 @@
 import { GraphQLFieldConfig, GraphQLList } from 'graphql';
-import { Query, AbstractQuery, GraphQLContext } from './../../lib/graphql';
+import { Query, AbstractGraphQLQuery, GraphQLContext } from './../../lib/graphql';
 import { PetService } from '../services/PetService';
 import { PetType } from './../types/PetType';
 import { Logger } from '../../core/Logger';
 import { Pet } from '../models/Pet';
 
 @Query()
-export class GetPetsQuery extends AbstractQuery<GraphQLContext<any, any>, Pet[], any> implements GraphQLFieldConfig {
+export class GetPetsQuery extends AbstractGraphQLQuery<GraphQLContext<any, any>, Pet[], any> implements GraphQLFieldConfig {
     public type = new GraphQLList(PetType);
     public allow = [];
     public args = {};

src/api/queries/GetUsersQuery.ts

@@ -1,12 +1,12 @@
 import { GraphQLFieldConfig, GraphQLList } from 'graphql';
-import { Query, AbstractQuery, GraphQLContext } from './../../lib/graphql';
+import { Query, AbstractGraphQLQuery, GraphQLContext } from './../../lib/graphql';
 import { UserService } from '../services/UserService';
 import { UserType } from './../types/UserType';
 import { User } from '../models/User';
 import { Logger } from '../../core/Logger';
 
 @Query()
-export class GetUsersQuery extends AbstractQuery<GraphQLContext<any, any>, User[], any> implements GraphQLFieldConfig {
+export class GetUsersQuery extends AbstractGraphQLQuery<GraphQLContext<any, any>, User[], any> implements GraphQLFieldConfig {
     public type = new GraphQLList(UserType);
     public allow = [];
     public args = {};

src/lib/graphql/AbstractGraphQLHooks.ts

@@ -1,11 +1,5 @@
 import { UserError } from './graphql-error-handling';
 
-// export interface GraphQLHooks {
-//     before<C, A, S>(context: C, args: A, source?: S): Promise<A> | A;
-//     after<R, C, A, S>(result: R, context: C, args: A, source?: S): Promise<R> | R;
-//     run<S, A, C, R>(rootOrSource: S, args: A, context: C): Promise<R> | Promise<undefined> | R | undefined;
-// }implements GraphQLHooks
-
 export abstract class AbstractGraphQLHooks<TContext, TResult, TArgs>  {
 
     /**

src/lib/graphql/AbstractGraphQLMutation.ts

@@ -1,20 +1,5 @@
-import { AbstractGraphQLHooks } from './AbstractGraphQLHooks';
+import { AbstractGraphQLQuery } from './AbstractGraphQLQuery';
 
 
-export abstract class AbstractGraphQLMutation<TContext, TResult, TArgs> extends AbstractGraphQLHooks<TContext, TResult, TArgs> {
-
-    /**
-     * This will be called by graphQL and they need to have it not as a
-     * member function of this class. We use this hook to add some more logic
-     * to it, like permission checking and before and after hooks to alter some data.
-     */
-    public resolve = async <S>(root: S, args: TArgs, context: TContext): Promise<TResult> => {
-        // We need to store the query arguments in the context so they can be accessed by subsequent resolvers
-        (context as any).resolveArgs = args;
-        args = await this.before(context, args);
-        let result = await this.run<S>(root, args, context);
-        result = await this.after(result, context, args);
-        return (result as TResult);
-    }
-
+export abstract class AbstractGraphQLMutation<TContext, TResult, TArgs> extends AbstractGraphQLQuery<TContext, TResult, TArgs> {
 }

src/lib/graphql/AbstractGraphQLQuery.ts

@@ -1,7 +1,7 @@
 import { AbstractGraphQLHooks } from './AbstractGraphQLHooks';
 
 
-export abstract class AbstractQuery<TContext, TResult, TArgs> extends AbstractGraphQLHooks<TContext, TResult, TArgs> {
+export abstract class AbstractGraphQLQuery<TContext, TResult, TArgs> extends AbstractGraphQLHooks<TContext, TResult, TArgs> {
 
     /**
      * This will be called by graphQL and they need to have it not as a