test: strip ANSI colour codes from stdout

Author: jamerst

AutoTag.CLI.Test/Helpers/CLIFixture.cs

@@ -1,5 +1,6 @@
 using System.Diagnostics;
 using System.Text;
+using System.Text.RegularExpressions;
 using AutoTag.CLI.Test.Helpers;
 using CliWrap;
 using CliWrap.Buffered;
@@ -10,10 +11,13 @@
 
 namespace AutoTag.CLI.Test.Helpers;
 
-public class CLIFixture(ITestContextAccessor context) : IAsyncLifetime
+public partial class CLIFixture(ITestContextAccessor context) : IAsyncLifetime
 {
     private string _cliPublishOutput = null!;
 
+    [GeneratedRegex(@"\x1b\[[0-9;]*m")]
+    private static partial Regex AnsiCodeRegex { get; }
+
     public async ValueTask InitializeAsync()
     {
         if (Debugger.IsAttached)
@@ -78,7 +82,7 @@ public ValueTask DisposeAsync()
 
             var appResult = await app.RunAsync(args);
 
-            return (appResult.Output, appResult.ExitCode);
+            return (RemoveAnsiColourCodes(appResult.Output), appResult.ExitCode);
         }
 
         if (string.IsNullOrEmpty(_cliPublishOutput))
@@ -88,11 +92,18 @@ public ValueTask DisposeAsync()
 
         var cmd = Cli.Wrap(_cliPublishOutput)
             .WithArguments(args)
-            .WithValidation(CommandResultValidation.None)
-            .WithEnvironmentVariables(env => env.Set("TERM", "dumb"));
+            .WithValidation(CommandResultValidation.None);
 
         var result = await cmd.ExecuteBufferedAsync();
 
-        return (result.StandardOutput, result.ExitCode);
+        if (!result.IsSuccess)
+        {
+            context.Current.SendDiagnosticMessage($"CLI returned non-zero exit code ({result.ExitCode})");
+            context.Current.SendDiagnosticMessage(result.StandardOutput);
+        }
+
+        return (RemoveAnsiColourCodes(result.StandardOutput), result.ExitCode);
     }
+
+    private static string RemoveAnsiColourCodes(string input) => AnsiCodeRegex.Replace(input, "");
 }

AutoTag.CLI.Test/ProcessTests.cs

@@ -38,11 +38,6 @@ public async Task Should_ProcessFilesWithRenamePattern()
             "--rename-subs"
         );
 
-        if (exitCode != 0)
-        {
-            TestContext.Current.SendDiagnosticMessage(stdout);
-        }
-
         exitCode.Should().Be(0);
 
         AssertFile(