Remove output window logging from integration tests, and see if it ex…

…poses a real error (like VSIX not installed I would guess)
dotnet · Dec 13, 2023 · 85372bc · 85372bc
1 parent f1efe63
commit 85372bc
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/src/Razor/test/Microsoft.VisualStudio.Razor.IntegrationTests/InProcess/OutputInProcess.cs b/src/Razor/test/Microsoft.VisualStudio.Razor.IntegrationTests/InProcess/OutputInProcess.cs
@@ -22,20 +22,20 @@ internal partial class OutputInProcess
 
     public async Task SetupIntegrationTestLoggerAsync(ITestOutputHelper testOutputHelper, CancellationToken cancellationToken)
     {
-        var logger = await TestServices.Shell.GetComponentModelServiceAsync<IOutputWindowLogger>(cancellationToken);
-        logger.SetTestLogger(new TestOutputLogger(testOutputHelper));
+        // var logger = await TestServices.Shell.GetComponentModelServiceAsync<IOutputWindowLogger>(cancellationToken);
+        // logger.SetTestLogger(new TestOutputLogger(testOutputHelper));
     }
 
     public async Task ClearIntegrationTestLoggerAsync(CancellationToken cancellationToken)
     {
-        var logger = await TestServices.Shell.GetComponentModelServiceAsync<IOutputWindowLogger>(cancellationToken);
-        logger.SetTestLogger(null);
+        // var logger = await TestServices.Shell.GetComponentModelServiceAsync<IOutputWindowLogger>(cancellationToken);
+        // logger.SetTestLogger(null);
     }
 
     public async Task LogStatusAsync(string message, CancellationToken cancellationToken)
     {
-        var logger = await TestServices.Shell.GetComponentModelServiceAsync<IOutputWindowLogger>(cancellationToken);
-        logger.LogInformation(message);
+        // var logger = await TestServices.Shell.GetComponentModelServiceAsync<IOutputWindowLogger>(cancellationToken);
+        // logger.LogInformation(message);
     }
 
     public async Task<bool> HasErrorsAsync(CancellationToken cancellationToken)

diff --git a/src/Razor/test/Microsoft.VisualStudio.Razor.IntegrationTests/LogIntegrationTestAttribute.cs b/src/Razor/test/Microsoft.VisualStudio.Razor.IntegrationTests/LogIntegrationTestAttribute.cs
@@ -16,12 +16,12 @@ public class LogIntegrationTestAttribute : BeforeAfterTestAttribute
 {
     public override void Before(MethodInfo methodUnderTest)
     {
-        GetLogger().LogInformation("#### Integration test start: {method}", methodUnderTest.Name);
+        // GetLogger().LogInformation("#### Integration test start: {method}", methodUnderTest.Name);
     }
 
     public override void After(MethodInfo methodUnderTest)
     {
-        GetLogger().LogInformation("#### Integration test end: {method}", methodUnderTest.Name);
+        // GetLogger().LogInformation("#### Integration test end: {method}", methodUnderTest.Name);
     }
 
     private static IOutputWindowLogger GetLogger()