Merge pull request #56 from shaneog/buildkite

Add Buildkite environment variable for commit ID
archived-codacy · Mar 20, 2018 · 2d176fd · 2d176fd
2 parents 4e9dae1 + 42c385d
commit 2d176fd
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git a/lib/getGitData.js b/lib/getGitData.js
@@ -14,7 +14,8 @@
                     process.env.GIT_COMMIT ||
                     process.env.CIRCLE_SHA1 ||
                     process.env.CI_COMMIT_ID ||
-                    process.env.WERCKER_GIT_COMMIT;
+                    process.env.WERCKER_GIT_COMMIT ||
+                    process.env.BUILDKITE_COMMIT;
 
                 if (gitCommit) {
                     logger.debug('Received Commit Id: ' + gitCommit);

diff --git a/test/getGitData.js b/test/getGitData.js
@@ -39,6 +39,10 @@
             process.env.WERCKER_GIT_COMMIT = '5232';
             return expect(getGitData.getCommitId()).to.eventually.equal('5232');
         });
+        it('should be able to get the commit id from the Buildkite environment variable', function () {
+            process.env.BUILDKITE_COMMIT = '47326';
+            return expect(getGitData.getCommitId()).to.eventually.equal('47326');
+        });
         it('should be able to get the commit id from Git', function () {
             // If we are currently running on Travis, we should be able to use the commit id environment variable
             // to check the git commit id method with actual git. But we can't do this for Pull Requests because

diff --git a/test/helper.js b/test/helper.js
@@ -73,6 +73,7 @@
             process.env.CIRCLE_SHA1 = '';
             process.env.CI_COMMIT_ID = '';
             process.env.WERCKER_GIT_COMMIT = '';
+            process.env.BUILDKITE_COMMIT = '';
         }
     };
 }(require('nock'), require('chai'), require('bluebird')));