fix: remove duplicated code and clean up a bit

Author: jlengstorf

src/model.js

@@ -29,7 +29,6 @@ export default class IMDBAPIModel extends GraphQLModel {
       .catch(res =>
         this.throwError(res, {
           description: 'Unable to search movies',
-          docsLink: 'https://github.com/gramps-express/data-source-imdbapi',
         }),
       );
   }
@@ -50,7 +49,6 @@ export default class IMDBAPIModel extends GraphQLModel {
       .catch(res =>
         this.throwError(res, {
           description: 'Unable to search people',
-          docsLink: 'https://github.com/gramps-express/data-source-imdbapi',
         }),
       );
   }
@@ -68,7 +66,6 @@ export default class IMDBAPIModel extends GraphQLModel {
       .catch(res =>
         this.throwError(res, {
           description: 'Unable to get movie by ID',
-          docsLink: 'https://github.com/gramps-express/data-source-imdbapi',
         }),
       );
   }
@@ -86,7 +83,6 @@ export default class IMDBAPIModel extends GraphQLModel {
       .catch(res =>
         this.throwError(res, {
           description: 'Unable to get person by ID',
-          docsLink: 'https://github.com/gramps-express/data-source-imdbapi',
         }),
       );
   }
@@ -104,7 +100,7 @@ export default class IMDBAPIModel extends GraphQLModel {
       description: error.message || 'Something went wrong.',
       targetEndpoint: error.options ? error.options.uri : null,
       graphqlModel: this.constructor.name,
-      docsLink: null,
+      docsLink: 'https://github.com/gramps-express/data-source-imdbapi',
     };
 
     throw GrampsError({

src/resolvers.js

@@ -1,13 +1,7 @@
 import { MockList } from 'graphql-tools';
 import casual from 'casual';
 
-/*
- * For more information on data source resolvers, see
- * https://ibm.biz/graphql-data-source-resolvers
- */
-
 export default {
-  // Queries (where does the data come from?)
   queryResolvers: {
     searchMoviesByTitle: (rootValue, { options }, context) =>
       new Promise((resolve, reject) => {
@@ -39,9 +33,9 @@ export default {
       }),
   },
 
-  // Data fields (which data from the response goes to which field?)
   dataResolvers: {
     IMDB_Person: {
+      // Convert the filmography object into an array for filtering/typing.
       filmography: ({ filmography }, { filter = 'all' }) =>
         Object.keys(filmography)
           .reduce(
@@ -61,12 +55,11 @@ export default {
           .filter(work => filter === 'all' || work.position === filter),
     },
     IMDB_Metadata: {
-      // Alias this field to fix the typo.
+      // Alias this field to fix a typo.
       asp_ratio: data => data.asp_retio,
     },
   },
 
-  // Mock data (How can I get real-feeling fake data while working offline?)
   mockResolvers: {
     IMDB_Movie: () => ({
       cast: () => new MockList([1, 10]),