Skip to content

Commit 526caf9

Browse files
committed
Modified the initial boolean RepositoryDirtyFlag to be an integer returning the count of UncommittedChanges
1 parent 95d2e67 commit 526caf9

File tree

35 files changed

+85
-64
lines changed

35 files changed

+85
-64
lines changed

docs/input/docs/more-info/variables.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ Running `GitVersion.exe` in your repo will show you what is available. For the
4040
"CommitsSinceVersionSource":1,
4141
"CommitsSinceVersionSourcePadded":"0001",
4242
"CommitDate":"2014-03-06",
43-
"RepositoryDirtyFlag": ""
43+
"UncommittedChanges": 0
4444
}
4545
```
4646

src/GitVersionCore.Tests/Core/GitVersionExecutorTests.cs

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ public void CacheFileExistsOnDisk()
148148
CommitsSinceVersionSource: 19
149149
CommitsSinceVersionSourcePadded: 0019
150150
CommitDate: 2015-11-10
151-
RepositoryDirtyFlag:
151+
UncommittedChanges: 0
152152
";
153153

154154
var stringBuilder = new StringBuilder();
@@ -209,6 +209,7 @@ public void CacheFileExistsOnDiskWhenOverrideConfigIsSpecifiedVersionShouldBeDyn
209209
CommitsSinceVersionSource: 19
210210
CommitsSinceVersionSourcePadded: 0019
211211
CommitDate: 2015-11-10
212+
UncommittedChanges: 0
212213
";
213214

214215
using var fixture = new EmptyRepositoryFixture();
@@ -295,7 +296,7 @@ public void ConfigChangeInvalidatesCache()
295296
CommitsSinceVersionSource: 19
296297
CommitsSinceVersionSourcePadded: 0019
297298
CommitDate: 2015-11-10
298-
RepositoryDirtyFlag:
299+
UncommittedChanges: 0
299300
";
300301

301302
using var fixture = new EmptyRepositoryFixture();
@@ -359,7 +360,7 @@ public void NoCacheBypassesCache()
359360
CommitsSinceVersionSource: 19
360361
CommitsSinceVersionSourcePadded: 0019
361362
CommitDate: 2015-11-10
362-
RepositoryDirtyFlag:
363+
UncommittedChanges: 0
363364
";
364365

365366
using var fixture = new EmptyRepositoryFixture();

src/GitVersionCore.Tests/Helpers/TestableVersionVariables.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,12 @@ public TestableVersionVariables(
1414
string weightedPreReleaseNumber = "", string informationalVersion = "", string commitDate = "",
1515
string nugetVersion = "", string nugetVersionV2 = "", string nugetPreReleaseTag = "",
1616
string nugetPreReleaseTagV2 = "", string versionSourceSha = "", string commitsSinceVersionSource = "",
17-
string commitsSinceVersionSourcePadded = "", string repositoryDirtyFlag = "") : base(
17+
string commitsSinceVersionSourcePadded = "", string uncommittedChanges = "") : base(
1818
major, minor, patch, buildMetaData, buildMetaDataPadded, fullBuildMetaData, branchName, escapedBranchName,
1919
sha, shortSha, majorMinorPatch, semVer, legacySemVer, legacySemVerPadded, fullSemVer,
2020
assemblySemVer, assemblySemFileVer, preReleaseTag, weightedPreReleaseNumber, preReleaseTagWithDash,
2121
preReleaseLabel, preReleaseNumber, informationalVersion, commitDate, nugetVersion, nugetVersionV2,
22-
nugetPreReleaseTag, nugetPreReleaseTagV2, versionSourceSha, commitsSinceVersionSource, commitsSinceVersionSourcePadded, repositoryDirtyFlag)
22+
nugetPreReleaseTag, nugetPreReleaseTagV2, versionSourceSha, commitsSinceVersionSource, commitsSinceVersionSourcePadded, uncommittedChanges)
2323
{
2424
}
2525
}

src/GitVersionCore.Tests/IntegrationTests/OtherScenarios.cs

Lines changed: 19 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
using System.Globalization;
12
using System.IO;
23
using System.Linq;
34
using GitTools.Testing;
@@ -97,23 +98,28 @@ public void DoNotBlowUpWhenDevelopAndFeatureBranchPointAtSameCommit()
9798
fixture.AssertFullSemver("1.1.0-alpha.1");
9899
}
99100

100-
[TestCase(true)]
101-
[TestCase(false)]
102-
public void HasDirtyFlagWhenUncommittedChangesAreInRepo(bool stageFile)
101+
[TestCase(true, 1)]
102+
[TestCase(false, 1)]
103+
[TestCase(true, 5)]
104+
[TestCase(false, 5)]
105+
public void HasDirtyFlagWhenUncommittedChangesAreInRepo(bool stageFile, int numberOfFiles)
103106
{
104107
using var fixture = new EmptyRepositoryFixture();
105108
fixture.Repository.MakeACommit();
106-
107-
var tempFile = Path.GetTempFileName();
108-
var repoFile = Path.Combine(fixture.RepositoryPath, Path.GetFileNameWithoutExtension(tempFile) + ".txt");
109-
File.Move(tempFile, repoFile);
110-
File.WriteAllText(repoFile, "Hello world");
111109

112-
if (stageFile)
113-
Commands.Stage(fixture.Repository, repoFile);
110+
for (int i = 0; i < numberOfFiles; i++)
111+
{
112+
var tempFile = Path.GetTempFileName();
113+
var repoFile = Path.Combine(fixture.RepositoryPath, Path.GetFileNameWithoutExtension(tempFile) + ".txt");
114+
File.Move(tempFile, repoFile);
115+
File.WriteAllText(repoFile, $"Hello world / testfile {i}");
116+
117+
if (stageFile)
118+
Commands.Stage(fixture.Repository, repoFile);
119+
}
114120

115121
var version = fixture.GetVersion();
116-
version.RepositoryDirtyFlag.ShouldBe("Dirty");
122+
version.UncommittedChanges.ShouldBe(numberOfFiles.ToString(CultureInfo.InvariantCulture));
117123
}
118124

119125
[Test]
@@ -123,7 +129,8 @@ public void NoDirtyFlagInCleanRepository()
123129
fixture.Repository.MakeACommit();
124130

125131
var version = fixture.GetVersion();
126-
version.RepositoryDirtyFlag.ShouldBe(null);
132+
var zero = 0;
133+
version.UncommittedChanges.ShouldBe(zero.ToString(CultureInfo.InvariantCulture));
127134
}
128135
}
129136
}

src/GitVersionCore.Tests/Model/CommitDateTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ public void CommitDateFormatTest(string format, string expectedOutcome)
2424
var formatValues = new SemanticVersionFormatValues(
2525
new SemanticVersion
2626
{
27-
BuildMetaData = new SemanticVersionBuildMetaData("950d2f830f5a2af12a6779a48d20dcbb02351f25", 0, "master", "3139d4eeb044f46057693473eacc2655b3b27e7d", "3139d4eeb", new DateTimeOffset(date, TimeSpan.Zero), false), // assume time zone is UTC
27+
BuildMetaData = new SemanticVersionBuildMetaData("950d2f830f5a2af12a6779a48d20dcbb02351f25", 0, "master", "3139d4eeb044f46057693473eacc2655b3b27e7d", "3139d4eeb", new DateTimeOffset(date, TimeSpan.Zero), 0), // assume time zone is UTC
2828

2929
},
3030
new EffectiveConfiguration(

src/GitVersionCore.Tests/VersionCalculation/Approved/JsonVersionBuilderTests.Json.approved.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,6 @@
2929
"VersionSourceSha":"versionSourceSha",
3030
"CommitsSinceVersionSource":5,
3131
"CommitsSinceVersionSourcePadded":"0005",
32-
"RepositoryDirtyFlag":"",
32+
"UncommittedChanges":0,
3333
"CommitDate":"2014-03-06"
3434
}

src/GitVersionCore.Tests/VersionCalculation/Approved/VariableProviderTests.ProvidesVariablesInContinuousDeliveryModeForFeatureBranch.approved.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,6 @@
2929
"VersionSourceSha":"versionSourceSha",
3030
"CommitsSinceVersionSource":5,
3131
"CommitsSinceVersionSourcePadded":"0005",
32-
"RepositoryDirtyFlag":"",
32+
"UncommittedChanges":0,
3333
"CommitDate":"2014-03-06"
3434
}

src/GitVersionCore.Tests/VersionCalculation/Approved/VariableProviderTests.ProvidesVariablesInContinuousDeliveryModeForFeatureBranchWithCustomAssemblyInfoFormat.approved.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,6 @@
2929
"VersionSourceSha":"versionSourceSha",
3030
"CommitsSinceVersionSource":5,
3131
"CommitsSinceVersionSourcePadded":"0005",
32-
"RepositoryDirtyFlag":"",
32+
"UncommittedChanges":0,
3333
"CommitDate":"2014-03-06"
3434
}

src/GitVersionCore.Tests/VersionCalculation/Approved/VariableProviderTests.ProvidesVariablesInContinuousDeliveryModeForPreRelease.approved.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,6 @@
2929
"VersionSourceSha":"versionSourceSha",
3030
"CommitsSinceVersionSource":5,
3131
"CommitsSinceVersionSourcePadded":"0005",
32-
"RepositoryDirtyFlag":"",
32+
"UncommittedChanges":0,
3333
"CommitDate":"2014-03-06"
3434
}

src/GitVersionCore.Tests/VersionCalculation/Approved/VariableProviderTests.ProvidesVariablesInContinuousDeliveryModeForPreReleaseWithPadding.approved.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,6 @@
2929
"VersionSourceSha":"versionSourceSha",
3030
"CommitsSinceVersionSource":5,
3131
"CommitsSinceVersionSourcePadded":"0005",
32-
"RepositoryDirtyFlag":"",
32+
"UncommittedChanges":0,
3333
"CommitDate":"2014-03-06"
3434
}

0 commit comments

Comments
 (0)