Merge pull request #5 from codacy/fix-commit-tests

Fix tests (thanks @DavidTPate)

Author: DavidTPate

test/getGitData.js

@@ -50,11 +50,12 @@
         });
         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) {
+            // 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;
         });
     });
-}(require('joi'), require('chai'), require('q'), require('util'), require('../lib/getGitData')));
+}(require('joi'), require('chai'), require('q'), require('util'), require('../lib/getGitData')));