Update webhook test, added exception for this

This commit has two fixes:
a. The pull request hook parsing should work
   test was failing, as it was not expecting
   the number parameter in plugin_data->github->pull_request
   We are parsing and sending the pull request
   number in webhooks.js around line 128
   Here, I expect that the source is correct and
   the test needs to be updated.

b. The Step module requires the use of "this"
   (https://www.npmjs.com/package/step)
   However JSHint has a problem with that usage of this.
   JSHint says "Possible strict violation" because we are using
   this inside something that, as far as it can tell, is not a method.

   That leads to a fail during npm test.
   Since the source code must use the Step module,
   hence added a jshint exception for W040.

Author: saraf

lib/index.js

@@ -12,6 +12,8 @@ var User = require('./models').User;
 
 var GITHUB_API_ENDPOINT = 'https://api.github.com';
 
+/* jshint -W040 */
+
 /**
  * get_oauth2()
  *

test/test_webhooks.js

@@ -42,7 +42,6 @@ describe('webhooks', function () {
       it('should work', function () {
         var fx = require('./sample_pull_request.json')
           , config = lib.pullRequestJob(fx.pull_request, fx.action)
-
         expect(config).to.eql({
           branch: 'master',
           deploy: false,
@@ -51,7 +50,8 @@ describe('webhooks', function () {
               pull_request: {
                 user: 'jaredly',
                 repo: 'petulant-wookie',
-                sha: 'f65ac3101a45bb9408c0459805b496cb73ae2d5f'
+                sha: 'f65ac3101a45bb9408c0459805b496cb73ae2d5f',
+                number: 1
               }
             }
           },