Don't allow non-trx logger on non-Windows OS

This failed consistently in CI and locally (WSL), so prevent users from having an unsupported combination.
devlooped · Aug 7, 2024 · 7d7b16f · 7d7b16f
1 parent 43f2da5
commit 7d7b16f
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/src/dotnet-retest/RetestCommand.cs b/src/dotnet-retest/RetestCommand.cs
@@ -45,18 +45,27 @@ public override async Task<int> ExecuteAsync(CommandContext context, RetestSetti
         string? path = null;
         var hastrx = false;
         var hasconsole = false;
+        var haslogger = false;
 
         new OptionSet
         {
             { "l|logger=", v =>
                 {
                     hastrx = v.StartsWith("trx");
                     hasconsole = v.StartsWith("console");
+                    haslogger = true;
                 }
             },
             { "results-directory=", v => path = v },
         }.Parse(args);
 
+        // In non-Windows OSes, the trx logger must be the only one if specified
+        if (haslogger && !hastrx && !RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
+        {
+            MarkupLine("[red]If a logger is specified, it can only be trx in non-Windows platforms.[/]");
+            return 1;
+        }
+
         var trx = new TrxCommand.TrxSettings
         {
             Path = path,