Skip to content

Commit 952659f

Browse files
committed
Fixed failing tests
1 parent 4a4f2a5 commit 952659f

File tree

5 files changed

+24
-39
lines changed

5 files changed

+24
-39
lines changed

GitVersionCore.Tests/TestEffectiveConfiguration.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ namespace GitVersionCore.Tests
55
public class TestEffectiveConfiguration : EffectiveConfiguration
66
{
77
public TestEffectiveConfiguration(
8-
AssemblyVersioningScheme assemblyVersioningScheme = AssemblyVersioningScheme.MajorMinorPatchMetadata,
8+
AssemblyVersioningScheme assemblyVersioningScheme = AssemblyVersioningScheme.MajorMinorPatch,
99
VersioningMode versioningMode = VersioningMode.ContinuousDelivery,
1010
string gitTagPrefix = "v",
1111
string tag = "",

GitVersionCore/Configuration/Config.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,13 +12,13 @@ public Config()
1212
{
1313
AssemblyVersioningScheme = AssemblyVersioningScheme.MajorMinorPatch;
1414
TagPrefix = "[vV]";
15-
VersioningMode = VersioningMode.ContinuousDelivery;
15+
VersioningMode = GitVersion.VersioningMode.ContinuousDelivery;
1616
Branches["release[/-]"] = new BranchConfig { Tag = "beta" };
1717
Branches["hotfix[/-]"] = new BranchConfig { Tag = "beta" };
1818
Branches["develop"] = new BranchConfig
1919
{
2020
Tag = "unstable",
21-
VersioningMode = VersioningMode.ContinuousDeployment
21+
VersioningMode = GitVersion.VersioningMode.ContinuousDeployment
2222
};
2323
}
2424

GitVersionCore/GitVersionContext.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ public GitVersionContext(IRepository repository, Branch currentBranch, Config co
2626
IsContextForTrackedBranchesOnly = isForTrackingBranchOnly;
2727

2828
if (currentBranch == null)
29-
return;
29+
throw new InvalidOperationException("Need a branch to operate on");
3030

3131
CurrentCommit = currentBranch.Tip;
3232

GitVersionTask.Tests/UpdateAssemblyInfoTests.cs

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -101,22 +101,6 @@ public void StandardExecutionMode_CanAcceptAssemblyVersioningSchemes(string asse
101101
task.InnerExecute();
102102
}
103103

104-
[Test]
105-
public void StandardExecutionMode_ThrowsUponUnexpectedAssemblyVersioningSchemes()
106-
{
107-
var repoPath = CheckoutLocal(ASBMTestRepoWorkingDirPath, "refs/heads/master");
108-
109-
var task = new UpdateAssemblyInfo
110-
{
111-
BuildEngine = new MockBuildEngine(),
112-
SolutionDirectory = repoPath,
113-
// TODO AssemblyVersioningScheme = "Boom"
114-
};
115-
116-
var exception = Assert.Throws<WarningException>(() => task.InnerExecute());
117-
Assert.AreEqual("Unexpected assembly versioning scheme 'Boom'.", exception.Message);
118-
}
119-
120104
[SetUp]
121105
public void SetUp()
122106
{

GitVersionTask.Tests/VersionOnMasterFinderTests.cs

Lines changed: 20 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -19,31 +19,32 @@ public void Should_find_previous_commit_that_was_at_least_a_minor_bump()
1919

2020
const string sha = "deadbeefdeadbeefdeadbeefdeadbeefdeadbeef";
2121

22+
var mockBranch = new MockBranch("master")
23+
{
24+
new MockMergeCommit(new ObjectId(sha))
25+
{
26+
MessageEx = "Merge branch 'hotfix-0.3.0'",
27+
CommitterEx = signature
28+
},
29+
new MockMergeCommit
30+
{
31+
MessageEx = "Merge branch 'hotfix-0.3.1'",
32+
CommitterEx = signature,
33+
},
34+
new MockMergeCommit
35+
{
36+
MessageEx = "Merge branch 'hotfix-0.2.0'",
37+
CommitterEx = signature
38+
},
39+
};
2240
var repository = new MockRepository
2341
{
2442
Branches = new MockBranchCollection
2543
{
26-
new MockBranch("master")
27-
{
28-
new MockMergeCommit(new ObjectId(sha))
29-
{
30-
MessageEx = "Merge branch 'hotfix-0.3.0'",
31-
CommitterEx = signature
32-
},
33-
new MockMergeCommit
34-
{
35-
MessageEx = "Merge branch 'hotfix-0.3.1'",
36-
CommitterEx = signature,
37-
},
38-
new MockMergeCommit
39-
{
40-
MessageEx = "Merge branch 'hotfix-0.2.0'",
41-
CommitterEx = signature
42-
},
43-
},
44+
mockBranch,
4445
}
4546
};
46-
var version = finder.Execute(new GitVersionContext(repository, null, new Config()), 1.Seconds().Ago());
47+
var version = finder.Execute(new GitVersionContext(repository, mockBranch, new Config()), 1.Seconds().Ago());
4748
ObjectApprover.VerifyWithJson(version);
4849
}
4950
}

0 commit comments

Comments
 (0)