Memory management: added test for duplicating time

Kyle Smith · Kyle Smith · commit ce354331b751 · 2016-07-26T15:51:12.000-07:00
diff --git a/test/tests/signature.js b/test/tests/signature.js
@@ -2,6 +2,7 @@ var assert = require("assert");
 var path = require("path");
 var local = path.join.bind(path, __dirname);
 
+var garbageCollect = require("../utils/garbage_collect.js");
 var exec = require("../../utils/execPromise");
 
 describe("Signature", function() {
@@ -83,4 +84,26 @@ describe("Signature", function() {
       });
     });
   });
+
+  it("duplicates time", function() {
+    garbageCollect();
+    var Time = NodeGit.Time;
+    var startSelfFreeingCount = Time.getSelfFreeingInstanceCount();
+    var startNonSelfFreeingCount =
+      Time.getNonSelfFreeingConstructedCount();
+    var time = Signature.now(name, email).when();
+
+    garbageCollect();
+    var endSelfFreeingCount = Time.getSelfFreeingInstanceCount();
+    var endNonSelfFreeingCount = Time.getNonSelfFreeingConstructedCount();
+    // we should get one duplicated, self-freeing time
+    assert.equal(startSelfFreeingCount + 1, endSelfFreeingCount);
+    assert.equal(startNonSelfFreeingCount, endNonSelfFreeingCount);
+
+    time = null;
+    garbageCollect();
+    endSelfFreeingCount = Time.getSelfFreeingInstanceCount();
+    // the self-freeing time should get freed
+    assert.equal(startSelfFreeingCount, endSelfFreeingCount);
+  });
 });
