Make CoffeeScript optional, improved coverage.

Author: ezzatron

lib/CouchBuilder.js

@@ -32,18 +32,21 @@
     "node": ">= 0.10.0",
     "npm": ">= 1.0.0"
   },
-  "devDependencies": {
-    "chai": "^3.0.0",
-    "codecov.io": "^0.1.4",
-    "coffee-coverage": "^1.0.1",
-    "coffee-script": "^1.9.2",
-    "coffeelint": "^1.10.1",
-    "istanbul": "^0.4.0",
-    "mocha": "^2.2.5",
-    "sinon": "^1.14.1"
-  },
   "dependencies": {
     "bluebird": "^3.1.5",
     "readdirp": "^2.0.0"
+  },
+  "optionalDependencies": {
+    "coffee-script": "^1.10.0"
+  },
+  "devDependencies": {
+    "chai": "^3.5.0",
+    "codecov.io": "^0.1.6",
+    "coffee-coverage": "^1.0.1",
+    "coffee-script": "^1.10.0",
+    "coffeelint": "^1.14.2",
+    "istanbul": "^0.4.2",
+    "mocha": "^2.4.5",
+    "sinon": "^1.17.3"
   }
 }

lib/handler/CommonCoffeeHandler.js

@@ -36,11 +36,7 @@ module.exports = class CouchBuilder
 
     _processEntries: (entries) -> new Promise (resolve, reject) =>
         entries.sort (left, right) ->
-            return -1 if left.path < right.path
-            return 1 if left.path > right.path
-
-            ### !pragma coverage-skip-next ###
-            return 0 # should never get two identical paths
+            return if left.path < right.path then -1 else 1
 
         Promise.all(@_processPath entry.fullPath for entry in entries)
         .then (results) =>

package.json

@@ -1,4 +1,3 @@
-coffee = require 'coffee-script'
 fs = require 'fs'
 path = require 'path'
 Promise = require 'bluebird'
@@ -27,6 +26,8 @@ module.exports = class CommonCoffeeHandler
 
                 return reject error
 
+            coffee = require 'coffee-script'
+
             try
                 js = coffee.compile data.toString()
             catch error

src/CouchBuilder.coffee

@@ -2,14 +2,22 @@ HandlerError = require '../../../../src/handler/error/HandlerError'
 
 describe 'HandlerError', ->
 
-    beforeEach ->
+    it 'stores the supplied error details', ->
         @cause = new Error 'Message.'
         @subject = new HandlerError 'HandlerName', '/path/to/file', @cause
 
-    it 'stores the supplied error details', ->
         assert.strictEqual @subject.message, "HandlerName could not process '/path/to/file'."
         assert.strictEqual @subject.toString(),
             "HandlerName could not process '/path/to/file'. Caused by:\n\nError: Message."
         assert.strictEqual @subject.handlerName, 'HandlerName'
         assert.strictEqual @subject.filename, '/path/to/file'
         assert.strictEqual @subject.cause, @cause
+
+    it 'handles errors with no cause', ->
+        @subject = new HandlerError 'HandlerName', '/path/to/file'
+
+        assert.strictEqual @subject.message, "HandlerName could not process '/path/to/file'."
+        assert.strictEqual @subject.toString(), "HandlerName could not process '/path/to/file'."
+        assert.strictEqual @subject.handlerName, 'HandlerName'
+        assert.strictEqual @subject.filename, '/path/to/file'
+        assert.isUndefined @subject.cause