Format

GitHubActionsTestLogger/CliOptionsProvider.cs

@@ -1,14 +1,14 @@
 using System.Collections.Generic;
 using System.Linq;
 using System.Threading.Tasks;
-
 using Microsoft.Testing.Platform.CommandLine;
 using Microsoft.Testing.Platform.Extensions;
 using Microsoft.Testing.Platform.Extensions.CommandLine;
 
 namespace GitHubActionsTestLogger;
 
-internal sealed class CliOptionsProvider(GitHubTestReporterExtension extension) : ICommandLineOptionsProvider
+internal sealed class CliOptionsProvider(GitHubTestReporterExtension extension)
+    : ICommandLineOptionsProvider
 {
     public const string ReportGitHubOption = "report-github";
     public const string ReportGitHubSummaryOption = "report-github-summary";
@@ -29,57 +29,94 @@ public static class ReportGitHubSummaryArguments
 
     public IReadOnlyCollection<CommandLineOption> GetCommandLineOptions() =>
         [
-            new(ReportGitHubOption, "Reports test run information to GitHub Actions", ArgumentArity.Zero, isHidden: false),
+            new(
+                ReportGitHubOption,
+                "Reports test run information to GitHub Actions",
+                ArgumentArity.Zero,
+                isHidden: false
+            ),
             // TODO: Do you prefer multiple zero argument options or a single option with argument?
-            new(ReportGitHubSummaryOption, "Defines the information to include in the summary", ArgumentArity.OneOrMore, isHidden: false),
-            new(ReportGitHubTitleOption, "Defines the annotation title format used for reporting test failures", ArgumentArity.ExactlyOne, isHidden: false),
-            new(ReportGitHubMessageOption, "Defines the annotation message format used for reporting test failures", ArgumentArity.ExactlyOne, isHidden: false),
+            new(
+                ReportGitHubSummaryOption,
+                "Defines the information to include in the summary",
+                ArgumentArity.OneOrMore,
+                isHidden: false
+            ),
+            new(
+                ReportGitHubTitleOption,
+                "Defines the annotation title format used for reporting test failures",
+                ArgumentArity.ExactlyOne,
+                isHidden: false
+            ),
+            new(
+                ReportGitHubMessageOption,
+                "Defines the annotation message format used for reporting test failures",
+                ArgumentArity.ExactlyOne,
+                isHidden: false
+            ),
         ];
 
     public Task<bool> IsEnabledAsync() => extension.IsEnabledAsync();
 
     // This method is called once after all options are parsed and is used to validate the combination of options.
-    public Task<ValidationResult> ValidateCommandLineOptionsAsync(ICommandLineOptions commandLineOptions)
+    public Task<ValidationResult> ValidateCommandLineOptionsAsync(
+        ICommandLineOptions commandLineOptions
+    )
     {
         // We just want to validate that the sub options are set only if the main option is set.
         if (commandLineOptions.IsOptionSet(ReportGitHubOption))
         {
             return ValidationResult.ValidTask;
         }
 
-        if (commandLineOptions.IsOptionSet(ReportGitHubSummaryOption)
+        if (
+            commandLineOptions.IsOptionSet(ReportGitHubSummaryOption)
             || commandLineOptions.IsOptionSet(ReportGitHubTitleOption)
-            || commandLineOptions.IsOptionSet(ReportGitHubMessageOption))
+            || commandLineOptions.IsOptionSet(ReportGitHubMessageOption)
+        )
         {
-            return ValidationResult.InvalidTask("The options 'report-github-summary', 'report-github-title', and 'report-github-message' can only be used if 'report-github' is set.");
+            return ValidationResult.InvalidTask(
+                "The options 'report-github-summary', 'report-github-title', and 'report-github-message' can only be used if 'report-github' is set."
+            );
         }
 
         return ValidationResult.ValidTask;
     }
 
     // This method is called once per option declared and is used to validate the arguments of the given option.
     // The arity of the option is checked before this method is called.
-    public Task<ValidationResult> ValidateOptionArgumentsAsync(CommandLineOption commandOption, string[] arguments)
+    public Task<ValidationResult> ValidateOptionArgumentsAsync(
+        CommandLineOption commandOption,
+        string[] arguments
+    )
     {
         if (commandOption.Name == ReportGitHubSummaryOption)
         {
             if (arguments.Length > 3)
             {
-                return ValidationResult.InvalidTask($"The option '{ReportGitHubSummaryOption}' can have at most 3 arguments.");
+                return ValidationResult.InvalidTask(
+                    $"The option '{ReportGitHubSummaryOption}' can have at most 3 arguments."
+                );
             }
 
             if (arguments.Distinct().Count() != arguments.Length)
             {
-                return ValidationResult.InvalidTask($"The option '{ReportGitHubSummaryOption}' cannot have duplicate arguments.");
+                return ValidationResult.InvalidTask(
+                    $"The option '{ReportGitHubSummaryOption}' cannot have duplicate arguments."
+                );
             }
 
             for (int i = 0; i < arguments.Length; i++)
             {
-                if (arguments[i] != ReportGitHubSummaryArguments.IncludePassedTests
+                if (
+                    arguments[i] != ReportGitHubSummaryArguments.IncludePassedTests
                     && arguments[i] != ReportGitHubSummaryArguments.IncludeSkippedTests
-                    && arguments[i] != ReportGitHubSummaryArguments.IncludeNotFoundTests)
+                    && arguments[i] != ReportGitHubSummaryArguments.IncludeNotFoundTests
+                )
                 {
-                    return ValidationResult.InvalidTask($"The option '{ReportGitHubSummaryOption}' can only have the arguments '{ReportGitHubSummaryArguments.IncludePassedTests}', '{ReportGitHubSummaryArguments.IncludeSkippedTests}', and '{ReportGitHubSummaryArguments.IncludeNotFoundTests}'.");
+                    return ValidationResult.InvalidTask(
+                        $"The option '{ReportGitHubSummaryOption}' can only have the arguments '{ReportGitHubSummaryArguments.IncludePassedTests}', '{ReportGitHubSummaryArguments.IncludeSkippedTests}', and '{ReportGitHubSummaryArguments.IncludeNotFoundTests}'."
+                    );
                 }
             }
         }

GitHubActionsTestLogger/GitHubReportExtensions.cs

@@ -14,8 +14,12 @@ public static void AddGitHubReportProvider(this ITestApplicationBuilder testAppl
     {
         var extension = new GitHubTestReporterExtension();
 
-        var compositeExtension = new CompositeExtensionFactory<GitHubTestReporter>(serviceProvider => 
-            new GitHubTestReporter(extension, serviceProvider.GetCommandLineOptions()));
+        var compositeExtension = new CompositeExtensionFactory<GitHubTestReporter>(
+            serviceProvider => new GitHubTestReporter(
+                extension,
+                serviceProvider.GetCommandLineOptions()
+            )
+        );
         testApplicationBuilder.TestHost.AddDataConsumer(compositeExtension);
         testApplicationBuilder.TestHost.AddTestSessionLifetimeHandle(compositeExtension);
 

GitHubActionsTestLogger/GitHubTestReporter.cs

@@ -1,7 +1,6 @@
 using System;
 using System.Threading;
 using System.Threading.Tasks;
-
 using Microsoft.Testing.Platform.CommandLine;
 using Microsoft.Testing.Platform.Extensions.Messages;
 using Microsoft.Testing.Platform.Extensions.TestHost;
@@ -12,34 +11,46 @@ namespace GitHubActionsTestLogger;
 /// <summary>
 /// A Microsoft.Testing.Platform extension that reports test run information to GitHub Actions.
 /// </summary>
-internal sealed class GitHubTestReporter(GitHubTestReporterExtension extension, ICommandLineOptions commandLineOptions) : IDataConsumer, ITestSessionLifetimeHandler
+internal sealed class GitHubTestReporter(
+    GitHubTestReporterExtension extension,
+    ICommandLineOptions commandLineOptions
+) : IDataConsumer, ITestSessionLifetimeHandler
 {
-    private readonly TestReporterContext _context = new(GitHubWorkflow.Default, TestReporterOptions.Resolve(commandLineOptions));
+    private readonly TestReporterContext _context = new(
+        GitHubWorkflow.Default,
+        TestReporterOptions.Resolve(commandLineOptions)
+    );
 
-    public Type[] DataTypesConsumed { get; } =
-        [
-            typeof(TestNodeUpdateMessage),
-        ];
+    public Type[] DataTypesConsumed { get; } = [typeof(TestNodeUpdateMessage)];
 
     public string Uid => extension.Uid;
     public string Version => extension.Version;
     public string DisplayName => extension.DisplayName;
     public string Description => extension.Description;
 
-    public Task ConsumeAsync(IDataProducer dataProducer, IData value, CancellationToken cancellationToken)
+    public Task ConsumeAsync(
+        IDataProducer dataProducer,
+        IData value,
+        CancellationToken cancellationToken
+    )
     {
         _context.HandleTestResult((TestNodeUpdateMessage)value);
         return Task.CompletedTask;
     }
 
     public Task<bool> IsEnabledAsync() => extension.IsEnabledAsync();
 
-    public Task OnTestSessionFinishingAsync(SessionUid sessionUid, CancellationToken cancellationToken)
+    public Task OnTestSessionFinishingAsync(
+        SessionUid sessionUid,
+        CancellationToken cancellationToken
+    )
     {
         _context.HandleTestRunComplete();
         return Task.CompletedTask;
     }
 
-    public Task OnTestSessionStartingAsync(SessionUid sessionUid, CancellationToken cancellationToken)
-        => Task.CompletedTask;
-}
+    public Task OnTestSessionStartingAsync(
+        SessionUid sessionUid,
+        CancellationToken cancellationToken
+    ) => Task.CompletedTask;
+}

GitHubActionsTestLogger/GitHubTestReporterExtension.cs

@@ -1,5 +1,4 @@
 using System.Threading.Tasks;
-
 using Microsoft.Testing.Platform.Extensions;
 
 namespace GitHubActionsTestLogger;

GitHubActionsTestLogger/TestReporterContext.cs

@@ -5,9 +5,7 @@
 using System.Reflection;
 using System.Text;
 using System.Threading;
-
 using GitHubActionsTestLogger.Utils.Extensions;
-
 using Microsoft.Testing.Platform.Extensions.Messages;
 
 namespace GitHubActionsTestLogger;
@@ -20,7 +18,11 @@ public class TestReporterContext(GitHubWorkflow github, TestReporterOptions opti
 
     public TestReporterOptions Options { get; } = options;
 
-    private string FormatAnnotation(string format, TestNode testNode, TestNodeStateProperty testNodeStateProperty)
+    private string FormatAnnotation(
+        string format,
+        TestNode testNode,
+        TestNodeStateProperty testNodeStateProperty
+    )
     {
         var buffer = new StringBuilder(format);
 
@@ -73,17 +75,22 @@ private string FormatAnnotation(string format, TestNode testNode, TestNodeStateP
         return buffer.Trim().ToString();
     }
 
-    private string FormatAnnotationTitle(TestNode testNode, TestNodeStateProperty testNodeStateProperty) =>
-        FormatAnnotation(Options.AnnotationTitleFormat, testNode, testNodeStateProperty);
+    private string FormatAnnotationTitle(
+        TestNode testNode,
+        TestNodeStateProperty testNodeStateProperty
+    ) => FormatAnnotation(Options.AnnotationTitleFormat, testNode, testNodeStateProperty);
 
-    private string FormatAnnotationMessage(TestNode testNode, TestNodeStateProperty testNodeStateProperty) =>
-        FormatAnnotation(Options.AnnotationMessageFormat, testNode, testNodeStateProperty);
+    private string FormatAnnotationMessage(
+        TestNode testNode,
+        TestNodeStateProperty testNodeStateProperty
+    ) => FormatAnnotation(Options.AnnotationMessageFormat, testNode, testNodeStateProperty);
 
     public void HandleTestResult(TestNodeUpdateMessage testNodeUpdateMessage)
     {
         using (_lock.EnterScope())
         {
-            var testNodeState = testNodeUpdateMessage.TestNode.Properties.Single<TestNodeStateProperty>();
+            var testNodeState =
+                testNodeUpdateMessage.TestNode.Properties.Single<TestNodeStateProperty>();
             // Report failed test results to job annotations
             if (testNodeState is FailedTestNodeStateProperty or ErrorTestNodeStateProperty)
             {
@@ -108,15 +115,13 @@ public void HandleTestRunStart()
         }
     }
 
-
     public void HandleTestRunComplete()
     {
         using (_lock.EnterScope())
         {
             _stopwatch.Stop();
 
-            var testSuite = Assembly.GetEntryAssembly()?.GetName().Name
-                ?? "Unknown Test Suite";
+            var testSuite = Assembly.GetEntryAssembly()?.GetName().Name ?? "Unknown Test Suite";
 
             var targetFramework =
                 // See line 65

GitHubActionsTestLogger/TestReporterOptions.cs

@@ -1,7 +1,5 @@
 using System.Linq;
-
 using Microsoft.Testing.Platform.CommandLine;
-
 using static GitHubActionsTestLogger.CliOptionsProvider;
 
 namespace GitHubActionsTestLogger;
@@ -25,17 +23,25 @@ public partial class TestReporterOptions
 
     public static TestReporterOptions Resolve(ICommandLineOptions commandLineOptions)
     {
-        var annotationTitleFormat = 
-            (commandLineOptions.TryGetOptionArgumentList(ReportGitHubTitleOption, out string[]? arguments)
-                ? arguments[0]
-                : null)
-            ?? Default.AnnotationTitleFormat;
+        var annotationTitleFormat =
+            (
+                commandLineOptions.TryGetOptionArgumentList(
+                    ReportGitHubTitleOption,
+                    out string[]? arguments
+                )
+                    ? arguments[0]
+                    : null
+            ) ?? Default.AnnotationTitleFormat;
 
         var annotationMessageFormat =
-            (commandLineOptions.TryGetOptionArgumentList(ReportGitHubMessageOption, out arguments)
-                ? arguments[0]
-                : null)
-            ?? Default.AnnotationMessageFormat;
+            (
+                commandLineOptions.TryGetOptionArgumentList(
+                    ReportGitHubMessageOption,
+                    out arguments
+                )
+                    ? arguments[0]
+                    : null
+            ) ?? Default.AnnotationMessageFormat;
 
         _ = commandLineOptions.TryGetOptionArgumentList(ReportGitHubSummaryOption, out arguments);
 
@@ -44,7 +50,7 @@ public static TestReporterOptions Resolve(ICommandLineOptions commandLineOptions
             AnnotationTitleFormat = annotationTitleFormat,
             AnnotationMessageFormat = annotationMessageFormat,
             SummaryIncludePassedTests =
-                arguments?.Contains(ReportGitHubSummaryArguments.IncludePassedTests) == true 
+                arguments?.Contains(ReportGitHubSummaryArguments.IncludePassedTests) == true
                 || Default.SummaryIncludePassedTests,
             SummaryIncludeSkippedTests =
                 arguments?.Contains(ReportGitHubSummaryArguments.IncludeSkippedTests) == true

GitHubActionsTestLogger/TestingPlatformBuilderHook.cs

@@ -12,6 +12,6 @@ public static class TestingPlatformBuilderHook
     /// </summary>
     /// <param name="testApplicationBuilder">The test application builder.</param>
     /// <param name="_">The command line arguments.</param>
-    public static void AddExtensions(ITestApplicationBuilder testApplicationBuilder, string[] _)
-        => testApplicationBuilder.AddGitHubReportProvider();
+    public static void AddExtensions(ITestApplicationBuilder testApplicationBuilder, string[] _) =>
+        testApplicationBuilder.AddGitHubReportProvider();
 }