Some errors shouldn't break tests

Author: davidwengier

src/Razor/test/Microsoft.AspNetCore.Razor.LanguageServer.Test/Formatting_NetFx/FormattingTestBase.cs

@@ -15,6 +15,7 @@
 using Microsoft.AspNetCore.Razor.LanguageServer.Hosting;
 using Microsoft.AspNetCore.Razor.Test.Common;
 using Microsoft.AspNetCore.Razor.Test.Common.LanguageServer;
+using Microsoft.AspNetCore.Razor.Test.Common.Logging;
 using Microsoft.AspNetCore.Razor.Test.Common.Workspaces;
 using Microsoft.AspNetCore.Razor.Threading;
 using Microsoft.CodeAnalysis;
@@ -46,6 +47,8 @@ public abstract class FormattingTestBase : RazorToolingIntegrationTestBase
     private protected FormattingTestBase(FormattingTestContext context, HtmlFormattingService htmlFormattingService, ITestOutputHelper testOutput)
         : base(testOutput)
     {
+        TestOutputLogger.ThrowOnErrorLogged = true;
+
         _htmlFormattingService = htmlFormattingService;
         _context = context;
     }

src/Razor/test/Microsoft.AspNetCore.Razor.Test.Common.Tooling/Logging/TestOutputLogger.cs

@@ -9,6 +9,8 @@ namespace Microsoft.AspNetCore.Razor.Test.Common.Logging;
 
 internal partial class TestOutputLogger(TestOutputLoggerProvider provider, string categoryName, LogLevel logLevel = LogLevel.Trace) : ILogger
 {
+    public static bool ThrowOnErrorLogged { get; set; } = false;
+
     private readonly TestOutputLoggerProvider _provider = provider;
     private readonly string _categoryName = categoryName;
     private readonly LogLevel _logLevel = logLevel;
@@ -18,7 +20,7 @@ public bool IsEnabled(LogLevel logLevel)
 
     public void Log(LogLevel logLevel, string message, Exception? exception)
     {
-        if (logLevel == LogLevel.Error)
+        if (ThrowOnErrorLogged && logLevel == LogLevel.Error)
         {
             throw exception ?? new InvalidOperationException(message);
         }

src/Razor/test/Microsoft.VisualStudio.LanguageServices.Razor.Test/Cohost/Formatting/FormattingTestBase.cs

@@ -7,6 +7,7 @@
 using Microsoft.AspNetCore.Razor;
 using Microsoft.AspNetCore.Razor.Language;
 using Microsoft.AspNetCore.Razor.Test.Common;
+using Microsoft.AspNetCore.Razor.Test.Common.Logging;
 using Microsoft.CodeAnalysis;
 using Microsoft.CodeAnalysis.ExternalAccess.Razor;
 using Microsoft.CodeAnalysis.ExternalAccess.Razor.Features;
@@ -30,6 +31,8 @@ public abstract class FormattingTestBase : CohostEndpointTestBase
     private protected FormattingTestBase(FormattingTestContext context, HtmlFormattingService htmlFormattingService, ITestOutputHelper testOutputHelper)
         : base(testOutputHelper)
     {
+        TestOutputLogger.ThrowOnErrorLogged = true;
+
         _context = context;
         _htmlFormattingService = htmlFormattingService;
     }