Attempt to fix Windows file locking bug

tbranyen · John Haley · commit 749826c81799 · 2015-02-25T17:17:02.000-07:00
diff --git a/generate/input/descriptor.json b/generate/input/descriptor.json
@@ -1431,9 +1431,6 @@
         "git_repository_fetchhead_foreach": {
           "ignore": true
         },
-        "git_repository_free": {
-          "ignore": true
-        },
         "git_repository_hashfile": {
           "ignore": true
         },
diff --git a/generate/scripts/helpers.js b/generate/scripts/helpers.js
@@ -288,8 +288,8 @@ var Helpers = {
     // available
     if (key == typeDef.cType + "_free") {
       typeDef.freeFunctionName = key;
-      fnDef.ignore = true;
-      return;
+      //fnDef.ignore = true;
+      //return;
     }
 
     fnDef.cppFunctionName = Helpers.cTypeToCppName(key, "git_" + typeDef.typeName);
diff --git a/test/tests/clone.js b/test/tests/clone.js
@@ -28,7 +28,9 @@ describe("Clone", function() {
       fse.remove(ssh),
       fse.remove(git),
       fse.remove(file)
-    ]).catch(function unhandledFunction() {});
+    ]).catch(function unhandledFunction(ex) {
+      console.log(ex.message);
+    });
   });
 
   it.skip("can clone with http", function() {
@@ -43,6 +45,7 @@ describe("Clone", function() {
 
     return Clone.clone(url, http, opts).then(function(repo) {
       assert.ok(repo instanceof Repository);
+      repo.free();
     });
   });
 
@@ -58,6 +61,8 @@ describe("Clone", function() {
 
     return Clone.clone(url, https, opts).then(function(repo) {
       assert.ok(repo instanceof Repository);
+      repo.stateCleanup();
+      repo.free();
     });
   });
 
@@ -76,6 +81,8 @@ describe("Clone", function() {
 
     return Clone.clone(url, ssh, opts).then(function(repo) {
       assert.ok(repo instanceof Repository);
+      repo.stateCleanup();
+      repo.free();
     });
   });
 
@@ -98,6 +105,8 @@ describe("Clone", function() {
 
     return Clone.clone(url, ssh, opts).then(function(repo) {
       assert.ok(repo instanceof Repository);
+      repo.stateCleanup();
+      repo.free();
     });
   });
 
@@ -113,6 +122,8 @@ describe("Clone", function() {
 
     return Clone.clone(url, git, opts).then(function(repo) {
       assert.ok(repo instanceof Repository);
+      repo.stateCleanup();
+      repo.free();
     });
   });
 
@@ -122,6 +133,8 @@ describe("Clone", function() {
 
     return Clone.clone(url, file).then(function(repo) {
       assert.ok(repo instanceof Repository);
+      repo.stateCleanup();
+      repo.free();
     });
   });
 
