Replace verbosity flags approach with CliWrap output suppression for successful operations

Co-authored-by: thomhurst <[email protected]>

Author: Copilot

.github/actions/execute-pipeline/action.yml

@@ -30,7 +30,7 @@ runs:
   steps:
     - name: Run Pipeline
       shell: bash
-      run: dotnet run -c Release --verbosity quiet --categories ${{ inputs.categories }}
+      run: dotnet run -c Release --categories ${{ inputs.categories }}
       working-directory: "TUnit.Pipeline"
       env:
         ADMIN_TOKEN: ${{ inputs.admin-token }}

.github/workflows/dotnet.yml

@@ -64,7 +64,7 @@ jobs:
         run: npx playwright install
 
       - name: Build
-        run: dotnet build -c Release --verbosity quiet
+        run: dotnet build -c Release
 
       - name: Run Pipeline
         uses: ./.github/actions/execute-pipeline

TUnit.Engine.Tests/GlobalHooks.cs

@@ -5,9 +5,18 @@ public class GlobalHooks
     [Before(TestSession)]
     public static async Task BuildTestProject()
     {
-        await CliWrap.Cli.Wrap("dotnet")
+        var result = await CliWrap.Cli.Wrap("dotnet")
             .WithArguments(["build", "-c", "Release"])
             .WithWorkingDirectory(FileSystemHelpers.FindFile(x => x.Name == "TUnit.TestProject.csproj")!.DirectoryName!)
-            .ExecuteAsync();
+            .WithValidation(CliWrap.CommandResultValidation.None)
+            .ExecuteBufferedAsync();
+
+        // Only show output if the command failed
+        if (result.ExitCode != 0)
+        {
+            Console.WriteLine(result.StandardOutput);
+            Console.Error.WriteLine(result.StandardError);
+            throw new InvalidOperationException($"Build failed with exit code {result.ExitCode}");
+        }
     }
 }

TUnit.Pipeline/Modules/Abstract/TestBaseModule.cs

@@ -44,33 +44,18 @@ protected virtual IEnumerable<string> TestableFrameworks
 
     private DotNetRunOptions SetDefaults(DotNetRunOptions testOptions)
     {
-        // Add quiet verbosity to reduce output for successful test runs
-        var arguments = testOptions.Arguments?.ToList() ?? [];
-        
-        // Add TUnit verbosity control (only if not already specified)
-        if (!arguments.Any(arg => arg.StartsWith("--verbosity")))
-        {
-            arguments.AddRange(["--verbosity", "minimal"]);
-        }
+        // Removed --fail-fast to allow all tests to run even if some fail
 
         if (testOptions.EnvironmentVariables?.Any(x => x.Key == "NET_VERSION") != true)
         {
             testOptions = testOptions with
             {
-                Arguments = [.. arguments],
                 EnvironmentVariables = new Dictionary<string, string?>
                 {
                     ["NET_VERSION"] = testOptions.Framework,
                 }
             };
         }
-        else
-        {
-            testOptions = testOptions with
-            {
-                Arguments = [.. arguments]
-            };
-        }
 
         return testOptions;
     }

TUnit.Pipeline/Modules/PackTUnitFilesModule.cs

@@ -39,7 +39,6 @@ await context.DotNet()
                         ],
                         IncludeSource = project == Sourcy.DotNet.Projects.TUnit_Templates ? false : true,
                         Configuration = Configuration.Release,
-                        Verbosity = "quiet", // Reduce output for successful packs
                     }, cancellationToken);
 
             packedProjects.Add(new PackedProject(project.NameWithoutExtension, version.SemVer!));

TUnit.Pipeline/Program.cs

@@ -1,6 +1,7 @@
 using System.CommandLine;
 using Microsoft.Extensions.Configuration;
 using Microsoft.Extensions.DependencyInjection;
+using Microsoft.Extensions.Logging;
 using ModularPipelines.Extensions;
 using ModularPipelines.Host;
 using ModularPipelines.Options;