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

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
Prev Previous commit
Next Next commit
Add unique directories to each clone test
  • Loading branch information
John Haley committed Feb 26, 2015
commit 9141d595f872887b64092d0e56a90c13a615f3bb
26 changes: 19 additions & 7 deletions test/tests/clone.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ describe("Clone", function() {
this.timeout(30000);

beforeEach(function() {
return fse.remove(clonePath).catch(function(err) {
return fse.remove(this.clonePath).catch(function(err) {
console.log(err);

throw err;
Expand Down Expand Up @@ -49,7 +49,9 @@ describe("Clone", function() {
}
};

return Clone.clone(url, clonePath, opts).then(function(repo) {
test.clonePath = local("../repos/http");

return Clone.clone(url, test.clonePath, opts).then(function(repo) {
assert.ok(repo instanceof Repository);
test.repository = repo;
});
Expand All @@ -66,7 +68,9 @@ describe("Clone", function() {
}
};

return Clone.clone(url, clonePath, opts).then(function(repo) {
test.clonePath = local("../repos/https");

return Clone.clone(url, test.clonePath, opts).then(function(repo) {
assert.ok(repo instanceof Repository);
test.repository = repo;
});
Expand All @@ -86,7 +90,9 @@ describe("Clone", function() {
}
};

return Clone.clone(url, clonePath, opts).then(function(repo) {
test.clonePath = local("../repos/ssh");

return Clone.clone(url, test.clonePath, opts).then(function(repo) {
assert.ok(repo instanceof Repository);
test.repository = repo;
});
Expand All @@ -110,7 +116,9 @@ describe("Clone", function() {
}
};

return Clone.clone(url, clonePath, opts).then(function(repo) {
test.clonePath = local("../repos/sshManual");

return Clone.clone(url, test.clonePath, opts).then(function(repo) {
assert.ok(repo instanceof Repository);
test.repository = repo;
});
Expand All @@ -126,7 +134,9 @@ describe("Clone", function() {
}
};

return Clone.clone(url, clonePath, opts).then(function(repo) {
test.clonePath = local("../repos/git");

return Clone.clone(url, test.clonePath, opts).then(function(repo) {
assert.ok(repo instanceof Repository);
});
});
Expand All @@ -136,7 +146,9 @@ describe("Clone", function() {
var prefix = process.platform === "win32" ? "" : "file://";
var url = prefix + local("../repos/empty");

return Clone.clone(url, clonePath).then(function(repo) {
test.clonePath = local("../repos/filesystem");

return Clone.clone(url, test.clonePath).then(function(repo) {
assert.ok(repo instanceof Repository);
test.repository = repo;
});
Expand Down