jobName,pipelineId are in updateCommitStatus params

Author: d2lam

index.js

@@ -572,18 +572,15 @@ class BitbucketScm extends Scm {
      * @param  {String}   config.buildStatus  The screwdriver build status to translate into scm commit status
      * @param  {String}   config.token        The token used to authenticate to the SCM
      * @param  {String}   config.url          Target Url of this commit status
-     * @param  {String}   [config.jobName]    Optional name of the job that finished
-     * @param  {Number}   [config.pipelineId] Pipeline ID
+     * @param  {String}   config.jobName      Optional name of the job that finished
+     * @param  {Number}   config.pipelineId   Pipeline ID
      * @return {Promise}
      */
     _updateCommitStatus(config) {
-        let context = `Screwdriver/${config.pipelineId}`;
         const scm = getScmUriParts(config.scmUri);
-        const jobName = config.jobName && /^PR/.test(config.jobName) ? 'PR' : config.jobName;
+        let context = `Screwdriver/${config.pipelineId}/`;
 
-        if (jobName) {
-            context += `/${jobName}`;
-        }
+        context += /^PR/.test(config.jobName) ? 'PR' : config.jobName;
 
         const options = {
             url: `${REPO_URL}/${scm.repoId}/commit/${config.sha}/statuses/build`,

test/index.test.js

@@ -1073,18 +1073,6 @@ describe('index', function () {
             })
         );
 
-        it('successfully update status with correct values', () => {
-            config.buildStatus = 'ABORTED';
-            delete config.jobName;
-
-            expectedOptions.body.state = 'STOPPED';
-            expectedOptions.body.description = 'Screwdriver/123';
-
-            return scm.updateCommitStatus(config).then(() => {
-                assert.calledWith(requestMock, expectedOptions);
-            });
-        });
-
         it('rejects if status code is not 201 or 200', () => {
             fakeResponse = {
                 statusCode: 401,