Merge the two summary layouts into one

Tyrrrz · May 19, 2023 · 6e0b37a · 6e0b37a
1 parent 7ed6f2e
commit 6e0b37a
Show file tree

Hide file tree

Showing 11 changed files with 198 additions and 268 deletions.
diff --git a/GitHubActionsTestLogger.Tests/InitializationSpecs.cs b/GitHubActionsTestLogger.Tests/InitializationSpecs.cs
@@ -34,7 +34,6 @@ public void I_can_use_the_logger_with_a_custom_configuration()
         {
             ["annotations.titleFormat"] = "TitleFormat",
             ["annotations.messageFormat"] = "MessageFormat",
-            ["summary.compactLayout"] = "true",
             ["summary.includePassedTests"] = "true",
             ["summary.includeSkippedTests"] = "true"
         };
@@ -46,7 +45,6 @@ public void I_can_use_the_logger_with_a_custom_configuration()
         logger.Context.Should().NotBeNull();
         logger.Context?.Options.AnnotationTitleFormat.Should().Be("TitleFormat");
         logger.Context?.Options.AnnotationMessageFormat.Should().Be("MessageFormat");
-        logger.Context?.Options.SummaryCompactLayout.Should().BeTrue();
         logger.Context?.Options.SummaryIncludePassedTests.Should().BeTrue();
         logger.Context?.Options.SummaryIncludeSkippedTests.Should().BeTrue();
     }

diff --git a/GitHubActionsTestLogger/Templates/MarkdownRazorTemplate.cs b/GitHubActionsTestLogger/Templates/MarkdownRazorTemplate.cs
diff --git a/GitHubActionsTestLogger/Templates/TestSummaryContext.cs b/GitHubActionsTestLogger/Templates/TestSummaryContext.cs
diff --git a/GitHubActionsTestLogger/Templates/TestSummaryDetailsTemplate.cs b/GitHubActionsTestLogger/Templates/TestSummaryDetailsTemplate.cs
diff --git a/GitHubActionsTestLogger/Templates/TestSummaryDetailsTemplate.cshtml b/GitHubActionsTestLogger/Templates/TestSummaryDetailsTemplate.cshtml
diff --git a/GitHubActionsTestLogger/Templates/TestSummaryTemplate.cs b/GitHubActionsTestLogger/Templates/TestSummaryTemplate.cs
diff --git a/GitHubActionsTestLogger/Templates/TestSummaryTemplate.cshtml b/GitHubActionsTestLogger/Templates/TestSummaryTemplate.cshtml
diff --git a/GitHubActionsTestLogger/TestLoggerContext.cs b/GitHubActionsTestLogger/TestLoggerContext.cs
@@ -2,7 +2,6 @@
 using System.IO;
 using System.Linq;
 using System.Text;
-using GitHubActionsTestLogger.Templates;
 using GitHubActionsTestLogger.Utils.Extensions;
 using Microsoft.VisualStudio.TestPlatform.ObjectModel;
 using Microsoft.VisualStudio.TestPlatform.ObjectModel.Client;
@@ -90,7 +89,7 @@ public void HandleTestRunComplete(TestRunCompleteEventArgs args)
     {
         lock (_lock)
         {
-            var context = new TestSummaryContext
+            var template = new TestSummaryTemplate
             {
                 Options = Options,
 
@@ -105,7 +104,7 @@ public void HandleTestRunComplete(TestRunCompleteEventArgs args)
                 TestRunResult = new TestRunResult(_testResults, args.ElapsedTimeInRunningTests)
             };
 
-            _github.CreateSummary(new TestSummaryTemplate(context).Render());
+            _github.CreateSummary(template.Render());
         }
     }
 }
diff --git a/GitHubActionsTestLogger/TestLoggerOptions.cs b/GitHubActionsTestLogger/TestLoggerOptions.cs
@@ -9,8 +9,6 @@ public partial class TestLoggerOptions
 
     public string AnnotationMessageFormat { get; init; } = "$error";
 
-    public bool SummaryCompactLayout { get; init; }
-
     public bool SummaryIncludePassedTests { get; init; }
 
     public bool SummaryIncludeSkippedTests { get; init; }
@@ -30,10 +28,6 @@ public partial class TestLoggerOptions
             parameters.GetValueOrDefault("annotations.messageFormat") ??
             Default.AnnotationMessageFormat,
 
-        SummaryCompactLayout =
-            parameters.GetValueOrDefault("summary.compactLayout")?.Pipe(bool.Parse) ??
-            Default.SummaryCompactLayout,
-
         SummaryIncludePassedTests =
             parameters.GetValueOrDefault("summary.includePassedTests")?.Pipe(bool.Parse) ??
             Default.SummaryIncludePassedTests,