GH-2127 - allow json and buildserver output at the same time

Author: arturcic

src/GitVersionCore/Arguments.cs

@@ -8,6 +8,10 @@ namespace GitVersion
 {
     public class Arguments
     {
+        public Arguments()
+        {
+            Output.Add(OutputType.Json);
+        }
         public AuthenticationInfo Authentication;
 
         public Config OverrideConfig;
@@ -44,7 +48,7 @@ public class Arguments
         public bool NoCache;
         public bool NoNormalize;
 
-        public OutputType Output = OutputType.Json;
+        public ISet<OutputType> Output = new HashSet<OutputType>();
         public Verbosity Verbosity = Verbosity.Normal;
 
         public bool UpdateAssemblyInfo;

src/GitVersionExe.Tests/ArgumentParserTests.cs

@@ -135,28 +135,36 @@ public void UnknownOutputShouldThrow()
         public void OutputDefaultsToJson()
         {
             var arguments = argumentParser.ParseArguments("targetDirectoryPath");
-            arguments.Output.ShouldBe(OutputType.Json);
+            arguments.Output.ShouldContain(OutputType.Json);
         }
 
         [Test]
         public void OutputJsonCanBeParsed()
         {
             var arguments = argumentParser.ParseArguments("targetDirectoryPath -output json");
-            arguments.Output.ShouldBe(OutputType.Json);
+            arguments.Output.ShouldContain(OutputType.Json);
         }
 
         [Test]
         public void OutputBuildserverCanBeParsed()
         {
             var arguments = argumentParser.ParseArguments("targetDirectoryPath -output buildserver");
-            arguments.Output.ShouldBe(OutputType.BuildServer);
+            arguments.Output.ShouldContain(OutputType.BuildServer);
+        }
+
+        [Test]
+        public void OutputBuildserverAndJsonCanBeParsed()
+        {
+            var arguments = argumentParser.ParseArguments("targetDirectoryPath -output buildserver -output json");
+            arguments.Output.ShouldContain(OutputType.BuildServer);
+            arguments.Output.ShouldContain(OutputType.Json);
         }
 
         [Test]
         public void MultipleArgsAndFlag()
         {
             var arguments = argumentParser.ParseArguments("targetDirectoryPath -output buildserver -updateAssemblyInfo");
-            arguments.Output.ShouldBe(OutputType.BuildServer);
+            arguments.Output.ShouldContain(OutputType.BuildServer);
         }
 
         [Test]

src/GitVersionExe/ArgumentParser.cs

@@ -269,7 +269,7 @@ public Arguments ParseArguments(string[] commandLineArguments)
                         throw new WarningException($"Value '{value}' cannot be parsed as output type, please use 'json' or 'buildserver'");
                     }
 
-                    arguments.Output = outputType;
+                    arguments.Output.Add(outputType);
                     continue;
                 }
 

src/GitVersionExe/ExecCommand.cs

@@ -40,32 +40,32 @@ public void Execute()
 
             var arguments = options.Value;
 
-            switch (arguments.Output)
+            if (arguments.Output.Contains(OutputType.BuildServer))
             {
-                case OutputType.BuildServer:
-                    var buildServer = buildServerResolver.Resolve();
-                    buildServer?.WriteIntegration(Console.WriteLine, variables);
-
-                    break;
-                case OutputType.Json:
-                    switch (arguments.ShowVariable)
-                    {
-                        case null:
-                            Console.WriteLine(variables.ToString());
-                            break;
-
-                        default:
-                            if (!variables.TryGetValue(arguments.ShowVariable, out var part))
-                            {
-                                throw new WarningException($"'{arguments.ShowVariable}' variable does not exist");
-                            }
-                            Console.WriteLine(part);
-                            break;
-                    }
-
-                    break;
-                default:
-                    throw new ArgumentOutOfRangeException();
+                var buildServer = buildServerResolver.Resolve();
+                buildServer?.WriteIntegration(Console.WriteLine, variables);
+            }
+            if (arguments.Output.Contains(OutputType.Json))
+            {
+                switch (arguments.ShowVariable)
+                {
+                    case null:
+                        Console.WriteLine(variables.ToString());
+                        break;
+
+                    default:
+                        if (!variables.TryGetValue(arguments.ShowVariable, out var part))
+                        {
+                            throw new WarningException($"'{arguments.ShowVariable}' variable does not exist");
+                        }
+
+                        Console.WriteLine(part);
+                        break;
+                }
+            }
+            else
+            {
+                throw new ArgumentOutOfRangeException();
             }
 
             if (arguments.UpdateWixVersionFile)

src/GitVersionExe/GitVersionExecutor.cs

@@ -73,12 +73,12 @@ private int VerifyArgumentsAndRun(Arguments arguments)
                 if (arguments.Diag)
                 {
                     arguments.NoCache = true;
-                    arguments.Output = OutputType.BuildServer;
+                    arguments.Output.Add(OutputType.BuildServer);
                 }
 
                 if (!string.IsNullOrEmpty(arguments.Proj) || !string.IsNullOrEmpty(arguments.Exec))
                 {
-                    arguments.Output = OutputType.BuildServer;
+                    arguments.Output.Add(OutputType.BuildServer);
                 }
 
                 var buildServer = buildServerResolver.Resolve();
@@ -154,7 +154,7 @@ private void VerifyConfiguration()
 
         private static void ConfigureLogging(Arguments arguments, ILog log)
         {
-            if (arguments.Output == OutputType.BuildServer || arguments.LogFilePath == "console" || arguments.Init)
+            if (arguments.Output.Contains(OutputType.BuildServer) || arguments.LogFilePath == "console" || arguments.Init)
             {
                 log.AddLogAppender(new ConsoleAppender());
             }