Skip to content

Allow head to move for dynamic repositories #1767

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
GeertvanHorrik merged 12 commits into from
Aug 12, 2019
Merged

Allow head to move for dynamic repositories #1767

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
b082835
Revive unit tests for dynamic repositories
GeertvanHorrik Aug 7, 2019
6005029
Never pass in null for authentication of dynamic repositories
GeertvanHorrik Aug 7, 2019
857643f
Shorten directory names
GeertvanHorrik Aug 7, 2019
72cedf0
Clean temp directory before running tests to fix build on Azure pipel…
GeertvanHorrik Aug 7, 2019
205c667
Fix directory creation and cleanup in tests
GeertvanHorrik Aug 7, 2019
b23408c
Add more dynamic repository tests
GeertvanHorrik Aug 7, 2019
a9c9227
Test cleanup
GeertvanHorrik Aug 7, 2019
32b4083
Fix expected output
GeertvanHorrik Aug 7, 2019
9fcc482
Allow move of repository head for dynamic repositories
GeertvanHorrik Aug 7, 2019
e3adfab
Run unit tests explicitly for dynamic repositories
GeertvanHorrik Aug 7, 2019
52e97ef
Fix build
GeertvanHorrik Aug 7, 2019
c408e4e
Ignore tests again, build agent is having issues
GeertvanHorrik Aug 7, 2019
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
Shorten directory names
  • Loading branch information
@GeertvanHorrik
GeertvanHorrik committed Aug 7, 2019
commit 857643f5ce1ccf68e84306041fb6500c901a2dc5
6 changes: 3 additions & 3 deletions src/GitVersionCore.Tests/DynamicRepositoryTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ public class DynamicRepositoryTests : TestBase
public void CreateTemporaryRepository()
{
// Note: we can't use guid because paths will be too long
workDirectory = Path.Combine(Path.GetTempPath(), "DynRepoTests");
workDirectory = Path.Combine(Path.GetTempPath(), "GV");
}


Expand All @@ -35,8 +35,8 @@ public void Cleanup()
public void FindsVersionInDynamicRepo(string name, string url, string targetBranch, string commitId, string expectedFullSemVer)
{
var root = Path.Combine(workDirectory, name);
var dynamicDirectory = Path.Combine(root, "dynamic");
var workingDirectory = Path.Combine(root, "working");
var dynamicDirectory = Path.Combine(root, "D"); // dynamic, keeping directory as short as possible
var workingDirectory = Path.Combine(root, "W"); // working, keeping directory as short as possible

//// Clear upfront
//if (Directory.Exists(root))
Expand Down