TODOs

Author: Kielek

test/IntegrationTests/AspNetTests.cs

@@ -83,6 +83,7 @@ public async Task SubmitsTraces(AppPoolMode appPoolMode, Gac useGac)
         var webPort = TcpPortProvider.GetOpenPort();
         var imageName = GetTestImageName(appPoolMode, useGac);
 #pragma warning disable CA2007 // Do not directly await a Task. https://github.com/dotnet/roslyn-analyzers/issues/7185
+        // TODO remove pragma when https://github.com/dotnet/roslyn-analyzers/issues/7185 is fixed
         await using var container = await IISContainerTestHelper.StartContainerAsync(imageName, webPort, environmentVariables, Output);
 #pragma warning restore CA2007 // Do not directly await a Task. https://github.com/dotnet/roslyn-analyzers/issues/7185
         await CallTestApplicationEndpoint(webPort).ConfigureAwait(true);
@@ -157,6 +158,7 @@ public async Task SubmitTracesCapturesHttpHeaders(AppPoolMode appPoolMode, Gac u
         var webPort = TcpPortProvider.GetOpenPort();
         var imageName = GetTestImageName(appPoolMode, useGac);
 #pragma warning disable CA2007 // Do not directly await a Task. https://github.com/dotnet/roslyn-analyzers/issues/7185
+        // TODO remove pragma when https://github.com/dotnet/roslyn-analyzers/issues/7185 is fixed
         await using var container = await IISContainerTestHelper.StartContainerAsync(imageName, webPort, environmentVariables, Output);
 #pragma warning restore CA2007 // Do not directly await a Task. https://github.com/dotnet/roslyn-analyzers/issues/7185
         await CallTestApplicationEndpoint(webPort).ConfigureAwait(true);
@@ -191,6 +193,7 @@ public async Task TracesResource()
 
         var webPort = TcpPortProvider.GetOpenPort();
 #pragma warning disable CA2007 // Do not directly await a Task. https://github.com/dotnet/roslyn-analyzers/issues/7185
+        // TODO remove pragma when https://github.com/dotnet/roslyn-analyzers/issues/7185 is fixed
         await using var container = await IISContainerTestHelper.StartContainerAsync("testapplication-aspnet-netframework-integrated", webPort, environmentVariables, Output);
 #pragma warning restore CA2007 // Do not directly await a Task. https://github.com/dotnet/roslyn-analyzers/issues/7185
         await CallTestApplicationEndpoint(webPort).ConfigureAwait(true);
@@ -222,6 +225,7 @@ public async Task SubmitMetrics()
         };
         var webPort = TcpPortProvider.GetOpenPort();
 #pragma warning disable CA2007 // Do not directly await a Task. https://github.com/dotnet/roslyn-analyzers/issues/7185
+        // TODO remove pragma when https://github.com/dotnet/roslyn-analyzers/issues/7185 is fixed
         await using var container = await IISContainerTestHelper.StartContainerAsync("testapplication-aspnet-netframework-integrated", webPort, environmentVariables, Output);
 #pragma warning restore CA2007 // Do not directly await a Task. https://github.com/dotnet/roslyn-analyzers/issues/7185
         await CallTestApplicationEndpoint(webPort).ConfigureAwait(true);

test/IntegrationTests/Helpers/MockMetricsCollector.cs

@@ -236,6 +236,7 @@ private void WriteOutput(string msg)
     }
 
 #pragma warning disable CA1812 // Mark members as static. There is some issue in dotnet format.
+// TODO remove pragma when dotnet format issue is fixed
     internal sealed class Collected
 #pragma warning restore CA1812 // Mark members as static. There is some issue in dotnet format.
     {

test/IntegrationTests/Helpers/MockSpansCollector.cs

@@ -17,6 +17,7 @@
 namespace IntegrationTests.Helpers;
 
 #pragma warning disable CA1812 // Mark members as static. There is some issue in dotnet format.
+// TODO remove pragma when dotnet format issue is fixed
 internal sealed class MockSpansCollector : IDisposable
 #pragma warning restore CA1812 // Mark members as static. There is some issue in dotnet format.
 {
@@ -225,6 +226,7 @@ private void WriteOutput(string msg)
     }
 
 #pragma warning disable CA1812 // Mark members as static. There is some issue in dotnet format.
+// TODO remove pragma when dotnet format issue is fixed
     internal sealed class Collected
 #pragma warning restore CA1812 // Mark members as static. There is some issue in dotnet format.
     {

test/IntegrationTests/Helpers/PathHandler.cs

@@ -7,6 +7,7 @@
 namespace IntegrationTests.Helpers;
 
 #pragma warning disable CA1812 // Mark members as static. There is some issue in dotnet format.
+// TODO remove pragma when dotnet format issue is fixed
 internal sealed class PathHandler
 #pragma warning restore CA1812 // Mark members as static. There is some issue in dotnet format.
 {

test/IntegrationTests/OwinIISTests.cs

@@ -47,6 +47,7 @@ public async Task SubmitsTraces()
         };
         var webPort = TcpPortProvider.GetOpenPort();
 #pragma warning disable CA2007 // Do not directly await a Task. https://github.com/dotnet/roslyn-analyzers/issues/7185
+        // TODO remove pragma when https://github.com/dotnet/roslyn-analyzers/issues/7185 is fixed
         await using var container = await IISContainerTestHelper.StartContainerAsync("testapplication-owin-iis-netframework", webPort, environmentVariables, Output);
 #pragma warning restore CA2007 // Do not directly await a Task. https://github.com/dotnet/roslyn-analyzers/issues/7185
 
@@ -80,6 +81,7 @@ public async Task SubmitMetrics()
         };
         var webPort = TcpPortProvider.GetOpenPort();
 #pragma warning disable CA2007 // Do not directly await a Task. https://github.com/dotnet/roslyn-analyzers/issues/7185
+        // TODO remove pragma when https://github.com/dotnet/roslyn-analyzers/issues/7185 is fixed
         await using var container = await IISContainerTestHelper.StartContainerAsync("testapplication-owin-iis-netframework", webPort, environmentVariables, Output);
 #pragma warning restore CA2007 // Do not directly await a Task. https://github.com/dotnet/roslyn-analyzers/issues/7185
 

test/IntegrationTests/VerboseTestFramework.cs

@@ -10,6 +10,7 @@
 namespace IntegrationTests.Helpers;
 
 #pragma warning disable CA1812 // Mark members as static. There is some issue in dotnet format.
+// TODO remove pragma when dotnet format issue is fixed
 public class VerboseTestFramework : XunitTestFramework
 {
     public VerboseTestFramework(IMessageSink messageSink)

test/IntegrationTests/WcfIISTests.cs

@@ -55,6 +55,7 @@ public async Task SubmitsTraces()
         _environmentVariables["OTEL_EXPORTER_OTLP_ENDPOINT"] = collectorUrl;
 
 #pragma warning disable CA2007 // Do not directly await a Task. https://github.com/dotnet/roslyn-analyzers/issues/7185
+        // TODO remove pragma when https://github.com/dotnet/roslyn-analyzers/issues/7185 is fixed
         await using var container = await StartContainerAsync(netTcpPort, httpPort);
 #pragma warning restore CA2007 // Do not directly await a Task. https://github.com/dotnet/roslyn-analyzers/issues/7185
 

test/OpenTelemetry.AutoInstrumentation.Bootstrapping.Tests/FactRequiringEnvVarAttribute.cs

@@ -6,6 +6,7 @@
 namespace OpenTelemetry.AutoInstrumentation.Bootstrapping.Tests;
 
 #pragma warning disable CA1812 // Mark members as static. There is some issue in dotnet format.
+// TODO remove pragma when dotnet format issue is fixed
 internal sealed class FactRequiringEnvVarAttribute : FactAttribute
 #pragma warning restore CA1812 // Mark members as static. There is some issue in dotnet format.
 {