Merge pull request #9 from slindberg/missing-tag-object

Fixed issue with misreported tag when git object does not exist

Author: rwjblue

index.js

@@ -49,7 +49,7 @@ function commitForTag(gitPath, tag) {
   var taggedObject = fs.readFileSync(tagPath, { encoding: 'utf8' }).trim();
   var objectPath = path.join(gitPath, 'objects', taggedObject.slice(0, 2), taggedObject.slice(2));
 
-  if (!zlib.inflateSync) {
+  if (!zlib.inflateSync || !fs.existsSync(objectPath)) {
     // we cannot support annotated tags on node v0.10 because
     // zlib does not allow sync access
     return taggedObject;

tests/fixtures/tagged-commit-unpacked-no-object/dot-git/HEAD

@@ -0,0 +1 @@
+ref: refs/heads/master

tests/fixtures/tagged-commit-unpacked-no-object/dot-git/objects/54/3b9bebdc6bd5c4b22136034a95dd097a57d3dd

@@ -0,0 +1 @@
+c1ee41c325d54f410b133e0018c7a6b1316f6cda

tests/fixtures/tagged-commit-unpacked-no-object/dot-git/objects/e6/9de29bb2d1d6434b8b29ae775ad8c2e48c5391

@@ -0,0 +1 @@
+c1ee41c325d54f410b133e0018c7a6b1316f6cda

tests/fixtures/tagged-commit-unpacked-no-object/dot-git/refs/heads/master

@@ -112,6 +112,20 @@ describe('git-repo-info', function() {
       assert.deepEqual(result, expected);
     });
 
+    it('returns an object with repo info, including the tag (unpacked tags) when a tag object does not exist', function() {
+      var repoRoot = path.join(testFixturesPath, 'tagged-commit-unpacked-no-object');
+      var result = repoInfo(path.join(repoRoot, gitDir));
+
+      var expected = {
+        branch: 'master',
+        sha: 'c1ee41c325d54f410b133e0018c7a6b1316f6cda',
+        abbreviatedSha: 'c1ee41c325',
+        tag: 'awesome-tag'
+      };
+
+      assert.deepEqual(result, expected);
+    });
+
     if (zlib.inflateSync) {
       it('returns an object with repo info, including the tag (annotated tags)', function() {
         var repoRoot = path.join(testFixturesPath, 'tagged-annotated');