dont emit 'error', that will raise an actual error if there are no li…

…steners. assume the user uses callbacks or the finished event to catch errors

lib/index.js

@@ -45,10 +45,7 @@ ChainGang = (function() {
   };
   ChainGang.prototype.finish = function(job, err) {
     this.current -= 1;
-    this.emit('finished', job.name);
-    if (err) {
-      this.emit('error', err, job.name);
-    }
+    this.emit('finished', job.name, err);
     delete this.index[job.name];
     delete job;
     if (this.active) {

src/index.coffee

@@ -60,9 +60,7 @@ class ChainGang extends events.EventEmitter
   # Emits ('finished', name)
   finish: (job, err) ->
     @current -= 1
-    @emit 'finished', job.name
-    if err
-      @emit 'error', err, job.name
+    @emit 'finished', job.name, err
     delete @index[job.name]
     delete job
 
@@ -84,8 +82,7 @@ class Job
   #
   # Returns nothing.
   # Emits ('starting', name)
-  # Emits ('error', err, name) if the callback raises an exception.
-  # Emits ('finished', name) when the job has completed.
+  # Emits ('finished', name, err) when the job has completed.
   perform: ->
     @chain.current += 1
     @chain.emit 'starting', @name

test/error_test.coffee

@@ -7,16 +7,11 @@ assert.equal 2, chain.limit
 
 called_finished = false
 called_callback = false
-called_error    = false
 
-chain.on 'finished', (name) ->
-  assert.equal 'foo', name
-  called_finished = true
-
-chain.on 'error', (err, name) ->
+chain.on 'finished', (name, err) ->
   assert.ok err
   assert.equal 'foo', name
-  called_error = true
+  called_finished = true
 
 chain.add (worker) ->
   a.b == c
@@ -25,5 +20,4 @@ chain.add (worker) ->
 
 process.on 'exit', ->
   assert.ok called_finished
-  assert.ok called_error
   assert.equal false, called_callback