[AzureMonitor] update live tests (Azure#49023)

* update tests

* cleanup

* cleanup csproj

* rearranging tests to fix git compare

* testing disabling timeout

Author: Timothy Mothra

sdk/monitor/Azure.Monitor.OpenTelemetry.AspNetCore/tests/Azure.Monitor.OpenTelemetry.AspNetCore.Integration.Tests/Azure.Monitor.OpenTelemetry.AspNetCore.Integration.Tests.csproj

@@ -14,12 +14,4 @@
     <ProjectReference Include="..\..\..\Azure.Monitor.OpenTelemetry.Exporter\src\Azure.Monitor.OpenTelemetry.Exporter.csproj" /> 
   </ItemGroup>
 
-  <ItemGroup Condition="'$(TargetFramework)' == 'net6.0'">
-    <FrameworkReference Include="Microsoft.AspNetCore.App" />
-  </ItemGroup>
-
-  <ItemGroup Condition="'$(TargetFramework)' == 'net7.0'">
-    <FrameworkReference Include="Microsoft.AspNetCore.App" />
-  </ItemGroup>
-
 </Project>

sdk/monitor/Azure.Monitor.OpenTelemetry.AspNetCore/tests/Azure.Monitor.OpenTelemetry.AspNetCore.Integration.Tests/BaseLiveTest.cs

@@ -50,6 +50,12 @@ public void Setup()
             ));
         }
 
+        public override void GlobalTimeoutTearDown()
+        {
+            // Turn off global timeout errors because these tests can be slower
+            // base.GlobalTimeoutTearDown();
+        }
+
         internal async Task QueryAndVerifyDependency(string workspaceId, string description, string query, ExpectedAppDependency expectedAppDependency)
         {
             LogsTable? logsTable = await _logsQueryClient!.QueryTelemetryAsync(workspaceId, description, query);

sdk/monitor/Azure.Monitor.OpenTelemetry.AspNetCore/tests/Azure.Monitor.OpenTelemetry.AspNetCore.Integration.Tests/DistroWebAppLiveTests.cs

@@ -12,6 +12,7 @@
 using Azure.Monitor.Query;
 using Azure.Monitor.Query.Models;
 using Microsoft.AspNetCore.Builder;
+using Microsoft.AspNetCore.Hosting;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Logging;
 using NUnit.Framework;
@@ -50,7 +51,7 @@ public DistroWebAppLiveTests(bool isAsync) : base(isAsync) { }
 
         [RecordedTest]
         [SyncOnly] // This test cannot run concurrently with another test because OTel instruments the process and will cause side effects.
-        public async Task VerifyDistro()
+        public async Task VerifyDistro_UseAzureMonitor()
         {
             // SETUP WEBAPPLICATION WITH OPENTELEMETRY
             var builder = WebApplication.CreateBuilder();
@@ -181,31 +182,27 @@ public async Task VerifySendingToTwoResources_UsingExporter()
 
         [RecordedTest]
         [SyncOnly] // This test cannot run concurrently with another test because OTel instruments the process and will cause side effects.
-        [Ignore("TODO: Repurpose this test to validate the error.")]
-        public async Task VerifySendingToTwoResources_UsingDistroWithExporter()
+        public async Task VerifyExporter_UseAzureMonitorExporter()
         {
             // SETUP WEBAPPLICATION WITH OPENTELEMETRY
             var builder = WebApplication.CreateBuilder();
+
+            builder.WebHost.UseUrls(TestServerUrl);
+
             builder.Logging.ClearProviders();
             builder.Services.ConfigureOpenTelemetryTracerProvider((sp, builder) => builder.AddProcessor(new ActivityEnrichingProcessor()));
             builder.Services.AddOpenTelemetry()
-                .UseAzureMonitor(options =>
+                .UseAzureMonitorExporter(options =>
                 {
                     options.EnableLiveMetrics = false;
                     options.ConnectionString = TestEnvironment.ConnectionString;
                 })
-                .WithTracing(builder =>
-                {
-                    builder.AddAzureMonitorTraceExporter(name: "secondary", configure: options => options.ConnectionString = TestEnvironment.SecondaryConnectionString);
-                })
-                .WithMetrics(builder =>
-                {
-                    builder.AddAzureMonitorMetricExporter(name: "secondary", configure: options => options.ConnectionString = TestEnvironment.SecondaryConnectionString);
-                })
-                .WithLogging(builder =>
-                {
-                    builder.AddAzureMonitorLogExporter(name: "secondary", configure: options => options.ConnectionString = TestEnvironment.SecondaryConnectionString);
-                })
+                .WithTracing(builder => builder
+                    .AddAspNetCoreInstrumentation()
+                    .AddHttpClientInstrumentation())
+                .WithMetrics(builder => builder
+                    .AddAspNetCoreInstrumentation()
+                    .AddHttpClientInstrumentation())
                 // Custom resources must be added AFTER AzureMonitor to override the included ResourceDetectors.
                 .ConfigureResource(x => x.AddAttributes(_testResourceAttributes));
 
@@ -218,7 +215,7 @@ public async Task VerifySendingToTwoResources_UsingDistroWithExporter()
                 return "Response from Test Server";
             });
 
-            _ = app.RunAsync(TestServerUrl);
+            await app.StartAsync().ConfigureAwait(false); // Start HostedServices
 
             // ACT
             using var httpClient = new HttpClient();
@@ -238,7 +235,6 @@ public async Task VerifySendingToTwoResources_UsingDistroWithExporter()
 
             // ASSERT
             await VerifyTelemetry(workspaceId: TestEnvironment.WorkspaceId);
-            await VerifyTelemetry(workspaceId: TestEnvironment.SecondaryWorkspaceId);
         }
 
         private async Task VerifyTelemetry(string workspaceId)