Fixed the comparison that was being done. Based on Travis's documentation the environment variable is set to: The pull request number if the current job is a pull request, "false" if it's not a pull request.

Author: DavidTPate

test/getGitData.js

@@ -50,8 +50,9 @@
         });
         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.
-            if (actualTravisCommit && process.env.TRAVIS_PULL_REQUEST !== 'true') {
+            // to check the git commit id method with actual git. But we can't do this for Pull Requests because
+            // Travis provides the Commit id that triggered the Pull Request not the current Commit Id
+            if (actualTravisCommit && process.env.TRAVIS_PULL_REQUEST === 'false') {
                 return expect(getGitData.getCommitId()).to.eventually.equal(actualTravisCommit);
             }
             return expect(getGitData.getCommitId()).to.eventually.be.fulfilled;