Enable ignored unit tests

GitTools · Mar 15, 2024 · fe27454 · fe27454
1 parent 61bcef9
commit fe27454
Show file tree

Hide file tree

Showing 21 changed files with 426 additions and 756 deletions.
diff --git a/docs/input/docs/reference/configuration.md b/docs/input/docs/reference/configuration.md
@@ -370,7 +370,7 @@ branches:
     - main
     pre-release-weight: 30000
   pull-request:
-    mode: ManualDeployment
+    mode: ContinuousDelivery
     label: PullRequest
     increment: Inherit
     label-number-pattern: '[/-](?<number>\d+)'
@@ -379,17 +379,16 @@ branches:
     - main
     pre-release-weight: 30000
   unknown:
-    mode: ManualDeployment
-    increment: Inherit
+    increment: Patch
     regex: (?<BranchName>.+)
+    prevent-increment:
+      when-current-commit-tagged: false
     source-branches:
     - main
-    - feature
-    - hotfix
-    - pull-request
+    pre-release-weight: 30000
 ignore:
   sha: []
-mode: ManualDeployment
+mode: ContinuousDelivery
 label: '{BranchName}'
 increment: Inherit
 prevent-increment:
@@ -403,6 +402,7 @@ regex: ''
 tracks-release-branches: false
 is-release-branch: false
 is-main-branch: false
+
 ```
 
 The details of the available options are as follows:

diff --git a/src/GitVersion.Configuration/SupportedWorkflows/TrunkBased/v1.yml b/src/GitVersion.Configuration/SupportedWorkflows/TrunkBased/v1.yml
@@ -45,7 +45,7 @@ branches:
     - main
     pre-release-weight: 30000
   pull-request:
-    mode: ManualDeployment
+    mode: ContinuousDelivery
     label: PullRequest
     increment: Inherit
     label-number-pattern: '[/-](?<number>\d+)'
@@ -54,17 +54,16 @@ branches:
     - main
     pre-release-weight: 30000
   unknown:
-    mode: ManualDeployment
-    increment: Inherit
+    increment: Patch
     regex: (?<BranchName>.+)
+    prevent-increment:
+      when-current-commit-tagged: false
     source-branches:
     - main
-    - feature
-    - hotfix
-    - pull-request
+    pre-release-weight: 30000
 ignore:
   sha: []
-mode: ManualDeployment
+mode: ContinuousDelivery
 label: '{BranchName}'
 increment: Inherit
 prevent-increment:

diff --git a/src/GitVersion.Configuration/TrunkBasedConfigurationBuilder.cs b/src/GitVersion.Configuration/TrunkBasedConfigurationBuilder.cs
@@ -26,7 +26,7 @@ private TrunkBasedConfigurationBuilder()
             VersionInBranchPattern = ConfigurationConstants.DefaultVersionInBranchPattern,
             TagPreReleaseWeight = ConfigurationConstants.DefaultTagPreReleaseWeight,
             UpdateBuildNumber = ConfigurationConstants.DefaultUpdateBuildNumber,
-            DeploymentMode = DeploymentMode.ManualDeployment,
+            DeploymentMode = DeploymentMode.ContinuousDelivery,
             RegularExpression = string.Empty,
             Label = ConfigurationConstants.BranchNamePlaceholder,
             Increment = IncrementStrategy.Inherit,
@@ -80,7 +80,7 @@ private TrunkBasedConfigurationBuilder()
         WithBranch(HotfixBranch.Name).WithConfiguration(new BranchConfiguration()
         {
             Increment = IncrementStrategy.Patch,
-            RegularExpression = HotfixBranch.RegexPattern,
+            RegularExpression = "^hotfix(es)?[/-](?<BranchName>.+)",
             SourceBranches =
             [
                 this.MainBranch.Name
@@ -96,7 +96,7 @@ private TrunkBasedConfigurationBuilder()
         {
             Increment = IncrementStrategy.Inherit,
             RegularExpression = PullRequestBranch.RegexPattern,
-            DeploymentMode = DeploymentMode.ManualDeployment,
+            DeploymentMode = DeploymentMode.ContinuousDelivery,
             SourceBranches =
             [
                 this.MainBranch.Name
@@ -108,15 +108,17 @@ private TrunkBasedConfigurationBuilder()
 
         WithBranch(UnknownBranch.Name).WithConfiguration(new BranchConfiguration
         {
+            Increment = IncrementStrategy.Patch,
             RegularExpression = UnknownBranch.RegexPattern,
-            DeploymentMode = DeploymentMode.ManualDeployment,
-            Increment = IncrementStrategy.Inherit,
             SourceBranches =
             [
-                this.MainBranch.Name,
-                this.FeatureBranch.Name,
-                this.PullRequestBranch.Name
-            ]
+                this.MainBranch.Name
+            ],
+            PreventIncrement = new PreventIncrementConfiguration()
+            {
+                WhenCurrentCommitTagged = false
+            },
+            PreReleaseWeight = 30000
         });
     }
 }
diff --git a/src/GitVersion.Core.Tests/Core/RepositoryStoreTests.cs b/src/GitVersion.Core.Tests/Core/RepositoryStoreTests.cs
@@ -213,7 +213,6 @@ public void GetBranchesContainingCommitThrowsDirectlyOnNullCommit()
     }
 
     [Test]
-    [Ignore("Needs more investigations.")]
     public void FindCommitBranchWasBranchedFromShouldReturnNullIfTheRemoteIsTheOnlySource()
     {
         using var fixture = new RemoteRepositoryFixture();

diff --git a/src/GitVersion.Core.Tests/IntegrationTests/ReleaseBranchScenarios.cs b/src/GitVersion.Core.Tests/IntegrationTests/ReleaseBranchScenarios.cs
@@ -452,17 +452,20 @@ public void CommitBeetweenMergeReleaseToDevelopShouldNotResetCount()
     }
 
     [Test]
-    [Ignore("Needs investigation")]
     public void ReleaseBranchShouldUseBranchNameVersionDespiteBumpInPreviousCommit()
     {
+        var configuration = GitFlowConfigurationBuilder.New
+            .WithSemanticVersionFormat(SemanticVersionFormat.Loose)
+            .Build();
+
         using var fixture = new EmptyRepositoryFixture();
         fixture.Repository.MakeATaggedCommit("1.0");
         fixture.Repository.MakeACommit("+semver:major");
         fixture.Repository.MakeACommit();
 
         fixture.BranchTo("release/2.0");
 
-        fixture.AssertFullSemver("2.0.0-beta.1+2");
+        fixture.AssertFullSemver("2.0.0-beta.1+2", configuration);
     }
 
     [Test]

diff --git a/...hAGitHubFlow+GivenAFeatureBranchWithOneCommitBranchedFromMainWhenCommitBTaggedAsStable.cs b/...hAGitHubFlow+GivenAFeatureBranchWithOneCommitBranchedFromMainWhenCommitBTaggedAsStable.cs
@@ -148,7 +148,6 @@ public string GetVersionWithNoLabelOnMain(
             return fixture!.GetVersion(trunkBased).FullSemVer;
         }
 
-        [Ignore("Enable if WithTakeIncrementedVersion(TakeIncrementedVersion.TakeAlwaysIncrementedVersion) feature has been implemented!")]
         [TestCase(IncrementStrategy.None, IncrementStrategy.None, null, ExpectedResult = "0.0.0-1+0")]
         [TestCase(IncrementStrategy.None, IncrementStrategy.Patch, null, ExpectedResult = "0.0.1-1+0")]
         [TestCase(IncrementStrategy.None, IncrementStrategy.Minor, null, ExpectedResult = "0.1.0-1+0")]
@@ -232,11 +231,11 @@ public string GetVersionWithNoLabelOnMain(
         [TestCase(IncrementStrategy.Major, IncrementStrategy.Patch, "{BranchName}", ExpectedResult = "0.0.1-foo.1+0")]
         [TestCase(IncrementStrategy.Major, IncrementStrategy.Minor, "{BranchName}", ExpectedResult = "0.1.0-foo.1+0")]
         [TestCase(IncrementStrategy.Major, IncrementStrategy.Major, "{BranchName}", ExpectedResult = "1.0.0-foo.1+0")]
-        public string GetVersionWithNoLabelOnMainAndTakeAlwaysIncrementedVersion(
+        public string GetVersionWithNoLabelAndPreventIncrementWhenCurrentCommitTaggedFalseOnMain(
              IncrementStrategy incrementOnMain, IncrementStrategy increment, string? label)
         {
             IGitVersionConfiguration trunkBased = TrunkBasedBuilder
-                //.WithTakeIncrementedVersion(TakeIncrementedVersion.TakeAlwaysIncrementedVersion)
+                .WithPreventIncrementWhenCurrentCommitTagged(false)
                 .WithBranch("main", _ => _.WithIncrement(incrementOnMain).WithLabel(null))
                 .WithBranch("feature", _ => _.WithIncrement(increment).WithLabel(label))
                 .Build();
@@ -358,7 +357,6 @@ public string GetVersionWithEmptyLabelOnMain(
             return fixture!.GetVersion(trunkBased).FullSemVer;
         }
 
-        [Ignore("Enable if WithTakeIncrementedVersion(TakeIncrementedVersion.TakeAlwaysIncrementedVersion) feature has been implemented!")]
         [TestCase(IncrementStrategy.None, IncrementStrategy.None, null, ExpectedResult = "0.0.0-1+0")]
         [TestCase(IncrementStrategy.None, IncrementStrategy.Patch, null, ExpectedResult = "0.0.1-1+0")]
         [TestCase(IncrementStrategy.None, IncrementStrategy.Minor, null, ExpectedResult = "0.1.0-1+0")]
@@ -442,11 +440,11 @@ public string GetVersionWithEmptyLabelOnMain(
         [TestCase(IncrementStrategy.Major, IncrementStrategy.Patch, "{BranchName}", ExpectedResult = "0.0.1-foo.1+0")]
         [TestCase(IncrementStrategy.Major, IncrementStrategy.Minor, "{BranchName}", ExpectedResult = "0.1.0-foo.1+0")]
         [TestCase(IncrementStrategy.Major, IncrementStrategy.Major, "{BranchName}", ExpectedResult = "1.0.0-foo.1+0")]
-        public string GetVersionWithEmptyLabelOnMainAndTakeAlwaysIncrementedVersion(
+        public string GetVersionWithEmptyLabelAndPreventIncrementWhenCurrentCommitTaggedFalseOnMain(
              IncrementStrategy incrementOnMain, IncrementStrategy increment, string? label)
         {
             IGitVersionConfiguration trunkBased = TrunkBasedBuilder
-                //.WithTakeIncrementedVersion(TakeIncrementedVersion.TakeAlwaysIncrementedVersion)
+                .WithPreventIncrementWhenCurrentCommitTagged(false)
                 .WithBranch("main", _ => _.WithIncrement(incrementOnMain).WithLabel(string.Empty))
                 .WithBranch("feature", _ => _.WithIncrement(increment).WithLabel(label))
                 .Build();
@@ -568,7 +566,6 @@ public string GetVersionWithLabelFooOnMain(
             return fixture!.GetVersion(trunkBased).FullSemVer;
         }
 
-        [Ignore("Enable if WithTakeIncrementedVersion(TakeIncrementedVersion.TakeAlwaysIncrementedVersion) feature has been implemented!")]
         [TestCase(IncrementStrategy.None, IncrementStrategy.None, null, ExpectedResult = "0.0.0-1+0")]
         [TestCase(IncrementStrategy.None, IncrementStrategy.Patch, null, ExpectedResult = "0.0.1-1+0")]
         [TestCase(IncrementStrategy.None, IncrementStrategy.Minor, null, ExpectedResult = "0.1.0-1+0")]
@@ -652,11 +649,11 @@ public string GetVersionWithLabelFooOnMain(
         [TestCase(IncrementStrategy.Major, IncrementStrategy.Patch, "{BranchName}", ExpectedResult = "0.0.1-foo.1+0")]
         [TestCase(IncrementStrategy.Major, IncrementStrategy.Minor, "{BranchName}", ExpectedResult = "0.1.0-foo.1+0")]
         [TestCase(IncrementStrategy.Major, IncrementStrategy.Major, "{BranchName}", ExpectedResult = "1.0.0-foo.1+0")]
-        public string GetVersionWithLabelFooOnMainAndTakeAlwaysIncrementedVersion(
+        public string GetVersionWithLabelFooAndPreventIncrementWhenCurrentCommitTaggedFalseOnMain(
              IncrementStrategy incrementOnMain, IncrementStrategy increment, string? label)
         {
             IGitVersionConfiguration trunkBased = TrunkBasedBuilder
-                //.WithTakeIncrementedVersion(TakeIncrementedVersion.TakeAlwaysIncrementedVersion)
+                .WithPreventIncrementWhenCurrentCommitTagged(false)
                 .WithBranch("main", _ => _.WithIncrement(incrementOnMain).WithLabel("foo"))
                 .WithBranch("feature", _ => _.WithIncrement(increment).WithLabel(label))
                 .Build();
@@ -778,7 +775,6 @@ public string GetVersionWithLabelBarOnMain(
             return fixture!.GetVersion(trunkBased).FullSemVer;
         }
 
-        [Ignore("Enable if WithTakeIncrementedVersion(TakeIncrementedVersion.TakeAlwaysIncrementedVersion) feature has been implemented!")]
         [TestCase(IncrementStrategy.None, IncrementStrategy.None, null, ExpectedResult = "0.0.0-1+0")]
         [TestCase(IncrementStrategy.None, IncrementStrategy.Patch, null, ExpectedResult = "0.0.1-1+0")]
         [TestCase(IncrementStrategy.None, IncrementStrategy.Minor, null, ExpectedResult = "0.1.0-1+0")]
@@ -862,11 +858,11 @@ public string GetVersionWithLabelBarOnMain(
         [TestCase(IncrementStrategy.Major, IncrementStrategy.Patch, "{BranchName}", ExpectedResult = "0.0.1-foo.1+0")]
         [TestCase(IncrementStrategy.Major, IncrementStrategy.Minor, "{BranchName}", ExpectedResult = "0.1.0-foo.1+0")]
         [TestCase(IncrementStrategy.Major, IncrementStrategy.Major, "{BranchName}", ExpectedResult = "1.0.0-foo.1+0")]
-        public string GetVersionWithLabelBarOnMainAndTakeAlwaysIncrementedVersion(
+        public string GetVersionWithLabelBarAndPreventIncrementWhenCurrentCommitTaggedFalseOnMain(
              IncrementStrategy incrementOnMain, IncrementStrategy increment, string? label)
         {
             IGitVersionConfiguration trunkBased = TrunkBasedBuilder
-                //.WithTakeIncrementedVersion(TakeIncrementedVersion.TakeAlwaysIncrementedVersion)
+                .WithPreventIncrementWhenCurrentCommitTagged(false)
                 .WithBranch("main", _ => _.WithIncrement(incrementOnMain).WithLabel("bar"))
                 .WithBranch("feature", _ => _.WithIncrement(increment).WithLabel(label))
                 .Build();

diff --git a/...dScenariosWithAGitHubFlow+GivenAFeatureBranchWithOneCommitWhenCommitTaggedAsPreRelease.cs b/...dScenariosWithAGitHubFlow+GivenAFeatureBranchWithOneCommitWhenCommitTaggedAsPreRelease.cs
@@ -58,7 +58,6 @@ public string GetVersion(IncrementStrategy increment, string? label)
             return fixture!.GetVersion(trunkBased).FullSemVer;
         }
 
-        [Ignore("Enable if WithTakeIncrementedVersion(TakeIncrementedVersion.TakeAlwaysIncrementedVersion) feature has been implemented!")]
         [TestCase(IncrementStrategy.None, null, ExpectedResult = "0.0.0-5+0")]
         [TestCase(IncrementStrategy.Patch, null, ExpectedResult = "0.0.0-5+0")]
         [TestCase(IncrementStrategy.Minor, null, ExpectedResult = "0.0.0-5+0")]
@@ -78,10 +77,10 @@ public string GetVersion(IncrementStrategy increment, string? label)
         [TestCase(IncrementStrategy.Patch, "bar", ExpectedResult = "0.0.1-bar.1+1")]
         [TestCase(IncrementStrategy.Minor, "bar", ExpectedResult = "0.1.0-bar.1+1")]
         [TestCase(IncrementStrategy.Major, "bar", ExpectedResult = "1.0.0-bar.1+1")]
-        public string GetVersionWithTakeAlwaysIncrementedVersion(IncrementStrategy increment, string? label)
+        public string GetVersionWithPreventIncrementWhenCurrentCommitTaggedFalse(IncrementStrategy increment, string? label)
         {
             IGitVersionConfiguration trunkBased = TrunkBasedBuilder
-                //.WithTakeIncrementedVersion(TakeIncrementedVersion.TakeAlwaysIncrementedVersion)
+                .WithPreventIncrementWhenCurrentCommitTagged(false)
                 .WithBranch("feature", _ => _.WithIncrement(increment).WithLabel(label))
                 .Build();
 

diff --git a/...enariosWithAGitHubFlow+GivenAFeatureBranchWithOneCommitWhenCommitTaggedAsPreReleaseBar.cs b/...enariosWithAGitHubFlow+GivenAFeatureBranchWithOneCommitWhenCommitTaggedAsPreReleaseBar.cs
@@ -58,11 +58,10 @@ public string GetVersion(IncrementStrategy increment, string? label)
             return fixture!.GetVersion(trunkBased).FullSemVer;
         }
 
-        [Ignore("Enable if WithTakeIncrementedVersion(TakeIncrementedVersion.TakeAlwaysIncrementedVersion) feature has been implemented!")]
-        [TestCase(IncrementStrategy.None, null, ExpectedResult = "0.0.0-bar")]
-        [TestCase(IncrementStrategy.Patch, null, ExpectedResult = "0.0.0-bar")]
-        [TestCase(IncrementStrategy.Minor, null, ExpectedResult = "0.0.0-bar")]
-        [TestCase(IncrementStrategy.Major, null, ExpectedResult = "0.0.0-bar")]
+        [TestCase(IncrementStrategy.None, null, ExpectedResult = "0.0.0-bar+0")]
+        [TestCase(IncrementStrategy.Patch, null, ExpectedResult = "0.0.0-bar+0")]
+        [TestCase(IncrementStrategy.Minor, null, ExpectedResult = "0.0.0-bar+0")]
+        [TestCase(IncrementStrategy.Major, null, ExpectedResult = "0.0.0-bar+0")]
 
         [TestCase(IncrementStrategy.None, "", ExpectedResult = "0.0.0-1+1")]
         [TestCase(IncrementStrategy.Patch, "", ExpectedResult = "0.0.1-1+1")]
@@ -74,14 +73,14 @@ public string GetVersion(IncrementStrategy increment, string? label)
         [TestCase(IncrementStrategy.Minor, "foo", ExpectedResult = "0.1.0-foo.1+1")]
         [TestCase(IncrementStrategy.Major, "foo", ExpectedResult = "1.0.0-foo.1+1")]
 
-        [TestCase(IncrementStrategy.None, "bar", ExpectedResult = "0.0.0-bar")]
-        [TestCase(IncrementStrategy.Patch, "bar", ExpectedResult = "0.0.0-bar")]
-        [TestCase(IncrementStrategy.Minor, "bar", ExpectedResult = "0.0.0-bar")]
-        [TestCase(IncrementStrategy.Major, "bar", ExpectedResult = "0.0.0-bar")]
-        public string GetVersionWithTakeAlwaysIncrementedVersion(IncrementStrategy increment, string? label)
+        [TestCase(IncrementStrategy.None, "bar", ExpectedResult = "0.0.0-bar+0")]
+        [TestCase(IncrementStrategy.Patch, "bar", ExpectedResult = "0.0.0-bar+0")]
+        [TestCase(IncrementStrategy.Minor, "bar", ExpectedResult = "0.0.0-bar+0")]
+        [TestCase(IncrementStrategy.Major, "bar", ExpectedResult = "0.0.0-bar+0")]
+        public string GetVersionWithPreventIncrementWhenCurrentCommitTaggedFalse(IncrementStrategy increment, string? label)
         {
             IGitVersionConfiguration trunkBased = TrunkBasedBuilder
-                //.WithTakeIncrementedVersion(TakeIncrementedVersion.TakeAlwaysIncrementedVersion)
+                .WithPreventIncrementWhenCurrentCommitTagged(false)
                 .WithBranch("feature", _ => _.WithIncrement(increment).WithLabel(label))
                 .Build();
 

diff --git a/...enariosWithAGitHubFlow+GivenAFeatureBranchWithOneCommitWhenCommitTaggedAsPreReleaseFoo.cs b/...enariosWithAGitHubFlow+GivenAFeatureBranchWithOneCommitWhenCommitTaggedAsPreReleaseFoo.cs
@@ -58,7 +58,6 @@ public string GetVersion(IncrementStrategy increment, string? label)
             return fixture!.GetVersion(trunkBased).FullSemVer;
         }
 
-        [Ignore("Enable if WithTakeIncrementedVersion(TakeIncrementedVersion.TakeAlwaysIncrementedVersion) feature has been implemented!")]
         [TestCase(IncrementStrategy.None, null, ExpectedResult = "0.0.0-foo.5+0")]
         [TestCase(IncrementStrategy.Patch, null, ExpectedResult = "0.0.0-foo.5+0")]
         [TestCase(IncrementStrategy.Minor, null, ExpectedResult = "0.0.0-foo.5+0")]
@@ -78,10 +77,10 @@ public string GetVersion(IncrementStrategy increment, string? label)
         [TestCase(IncrementStrategy.Patch, "bar", ExpectedResult = "0.0.1-bar.1+1")]
         [TestCase(IncrementStrategy.Minor, "bar", ExpectedResult = "0.1.0-bar.1+1")]
         [TestCase(IncrementStrategy.Major, "bar", ExpectedResult = "1.0.0-bar.1+1")]
-        public string GetVersionWithTakeAlwaysIncrementedVersion(IncrementStrategy increment, string? label)
+        public string GetVersionWithPreventIncrementWhenCurrentCommitTaggedFalse(IncrementStrategy increment, string? label)
         {
             IGitVersionConfiguration trunkBased = TrunkBasedBuilder
-                //.WithTakeIncrementedVersion(TakeIncrementedVersion.TakeAlwaysIncrementedVersion)
+                .WithPreventIncrementWhenCurrentCommitTagged(false)
                 .WithBranch("feature", _ => _.WithIncrement(increment).WithLabel(label))
                 .Build();
 

diff --git a/...BasedScenariosWithAGitHubFlow+GivenAFeatureBranchWithOneCommitWhenCommitTaggedAsStable.cs b/...BasedScenariosWithAGitHubFlow+GivenAFeatureBranchWithOneCommitWhenCommitTaggedAsStable.cs
@@ -58,7 +58,6 @@ public string GetVersion(IncrementStrategy increment, string? label)
             return fixture!.GetVersion(trunkBased).FullSemVer;
         }
 
-        [Ignore("Enable if WithTakeIncrementedVersion(TakeIncrementedVersion.TakeAlwaysIncrementedVersion) feature has been implemented!")]
         [TestCase(IncrementStrategy.None, null, ExpectedResult = "0.0.0-1+0")]
         [TestCase(IncrementStrategy.Patch, null, ExpectedResult = "0.0.1-1+0")]
         [TestCase(IncrementStrategy.Minor, null, ExpectedResult = "0.1.0-1+0")]
@@ -78,10 +77,10 @@ public string GetVersion(IncrementStrategy increment, string? label)
         [TestCase(IncrementStrategy.Patch, "bar", ExpectedResult = "0.0.1-bar.1+0")]
         [TestCase(IncrementStrategy.Minor, "bar", ExpectedResult = "0.1.0-bar.1+0")]
         [TestCase(IncrementStrategy.Major, "bar", ExpectedResult = "1.0.0-bar.1+0")]
-        public string GetVersionWithTakeAlwaysIncrementedVersion(IncrementStrategy increment, string? label)
+        public string GetVersionWithPreventIncrementWhenCurrentCommitTaggedFalse(IncrementStrategy increment, string? label)
         {
             IGitVersionConfiguration trunkBased = TrunkBasedBuilder
-                //.WithTakeIncrementedVersion(TakeIncrementedVersion.TakeAlwaysIncrementedVersion)
+                .WithPreventIncrementWhenCurrentCommitTagged(false)
                 .WithBranch("feature", _ => _.WithIncrement(increment).WithLabel(label))
                 .Build();