Fix tests

Author: JamesNK

src/Aspire.Dashboard/ConsoleLogs/LogParser.cs

@@ -35,8 +35,8 @@ public LogEntry CreateLogEntry(string rawText, bool isErrorOutput)
         if (TimestampParser.TryColorizeTimestamp(content, out var timestampParseResult))
         {
             isFirstLine = true;
-            content = timestampParseResult.ModifiedText;
-            timestamp = timestampParseResult.Timestamp;
+            content = timestampParseResult.Value.ModifiedText;
+            timestamp = timestampParseResult.Value.Timestamp;
         }
         // 2. Parse the content to look for info/warn/dbug header
         // TODO extract log level and use here

src/Aspire.Dashboard/ConsoleLogs/TimestampParser.cs

@@ -1,6 +1,7 @@
 // Licensed to the .NET Foundation under one or more agreements.
 // The .NET Foundation licenses this file to you under the MIT license.
 
+using System.Diagnostics.CodeAnalysis;
 using System.Globalization;
 using System.Text.RegularExpressions;
 using Aspire.Dashboard.Extensions;
@@ -12,7 +13,7 @@ public static partial class TimestampParser
 {
     private static readonly Regex s_rfc3339RegEx = GenerateRfc3339RegEx();
 
-    public static bool TryColorizeTimestamp(string text, out TimestampParserResult result)
+    public static bool TryColorizeTimestamp(string text, [NotNullWhen(true)] out TimestampParserResult? result)
     {
         var match = s_rfc3339RegEx.Match(text);
 

tests/Aspire.Dashboard.Tests/ConsoleLogsTests/TimestampParserTests.cs

@@ -21,17 +21,26 @@ public void TryColorizeTimestamp_DoesNotStartWithTimestamp_ReturnsFalse(string i
     }
 
     [Theory]
-    [InlineData("2023-10-10T15:05:30.123456789Z", true, "<span class=\"timestamp\">2023-10-10T15:05:30.123456789Z</span>", "2023-10-10T15:05:30.123456789Z")]
-    [InlineData("2023-10-10T15:05:30.123456789Z ", true, "<span class=\"timestamp\">2023-10-10T15:05:30.123456789Z</span> ", "2023-10-10T15:05:30.123456789Z")]
-    [InlineData("2023-10-10T15:05:30.123456789Z with some text after it", true, "<span class=\"timestamp\">2023-10-10T15:05:30.123456789Z</span> with some text after it", "2023-10-10T15:05:30.123456789Z")]
+    [InlineData("2023-10-10T15:05:30.123456789Z", true, "", "2023-10-10T15:05:30.123456789Z")]
+    [InlineData("2023-10-10T15:05:30.123456789Z ", true, " ", "2023-10-10T15:05:30.123456789Z")]
+    [InlineData("2023-10-10T15:05:30.123456789Z with some text after it", true, " with some text after it", "2023-10-10T15:05:30.123456789Z")]
     [InlineData("With some text before it 2023-10-10T15:05:30.123456789Z", false, null, null)]
     public void TryColorizeTimestamp_ReturnsCorrectResult(string input, bool expectedResult, string? expectedOutput, string? expectedTimestamp)
     {
         var result = TimestampParser.TryColorizeTimestamp(input, out var parseResult);
 
         Assert.Equal(expectedResult, result);
-        Assert.Equal(expectedOutput, parseResult.ModifiedText);
-        Assert.Equal(expectedTimestamp != null ? (DateTimeOffset?)DateTimeOffset.Parse(expectedTimestamp, CultureInfo.InvariantCulture) : null, parseResult.Timestamp);
+
+        if (result)
+        {
+            Assert.NotNull(parseResult);
+            Assert.Equal(expectedOutput, parseResult.Value.ModifiedText);
+            Assert.Equal(expectedTimestamp != null ? (DateTimeOffset?)DateTimeOffset.Parse(expectedTimestamp, CultureInfo.InvariantCulture) : null, parseResult.Value.Timestamp);
+        }
+        else
+        {
+            Assert.Null(parseResult);
+        }
     }
 
     [Theory]