Skip to content

Commit 1017450

Browse files
HHobeckHHobeck
committed
Changing the following classes and define the properties as immutable: SemanticVersion, SemanticVersionPreReleaseTag and SemanticVersionBuildMetaData
1 parent 5ac47d5 commit 1017450

23 files changed

+398
-247
lines changed

‎src/GitVersion.BuildAgents.Tests/Agents/BitBucketPipelinesTests.cs‎

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -139,16 +139,17 @@ private void AssertVariablesAreWrittenToFile(string file)
139139
Minor = 2,
140140
Patch = 3,
141141
PreReleaseTag = "beta1",
142-
BuildMetaData = "5"
142+
BuildMetaData = new SemanticVersionBuildMetaData("5")
143+
{
144+
Sha = "f28807e615e9f06aec8a33c87780374e0c1f6fb8",
145+
CommitDate = new DateTimeOffset(2022, 4, 6, 16, 10, 59, TimeSpan.FromHours(10))
146+
}
143147
};
144148

145-
semanticVersion.BuildMetaData.CommitDate = new DateTimeOffset(2022, 4, 6, 16, 10, 59, TimeSpan.FromHours(10));
146-
semanticVersion.BuildMetaData.Sha = "f28807e615e9f06aec8a33c87780374e0c1f6fb8";
147-
148149
var configuration = new TestEffectiveConfiguration();
149150
var variableProvider = this.sp.GetRequiredService<IVariableProvider>();
150151

151-
var variables = variableProvider.GetVariablesFor(semanticVersion, configuration, false);
152+
var variables = variableProvider.GetVariablesFor(semanticVersion, configuration, null);
152153

153154
this.buildServer.WithPropertyFile(file);
154155

‎src/GitVersion.BuildAgents.Tests/Agents/BuildServerBaseTests.cs‎

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -29,15 +29,16 @@ public void BuildNumberIsFullSemVer()
2929
Minor = 2,
3030
Patch = 3,
3131
PreReleaseTag = "beta1",
32-
BuildMetaData = "5"
32+
BuildMetaData = new SemanticVersionBuildMetaData("5")
33+
{
34+
Sha = "commitSha",
35+
CommitDate = DateTimeOffset.Parse("2014-03-06 23:59:59Z")
36+
}
3337
};
3438

35-
semanticVersion.BuildMetaData.CommitDate = DateTimeOffset.Parse("2014-03-06 23:59:59Z");
36-
semanticVersion.BuildMetaData.Sha = "commitSha";
37-
3839
var configuration = new TestEffectiveConfiguration();
3940

40-
var variables = this.buildServer.GetVariablesFor(semanticVersion, configuration, false);
41+
var variables = this.buildServer.GetVariablesFor(semanticVersion, configuration, null);
4142
var buildAgent = this.sp.GetRequiredService<BuildAgent>();
4243
buildAgent.WriteIntegration(writes.Add, variables);
4344

‎src/GitVersion.BuildAgents.Tests/Agents/CodeBuildTests.cs‎

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -74,17 +74,18 @@ private void AssertVariablesAreWrittenToFile(string file)
7474
Minor = 2,
7575
Patch = 3,
7676
PreReleaseTag = "beta1",
77-
BuildMetaData = "5"
77+
BuildMetaData = new SemanticVersionBuildMetaData("5")
78+
{
79+
Sha = "commitSha",
80+
CommitDate = DateTimeOffset.Parse("2014-03-06 23:59:59Z")
81+
}
7882
};
7983

80-
semanticVersion.BuildMetaData.CommitDate = DateTimeOffset.Parse("2014-03-06 23:59:59Z");
81-
semanticVersion.BuildMetaData.Sha = "commitSha";
82-
8384
var configuration = new TestEffectiveConfiguration();
8485

8586
var variableProvider = this.sp.GetRequiredService<IVariableProvider>();
8687

87-
var variables = variableProvider.GetVariablesFor(semanticVersion, configuration, false);
88+
var variables = variableProvider.GetVariablesFor(semanticVersion, configuration, null);
8889

8990
this.buildServer.WithPropertyFile(file);
9091

‎src/GitVersion.BuildAgents.Tests/Agents/GitLabCiTests.cs‎

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -59,16 +59,17 @@ private void AssertVariablesAreWrittenToFile(string file)
5959
Minor = 2,
6060
Patch = 3,
6161
PreReleaseTag = "beta1",
62-
BuildMetaData = "5"
62+
BuildMetaData = new SemanticVersionBuildMetaData("5")
63+
{
64+
Sha = "commitSha",
65+
CommitDate = DateTimeOffset.Parse("2014-03-06 23:59:59Z")
66+
}
6367
};
6468

65-
semanticVersion.BuildMetaData.CommitDate = DateTimeOffset.Parse("2014-03-06 23:59:59Z");
66-
semanticVersion.BuildMetaData.Sha = "commitSha";
67-
6869
var configuration = new TestEffectiveConfiguration();
6970
var variableProvider = this.sp.GetRequiredService<IVariableProvider>();
7071

71-
var variables = variableProvider.GetVariablesFor(semanticVersion, configuration, false);
72+
var variables = variableProvider.GetVariablesFor(semanticVersion, configuration, null);
7273

7374
this.buildServer.WithPropertyFile(file);
7475

‎src/GitVersion.BuildAgents.Tests/Agents/JenkinsTests.cs‎

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -130,17 +130,18 @@ private void AssertVariablesAreWrittenToFile(string file)
130130
Minor = 2,
131131
Patch = 3,
132132
PreReleaseTag = "beta1",
133-
BuildMetaData = "5"
133+
BuildMetaData = new SemanticVersionBuildMetaData("5")
134+
{
135+
Sha = "commitSha",
136+
CommitDate = DateTimeOffset.Parse("2014-03-06 23:59:59Z")
137+
}
134138
};
135139

136-
semanticVersion.BuildMetaData.CommitDate = DateTimeOffset.Parse("2014-03-06 23:59:59Z");
137-
semanticVersion.BuildMetaData.Sha = "commitSha";
138-
139140
var configuration = new TestEffectiveConfiguration();
140141

141142
var variableProvider = this.sp.GetRequiredService<IVariableProvider>();
142143

143-
var variables = variableProvider.GetVariablesFor(semanticVersion, configuration, false);
144+
var variables = variableProvider.GetVariablesFor(semanticVersion, configuration, null);
144145

145146
this.buildServer.WithPropertyFile(file);
146147

‎src/GitVersion.Core.Tests/Extensions/GitToolsTestingExtensions.cs‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,9 @@ public static VersionVariables GetVersion(this RepositoryFixtureBase fixture, IG
9191
var context = contextOptions.Value;
9292

9393
var nextVersion = nextVersionCalculator.FindVersion();
94-
return variableProvider.GetVariablesFor(nextVersion.IncrementedVersion, nextVersion.Configuration, context.IsCurrentCommitTagged);
94+
return variableProvider.GetVariablesFor(
95+
nextVersion.IncrementedVersion, nextVersion.Configuration, context.CurrentCommitTaggedVersion
96+
);
9597
}
9698
catch (Exception)
9799
{

‎src/GitVersion.Core.Tests/IntegrationTests/OtherBranchScenarios.cs‎

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,26 @@ namespace GitVersion.Core.Tests.IntegrationTests;
77
[TestFixture]
88
public class OtherBranchScenarios : TestBase
99
{
10+
[Test]
11+
public void __Just_A_Test__()
12+
{
13+
var configuration = GitFlowConfigurationBuilder.New
14+
.WithBranch("develop", builder => builder.WithLabel("snapshot"))
15+
.WithBranch("release", builder => builder.WithLabel("rc"))
16+
.Build();
17+
18+
using var fixture = new EmptyRepositoryFixture();
19+
20+
fixture.MakeACommit();
21+
fixture.BranchTo("develop");
22+
fixture.MakeACommit();
23+
fixture.MakeATaggedCommit("0.1.2-snapshot.2");
24+
fixture.BranchTo("release/0.1.2");
25+
26+
// ✅ succeeds as expected
27+
fixture.AssertFullSemver("0.1.2-rc.1+0", configuration);
28+
}
29+
1030
[Test]
1131
public void CanTakeVersionFromReleaseBranch()
1232
{

‎src/GitVersion.Core.Tests/VersionCalculation/JsonVersionBuilderTests.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ public void Json()
2727
var serviceProvider = ConfigureServices();
2828

2929
var variableProvider = serviceProvider.GetRequiredService<IVariableProvider>();
30-
var variables = variableProvider.GetVariablesFor(semanticVersion, configuration, false);
30+
var variables = variableProvider.GetVariablesFor(semanticVersion, configuration, null);
3131
var json = variables.ToString();
3232
json.ShouldMatchApproved(c => c.SubFolder("Approved"));
3333
}

‎src/GitVersion.Core.Tests/VersionCalculation/SemanticVersionTests.cs‎

Lines changed: 13 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -131,16 +131,20 @@ public string ToStringWithFormatITests(int major, int minor, int patch, string p
131131
return semVer.ToString("i");
132132
}
133133

134-
private static SemanticVersion BuildSemVer(int major, int minor, int patch, string? preReleaseName, int? preReleaseVersion, int? buildCount, string? branchName = null, string? sha = null, string? otherMetadata = null)
134+
private static SemanticVersion BuildSemVer(int major, int minor, int patch, string? preReleaseName, int? preReleaseVersion,
135+
int? buildCount, string? branchName = null, string? sha = null, string? otherMetadata = null)
135136
{
136-
var semVer = new SemanticVersion(major, minor, patch);
137+
138+
SemanticVersionPreReleaseTag? preReleaseTag = null;
137139
if (preReleaseName != null)
138140
{
139-
semVer.PreReleaseTag = new SemanticVersionPreReleaseTag(preReleaseName, preReleaseVersion);
141+
preReleaseTag = new SemanticVersionPreReleaseTag(preReleaseName, preReleaseVersion);
140142
}
143+
144+
SemanticVersionBuildMetaData? buildMetaDate = null;
141145
if (buildCount.HasValue)
142146
{
143-
semVer.BuildMetaData = new SemanticVersionBuildMetaData
147+
buildMetaDate = new SemanticVersionBuildMetaData
144148
{
145149
CommitsSinceTag = buildCount.Value,
146150
Sha = sha,
@@ -149,6 +153,10 @@ private static SemanticVersion BuildSemVer(int major, int minor, int patch, stri
149153
};
150154
}
151155

152-
return semVer;
156+
return new SemanticVersion(major, minor, patch)
157+
{
158+
PreReleaseTag = preReleaseTag ?? new(),
159+
BuildMetaData = buildMetaDate ?? new()
160+
};
153161
}
154162
}

0 commit comments

Comments
 (0)