So long, potato

FaceFactory is marginally more informative, if not less fun.

Author: rylnd

src/lib/potato.ts renamed to src/lib/FaceFactory.ts

@@ -1,7 +1,7 @@
 import { Hash, hashFactory, sumAndDiff } from 'avatars-utils';
 import { eyeImages, noseImages, mouthImages } from './imageFiles';
 
-export class Potato {
+export class FaceFactory {
   private colorHash: Hash<string>;
   private eyeHash: Hash<string>;
   private noseHash: Hash<string>;
@@ -19,7 +19,7 @@ export class Potato {
     this.mouthHash = new Hash(mouths, hashFactory(sumAndDiff));
   }
 
-  public parts(string): Face {
+  public create(string): Face {
     return {
       color: this.colorHash.get(string),
       eyes: this.eyeHash.get(string),
@@ -41,4 +41,9 @@ const defaultColors = [
   '#f6be5d',
 ];
 
-export default new Potato(defaultColors, eyeImages, noseImages, mouthImages);
+export default new FaceFactory(
+  defaultColors,
+  eyeImages,
+  noseImages,
+  mouthImages,
+);

src/routes/index.ts

@@ -3,7 +3,7 @@ import uuid from 'uuid';
 
 import { imageFileNames, imageFilePaths } from '../lib/imageFiles';
 import { combine } from '../lib/imager';
-import potato from '../lib/potato';
+import FaceFactory from '../lib/FaceFactory';
 
 const imageTypes: ImageType[] = ['eyes', 'nose', 'mouth'];
 
@@ -23,17 +23,17 @@ router.get('/list', (req, res) => {
 });
 
 router.get('/:size?/random', (req, res) => {
-  const faceParts = potato.parts(uuid.v4());
+  const face = FaceFactory.create(uuid.v4());
 
-  return combine(faceParts, req.params.size, (err, stdout) =>
+  return combine(face, req.params.size, (err, stdout) =>
     sendImage({ stdout, response: res }),
   );
 });
 
 router.get('/:size?/:id', (req, res, next) => {
-  const faceParts = potato.parts(req.params.id);
+  const face = FaceFactory.create(req.params.id);
 
-  return combine(faceParts, req.params.size, (err, stdout) =>
+  return combine(face, req.params.size, (err, stdout) =>
     sendImage({ stdout, response: res }),
   );
 });