fix: Replace url with webhookUrl for _addWebhook

Author: Filbird

index.js

@@ -214,10 +214,10 @@ class BitbucketScm extends Scm {
      * is instead updated.
      * @method _addWebhook
      * @param  {Object}    config
-     * @param  {String}    config.scmUri  The SCM URI to add the webhook to
-     * @param  {String}    config.token   Oauth2 token to authenticate with Bitbucket
-     * @param  {String}    config.url     The URL to use for webhook notifications
-     * @return {Promise}                  Resolves upon success
+     * @param  {String}    config.scmUri    The SCM URI to add the webhook to
+     * @param  {String}    config.token     Oauth2 token to authenticate with Bitbucket
+      @param  {String}    config.webhookUrl The URL to use for the webhook notifications
+     * @return {Promise}                    Resolves upon success
      */
     _addWebhook(config) {
         const repoInfo = getScmUriParts(config.scmUri);
@@ -226,14 +226,14 @@ class BitbucketScm extends Scm {
             page: 1,
             repoId: repoInfo.repoId,
             token: config.token,
-            url: config.url
+            url: config.webhookUrl
         })
             .then(hookInfo =>
                 this._createWebhook({
                     hookInfo,
                     repoId: repoInfo.repoId,
                     token: config.token,
-                    url: config.url
+                    url: config.webhookUrl
                 })
             );
     }

test/index.test.js

@@ -1332,7 +1332,7 @@ describe('index', function () {
         });
     });
 
-    describe('_addWebhook', () => {
+    describe.only('_addWebhook', () => {
         const oauthToken = 'oauthToken';
         const scmUri = 'hostName:repoId:branchName';
 
@@ -1356,7 +1356,7 @@ describe('index', function () {
             /* eslint-enable no-underscore-dangle */
                 scmUri,
                 token: oauthToken,
-                url: 'url'
+                webhookUrl: 'url'
             })
                 .then(() => {
                     assert.calledWith(requestMock, {
@@ -1411,7 +1411,7 @@ describe('index', function () {
             /* eslint-enable no-underscore-dangle */
                 scmUri,
                 token: oauthToken,
-                url: 'url'
+                webhookUrl: 'url'
             }).then(() => {
                 assert.calledWith(requestMock, {
                     json: true,
@@ -1479,7 +1479,7 @@ describe('index', function () {
             /* eslint-enable no-underscore-dangle */
                 scmUri,
                 token: oauthToken,
-                url: 'url'
+                webhookUrl: 'url'
             }).then(() => {
                 assert.calledWith(requestMock, {
                     json: true,
@@ -1538,7 +1538,7 @@ describe('index', function () {
             /* eslint-enable no-underscore-dangle */
                 scmUri,
                 token,
-                url: 'url'
+                webhookUrl: 'url'
             }).then(assert.fail, (err) => {
                 assert.strictEqual(err.message, expectedMessage);
             });
@@ -1558,7 +1558,7 @@ describe('index', function () {
             /* eslint-enable no-underscore-dangle */
                 scmUri,
                 token,
-                url: 'url'
+                webhookUrl: 'url'
             }).then(assert.fail, (err) => {
                 assert.strictEqual(err.message, expectedMessage);
             });
@@ -1593,7 +1593,7 @@ describe('index', function () {
             /* eslint-enable no-underscore-dangle */
                 scmUri,
                 token,
-                url: 'url'
+                webhookUrl: 'url'
             }).then(assert.fail, (err) => {
                 assert.strictEqual(err.message, [
                     'Your credentials lack one or more required privilege scopes.',
@@ -1638,7 +1638,7 @@ describe('index', function () {
             /* eslint-enable no-underscore-dangle */
                 scmUri,
                 token,
-                url: 'url'
+                webhookUrl: 'url'
             }).then(assert.fail, (err) => {
                 assert.strictEqual(err.message, expectedMessage);
             });
@@ -1668,7 +1668,7 @@ describe('index', function () {
             /* eslint-enable no-underscore-dangle */
                 scmUri,
                 token,
-                url: 'url'
+                webhookUrl: 'url'
             }).then(assert.fail, (err) => {
                 assert.strictEqual(err.message, expectedMessage);
             });