Smart defaults for flowerType and startDirection

Is there a benefit to requiring these to be specified every time? Otherwise the API is more friendly if we have sensible default values.

Author: Josh Lory

src/bee.js

@@ -13,15 +13,12 @@ const HONEY_SOUND = 'honey';
 const NECTAR_SOUND = 'nectar';
 
 module.exports = class Bee extends Gatherer {
-  constructor(maze, config) {
+  constructor(maze, config = {}) {
     super(maze, config);
+    const {level} = config;
 
-    this.defaultFlowerColor_ = (config.level.flowerType === 'redWithNectar' ?
-      'red' : 'purple');
-    if (this.defaultFlowerColor_ === 'purple' &&
-        config.level.flowerType !== 'purpleNectarHidden') {
-      throw new Error(`bad flowerType for Bee: ${config.level.flowerType}`);
-    }
+    this.defaultFlowerColor_ = level && level.flowerType === 'redWithNectar' ?
+      'red' : 'purple';
 
     // at each location, tracks whether user checked to see if it was a flower or
     // honeycomb using an if block

src/subtype.js

@@ -31,13 +31,12 @@ const TILE_SHAPES = {
 };
 
 module.exports = class Subtype extends EventEmitter {
-  constructor(maze, config) {
+  constructor(maze, {skin, level} = {}) {
     super();
 
     this.maze_ = maze;
-    this.skin_ = config.skin;
-    this.level_ = config.level;
-    this.startDirection = config.level.startDirection;
+    this.skin_ = skin;
+    this.startDirection = level ? level.startDirection : 0;
   }
 
   /**

test/unit/bee.test.js

@@ -1,8 +1,10 @@
+/* global describe, it, expect */
+
 import Bee from '../../src/bee';
 import BeeCell from '../../src/beeCell';
 import MazeMap from '../../src/mazeMap';
 
-var baseLevel = {
+const baseLevel = {
   honeyGoal: 1,
   map: [
     [0]
@@ -15,30 +17,6 @@ var baseLevel = {
 };
 
 describe("Bee", function () {
-  it("fails if no flowerType", function () {
-    var maze = {};
-    var config = {
-      level: baseLevel
-    };
-    delete config.level.flowerType;
-    expect(() => {
-      new Bee(maze, config);
-    }).toThrowError(/bad flowerType for Bee/);
-  });
-
-
-  it("fails if invalid flowerType", function () {
-    var maze = {};
-    var config = {
-      level: Object.assign(baseLevel, {
-        flowerType: 'invalid'
-      })
-    };
-    expect(() => {
-      new Bee(maze, config);
-    }).toThrowError(/bad flowerType for Bee/);
-  });
-
   describe("isRedFlower", function () {
     /**
      * Shim a 1x1 maze with the given values and validate that we get the