Specify context for onComplete.

It seems to be a nice feature that the oncomplete method be allowed to
have a specific context within which to execute (GH-47).

Signed-off-by: Nick Campbell <nicholas.j.campbell@gmail.com>

lib/cron.js

@@ -231,18 +231,20 @@ CronTime.prototype = {
 
 
 
-function CronJob(cronTime, onTick, onComplete, start, timeZone) {
+function CronJob(cronTime, onTick, onComplete, start, timeZone, context) {
   if (typeof cronTime != "string" && arguments.length == 1) {
     //crontime is an object...
     onTick = cronTime.onTick;
     onComplete = cronTime.onComplete;
+    context = cronTime.context;
     start = cronTime.start;
     timeZone = cronTime.timeZone;
     cronTime = cronTime.cronTime;
   }
 
   if (timeZone && !(CronDate.prototype.setTimezone)) console.log('You specified a Timezone but have not included the `time` module. Timezone functionality is disabled. Please install the `time` module to use Timezones in your application.');
 
+  this.context = (context || this);
   this._callbacks = [];
   this.onComplete = onComplete;
   this.cronTime   = new CronTime(cronTime, timeZone);
@@ -270,7 +272,7 @@ CronJob.prototype = {
     for (var i = (this._callbacks.length - 1); i >= 0; i--) { 
 
       //send this so the callback can call this.stop();
-      this._callbacks[i].call(this, this.onComplete);
+      this._callbacks[i].call(this.context, this.onComplete);
     }
   },
 

tests/test-cron.js

@@ -361,5 +361,43 @@ module.exports = testCase({
       });
       assert.done();
     }, 1250);
+  },
+  'test cronjob scoping': function(assert) {
+    assert.expect(2);
+    var c = new cron.CronJob('* * * * * *', function() {
+      assert.ok(true);
+      assert.ok(c instanceof cron.CronJob);
+    }, null, true);
+    setTimeout(function() {
+      c.stop();
+      assert.done();
+    }, 1250);
+  },
+  'test non-cronjob scoping': function(assert) {
+    assert.expect(2);
+    var c = new cron.CronJob('* * * * * *', function() {
+      assert.ok(true);
+      assert.equal(this.hello, 'world');
+    }, null, true, null, {'hello':'world'});
+    setTimeout(function() {
+      c.stop();
+      assert.done();
+    }, 1250);
+  },
+  'test non-cronjob scoping inside object': function(assert) {
+    assert.expect(2);
+    var c = new cron.CronJob({
+      cronTime: '* * * * * *',
+      onTick: function() {
+        assert.ok(true);
+        assert.equal(this.hello, 'world');
+      },
+      start: true,
+      context: {hello: 'world'}
+    });
+    setTimeout(function() {
+      c.stop();
+      assert.done();
+    }, 1250);
   }
 });