Skip to content

Attempt to fix Windows file locking bug #425

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
tbranyen merged 12 commits into from
Feb 26, 2015
Merged

Attempt to fix Windows file locking bug #425

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
749826c
Attempt to fix Windows file locking bug
tbranyen Feb 25, 2015
aea1ae7
Change to a single clone path
tbranyen Feb 25, 2015
0e80c58
Ensure gc exists
tbranyen Feb 25, 2015
be57dcb
Trying to fix issue
tbranyen Feb 25, 2015
7eb2108
Add ClearValue method to structs and classes
Feb 25, 2015
98119a6
Cleanup repo after each clone test
Feb 25, 2015
9141d59
Add unique directories to each clone test
Feb 25, 2015
762fdf3
Fix null ref error in beforeEach
Feb 25, 2015
90d580a
Fix test var null ref
Feb 25, 2015
796c09f
Add repo from git clone to test instance so it can be cleaned up
Feb 25, 2015
e19e8ea
Added global `afterEach`, standardized repo attach
tbranyen Feb 25, 2015
342b6e6
Converted to `beforeEach` for all tests
tbranyen Feb 25, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Attempt to fix Windows file locking bug
  • Loading branch information
@tbranyen
tbranyen authored and John Haley committed Feb 26, 2015
commit 749826c81799fc5b00a0ed4a18d98e67f27f1eaa
3 changes: 0 additions & 3 deletions generate/input/descriptor.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1431,9 +1431,6 @@
"git_repository_fetchhead_foreach": {
"ignore": true
},
"git_repository_free": {
"ignore": true
},
"git_repository_hashfile": {
"ignore": true
},
Expand Down
4 changes: 2 additions & 2 deletions generate/scripts/helpers.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down
15 changes: 14 additions & 1 deletion test/tests/clone.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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() {
Expand All @@ -43,6 +45,7 @@ describe("Clone", function() {

return Clone.clone(url, http, opts).then(function(repo) {
assert.ok(repo instanceof Repository);
repo.free();
});
});

Expand All @@ -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();
});
});

Expand All @@ -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();
});
});

Expand All @@ -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();
});
});

Expand All @@ -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();
});
});

Expand All @@ -122,6 +133,8 @@ describe("Clone", function() {

return Clone.clone(url, file).then(function(repo) {
assert.ok(repo instanceof Repository);
repo.stateCleanup();
repo.free();
});
});

Expand Down