Add --skip-cloud-build-number to CLI and CloudBuildNumber to Cake

Author: MattKotsenas

src/Cake.GitVersioning/GitVersioningAliases.cs

@@ -87,6 +87,7 @@ public static void GitVersioningCloud(this ICakeContext cakeContext, string proj
                 settings.CISystem?.ToString(),
                 settings.AllVariables,
                 settings.CommonVariables,
+                settings.CloudBuildNumber,
                 settings.AdditionalVariables,
                 false);
         }

src/Cake.GitVersioning/GitVersioningCloudSettings.cs

@@ -37,6 +37,11 @@ public class GitVersioningCloudSettings
         /// </summary>
         public bool CommonVariables { get; set; }
 
+        /// <summary>
+        /// Gets or sets a value indicating whether to set the cloud build number.
+        /// </summary>
+        public bool CloudBuildNumber { get; set; }
+
         /// <summary>
         /// Gets additional cloud build variables to define.
         /// </summary>

src/NerdBank.GitVersioning/Commands/CloudCommand.cs

@@ -69,13 +69,16 @@ public enum CloudCommandError
     /// <param name="commonVars">
     /// Controls whether to define common version variables as cloud build variables.
     /// </param>
+    /// <param name="cloudBuildNumber">
+    /// Controls whether to emit the cloud build variable to set the build number.
+    /// </param>
     /// <param name="additionalVariables">
     /// Additional cloud build variables to define.
     /// </param>
     /// <param name="alwaysUseLibGit2">
     /// Force usage of LibGit2 for accessing the git repository.
     /// </param>
-    public void SetBuildVariables(string projectDirectory, IEnumerable<string> metadata, string version, string ciSystem, bool allVars, bool commonVars, IEnumerable<KeyValuePair<string, string>> additionalVariables, bool alwaysUseLibGit2)
+    public void SetBuildVariables(string projectDirectory, IEnumerable<string> metadata, string version, string ciSystem, bool allVars, bool commonVars, bool cloudBuildNumber, IEnumerable<KeyValuePair<string, string>> additionalVariables, bool alwaysUseLibGit2)
     {
         Requires.NotNull(projectDirectory, nameof(projectDirectory));
         Requires.NotNull(additionalVariables, nameof(additionalVariables));
@@ -137,7 +140,10 @@ public void SetBuildVariables(string projectDirectory, IEnumerable<string> metad
                 version = oracle.CloudBuildNumber;
             }
 
-            activeCloudBuild.SetCloudBuildNumber(version, this.stdout, this.stderr);
+            if (cloudBuildNumber)
+            {
+                activeCloudBuild.SetCloudBuildNumber(version, this.stdout, this.stderr);
+            }
 
             foreach (KeyValuePair<string, string> pair in variables)
             {

src/nbgv/Program.cs

@@ -194,6 +194,7 @@ private static Parser BuildCommandLine()
                 var ciSystem = new Option<string>(new[] { "--ci-system", "-s" }, "Force activation for a particular CI system. If not specified, auto-detection will be used. Supported values are: " + string.Join(", ", CloudProviderNames)).FromAmong(CloudProviderNames);
                 var allVars = new Option<bool>(new[] { "--all-vars", "-a" }, "Defines ALL version variables as cloud build variables, with a \"NBGV_\" prefix.");
                 var commonVars = new Option<bool>(new[] { "--common-vars", "-c" }, "Defines a few common version variables as cloud build variables, with a \"Git\" prefix (e.g. GitBuildVersion, GitBuildVersionSimple, GitAssemblyInformationalVersion).");
+                var skipCloudBuildNumber = new Option<bool>(new[] { "--skip-cloud-build-number", "-b" }, "Do not emit the cloud build variable to set the build number. This is useful when you want to set other cloud build variables but not the build number.");
                 var define = new Option<string[]>(new[] { "--define", "-d" }, () => Array.Empty<string>(), "Additional cloud build variables to define. Each should be in the NAME=VALUE syntax.")
                 {
                     Arity = ArgumentArity.OneOrMore,
@@ -207,10 +208,11 @@ private static Parser BuildCommandLine()
                     ciSystem,
                     allVars,
                     commonVars,
+                    skipCloudBuildNumber,
                     define,
                 };
 
-                cloud.SetHandler(OnCloudCommand, project, metadata, version, ciSystem, allVars, commonVars, define);
+                cloud.SetHandler(OnCloudCommand, project, metadata, version, ciSystem, allVars, commonVars, skipCloudBuildNumber, define);
             }
 
             Command prepareRelease;
@@ -656,7 +658,7 @@ private static Task<int> OnGetCommitsCommand(string project, bool quiet, string
             return Task.FromResult((int)ExitCodes.OK);
         }
 
-        private static Task<int> OnCloudCommand(string project, string[] metadata, string version, string ciSystem, bool allVars, bool commonVars, string[] define)
+        private static Task<int> OnCloudCommand(string project, string[] metadata, string version, string ciSystem, bool allVars, bool commonVars, bool skipCloudBuildNumber, string[] define)
         {
             string searchPath = GetSpecifiedOrCurrentDirectoryPath(project);
             if (!Directory.Exists(searchPath))
@@ -690,7 +692,7 @@ private static Task<int> OnCloudCommand(string project, string[] metadata, strin
             try
             {
                 var cloudCommand = new CloudCommand(Console.Out, Console.Error);
-                cloudCommand.SetBuildVariables(searchPath, metadata, version, ciSystem, allVars, commonVars, additionalVariables, AlwaysUseLibGit2);
+                cloudCommand.SetBuildVariables(searchPath, metadata, version, ciSystem, allVars, commonVars, !skipCloudBuildNumber, additionalVariables, AlwaysUseLibGit2);
             }
             catch (CloudCommand.CloudCommandException ex)
             {

test/Nerdbank.GitVersioning.Tests/CommandTests.cs

@@ -7,31 +7,34 @@
 
 public class CommandTests : RepoTestBase
 {
-    private readonly TestOutputHelperToTextWriterAdapter adapter;
-
     public CommandTests(ITestOutputHelper logger)
         : base(logger)
     {
-        this.adapter = new(logger);
     }
 
     // TODO: This is tightly coupled to the cloud build service. Can I use a FakeCloudBuildService instead?
-    [Theory]
-    [InlineData("VisualStudioTeamServices")]
-    [InlineData("TeamCity")]
-    [InlineData("Jenkins")]
-    public void CloudCommand_BuildNumber(string ciSystem)
+    [Theory, CombinatorialData]
+    public void CloudCommand_CloudBuildNumber([CombinatorialValues("VisualStudioTeamServices", "TeamCity", "Jenkins")] string ciSystem, bool setCloudBuildNumber)
     {
         var outWriter = new StringWriter();
         var errWriter = new StringWriter();
 
         var command = new CloudCommand(outWriter, errWriter);
 
-        command.SetBuildVariables(this.RepoPath, metadata: [], version: "1.2.3.4", ciSystem, allVars: false, commonVars: false, additionalVariables: [], alwaysUseLibGit2: false);
+        command.SetBuildVariables(this.RepoPath, metadata: [], version: "1.2.3.4", ciSystem, allVars: false, commonVars: false, setCloudBuildNumber, additionalVariables: [], alwaysUseLibGit2: false);
 
         outWriter.Flush();
         errWriter.Flush();
-        Assert.NotEmpty(outWriter.ToString());
+
+        if (setCloudBuildNumber)
+        {
+            Assert.NotEmpty(outWriter.ToString());
+        }
+        else
+        {
+            Assert.Empty(outWriter.ToString());
+        }
+
         Assert.Empty(errWriter.ToString());
     }