diff --git a/.github/workflows/run-tests.yml b/.github/workflows/run-tests.yml index ffa66ee7217..6ddb9d3b299 100644 --- a/.github/workflows/run-tests.yml +++ b/.github/workflows/run-tests.yml @@ -16,7 +16,7 @@ on: testSessionTimeoutMs: required: false type: string - default: "600000" + default: "900000" testHangTimeout: required: false type: string diff --git a/tests/Aspire.Dashboard.Tests/Integration/Playwright/AppBarTests.cs b/tests/Aspire.Dashboard.Tests/Integration/Playwright/AppBarTests.cs index 9b6c8dcb6bf..aa5a20b9532 100644 --- a/tests/Aspire.Dashboard.Tests/Integration/Playwright/AppBarTests.cs +++ b/tests/Aspire.Dashboard.Tests/Integration/Playwright/AppBarTests.cs @@ -65,7 +65,6 @@ await AsyncTestHelpers.AssertIsTrueRetryAsync( } [Fact] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/7943")] public async Task AppBar_Change_Theme_ReloadPage() { // Arrange diff --git a/tests/Aspire.Hosting.Testing.Tests/TestingBuilderTests.cs b/tests/Aspire.Hosting.Testing.Tests/TestingBuilderTests.cs index a15231fb50e..54f8f357f75 100644 --- a/tests/Aspire.Hosting.Testing.Tests/TestingBuilderTests.cs +++ b/tests/Aspire.Hosting.Testing.Tests/TestingBuilderTests.cs @@ -210,7 +210,6 @@ public async Task GetHttpClientBeforeStart(bool genericEntryPoint) [InlineData(false, true)] [InlineData(true, false)] [InlineData(true, true)] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/7930")] public async Task ArgsPropagateToAppHostConfiguration(bool genericEntryPoint, bool directArgs) { string[] args = directArgs ? ["APP_HOST_ARG=42"] : []; diff --git a/tests/Aspire.Hosting.Tests/DistributedApplicationTests.cs b/tests/Aspire.Hosting.Tests/DistributedApplicationTests.cs index fe627c6a35a..fe92fdc926a 100644 --- a/tests/Aspire.Hosting.Tests/DistributedApplicationTests.cs +++ b/tests/Aspire.Hosting.Tests/DistributedApplicationTests.cs @@ -311,7 +311,6 @@ public void TryAddWillNotAddTheSameLifecycleHook() } [Fact] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/4651")] public async Task AllocatedPortsAssignedAfterHookRuns() { using var testProgram = CreateTestProgram("ports-assigned-after-hook-runs"); @@ -344,7 +343,6 @@ public Task AfterEndpointsAllocatedAsync(DistributedApplicationModel appModel, C } [Fact] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/4651")] public async Task TestServicesWithMultipleReplicas() { var replicaCount = 3; @@ -401,7 +399,6 @@ public async Task TestServicesWithMultipleReplicas() [Fact] [RequiresDocker] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/4651")] public async Task VerifyContainerArgs() { using var testProgram = CreateTestProgram("verify-container-args"); @@ -486,7 +483,6 @@ public async Task VerifyContainerCreateFile() [Fact] [RequiresDocker] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/4651")] public async Task VerifyContainerStopStartWorks() { using var testProgram = CreateTestProgram("container-start-stop", randomizePorts: false); @@ -574,7 +570,6 @@ public async Task VerifyExecutableStopStartWorks() [Fact] [RequiresDocker] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/4651")] public async Task SpecifyingEnvPortInEndpointFlowsToEnv() { const string testName = "ports-flow-to-env"; @@ -631,7 +626,6 @@ public async Task SpecifyingEnvPortInEndpointFlowsToEnv() } [Fact] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/4651")] public async Task StartAsync_DashboardAuthConfig_PassedToDashboardProcess() { const string testName = "dashboard-auth-config"; @@ -709,7 +703,6 @@ public async Task StartAsync_UnsecuredAllowAnonymous_PassedToDashboardProcess() [Fact] [RequiresDocker] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/4651")] public async Task VerifyDockerWithEntrypointWorks() { const string testName = "docker-entrypoint"; @@ -738,7 +731,6 @@ public async Task VerifyDockerWithEntrypointWorks() [Fact] [RequiresDocker] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/4651")] public async Task VerifyDockerWithBindMountWorksWithAbsolutePaths() { const string testName = "docker-bindmount-absolute"; @@ -769,7 +761,6 @@ public async Task VerifyDockerWithBindMountWorksWithAbsolutePaths() [Fact] [RequiresDocker] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/4651")] public async Task VerifyDockerWithBindMountWorksWithRelativePaths() { const string testName = "docker-bindmount-relative"; @@ -800,7 +791,6 @@ public async Task VerifyDockerWithBindMountWorksWithRelativePaths() [Fact] [RequiresDocker] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/4651")] public async Task VerifyDockerWithVolumeWorksWithName() { const string testName = "docker-volume"; @@ -830,7 +820,6 @@ public async Task VerifyDockerWithVolumeWorksWithName() [Fact] [RequiresDocker] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/4651")] public async Task KubernetesHasResourceNameForContainersAndExes() { const string testName = "kube-resource-names"; @@ -888,7 +877,6 @@ public async Task KubernetesHasResourceNameForContainersAndExes() } [Fact] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/4651")] public async Task ReplicasAndProxylessEndpointThrows() { const string testName = "replicas-no-proxyless-endpoints"; @@ -907,7 +895,6 @@ public async Task ReplicasAndProxylessEndpointThrows() } [Fact] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/4651")] public async Task ProxylessEndpointWithoutPortThrows() { const string testName = "proxyess-endpoint-without-port"; @@ -927,7 +914,6 @@ public async Task ProxylessEndpointWithoutPortThrows() } [Fact] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/4651")] public async Task ProxylessEndpointWorks() { const string testName = "proxyless-endpoint-works"; @@ -1031,7 +1017,6 @@ public async Task ProxylessAndProxiedEndpointBothWorkOnSameResource() [Fact] [RequiresDocker] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/4651")] public async Task ProxylessContainerCanBeReferenced() { const string testName = "proxyless-container"; @@ -1137,7 +1122,6 @@ public async Task WithEndpointProxySupportDisablesProxies() [Fact] [RequiresDocker] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/4651")] public async Task ProxylessContainerWithoutPortThrows() { const string testName = "proxyless-container-without-ports"; @@ -1156,7 +1140,6 @@ public async Task ProxylessContainerWithoutPortThrows() [Fact] [RequiresDocker] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/4651")] public async Task AfterResourcesCreatedLifecycleHookWorks() { const string testName = "lifecycle-hook-after-resource-created"; diff --git a/tests/Aspire.Hosting.Tests/Health/ResourceHealthCheckServiceTests.cs b/tests/Aspire.Hosting.Tests/Health/ResourceHealthCheckServiceTests.cs index 744cf1e0c63..5d74675f18a 100644 --- a/tests/Aspire.Hosting.Tests/Health/ResourceHealthCheckServiceTests.cs +++ b/tests/Aspire.Hosting.Tests/Health/ResourceHealthCheckServiceTests.cs @@ -4,7 +4,6 @@ using System.Threading.Channels; using Aspire.Hosting.Health; using Aspire.Hosting.Utils; -using Aspire.TestUtilities; using Microsoft.AspNetCore.InternalTesting; using Microsoft.Extensions.DependencyInjection; using Microsoft.Extensions.Diagnostics.HealthChecks; @@ -97,7 +96,6 @@ await app.ResourceNotifications.PublishUpdateAsync(resource.Resource, s => s wit } [Fact] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/8326")] public async Task ResourcesWithHealthCheck_CreationErrorIsReported() { using var builder = TestDistributedApplicationBuilder.Create(testOutputHelper); @@ -138,7 +136,6 @@ await app.ResourceNotifications.PublishUpdateAsync(resource.Resource, s => s wit } [Fact] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/8103")] public async Task ResourcesWithHealthCheck_StopsAndRestartsMonitoringWithResource() { using var builder = TestDistributedApplicationBuilder.Create(testOutputHelper); diff --git a/tests/Aspire.Hosting.Tests/OperationModesTests.cs b/tests/Aspire.Hosting.Tests/OperationModesTests.cs index 36dd5c9e81e..5485846f872 100644 --- a/tests/Aspire.Hosting.Tests/OperationModesTests.cs +++ b/tests/Aspire.Hosting.Tests/OperationModesTests.cs @@ -14,7 +14,6 @@ namespace Aspire.Hosting.Tests; public class OperationModesTests(ITestOutputHelper outputHelper) { [Fact] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/8400")] public async Task VerifyBackwardsCompatableRunModeInvocation() { // The purpose of this test is to verify that the apphost executable will continue diff --git a/tests/Aspire.Hosting.Tests/SlimTestProgramTests.cs b/tests/Aspire.Hosting.Tests/SlimTestProgramTests.cs index 98a119081ed..0f3cb1ff757 100644 --- a/tests/Aspire.Hosting.Tests/SlimTestProgramTests.cs +++ b/tests/Aspire.Hosting.Tests/SlimTestProgramTests.cs @@ -1,7 +1,6 @@ // Licensed to the .NET Foundation under one or more agreements. // The .NET Foundation licenses this file to you under the MIT license. -using Aspire.TestUtilities; using Aspire.Hosting.Testing.Tests; using Microsoft.AspNetCore.InternalTesting; using Xunit; @@ -19,7 +18,6 @@ public SlimTestProgramTests(SlimTestProgramFixture slimTestProgramFixture) } [Fact] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/7923")] public async Task TestProjectStartsAndStopsCleanly() { var testProgram = _slimTestProgramFixture.TestProgram; @@ -44,7 +42,6 @@ private static async Task EnsureServicesAreRunning(TestProgram testProgram, Canc } [Fact] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/7923")] public async Task TestPortOnEndpointAnnotationAndAllocatedEndpointAnnotationMatch() { var testProgram = _slimTestProgramFixture.TestProgram; @@ -63,7 +60,6 @@ public async Task TestPortOnEndpointAnnotationAndAllocatedEndpointAnnotationMatc } [Fact] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/7923")] public async Task TestPortOnEndpointAnnotationAndAllocatedEndpointAnnotationMatchForReplicatedServices() { var testProgram = _slimTestProgramFixture.TestProgram; diff --git a/tests/Aspire.Hosting.Tests/WithHttpCommandTests.cs b/tests/Aspire.Hosting.Tests/WithHttpCommandTests.cs index d11184aedf4..5f5afe3d20b 100644 --- a/tests/Aspire.Hosting.Tests/WithHttpCommandTests.cs +++ b/tests/Aspire.Hosting.Tests/WithHttpCommandTests.cs @@ -161,7 +161,6 @@ public async Task WithHttpCommand_ResultsInExpectedResultForStatusCode(int statu [InlineData("get", true)] [InlineData("post", false)] [Theory] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/8514")] public async Task WithHttpCommand_ResultsInExpectedResultForHttpMethod(string? httpMethod, bool expectSuccess) { // Arrange @@ -263,7 +262,6 @@ public async Task WithHttpCommand_UsesEndpointSelector() } [Fact] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/8192")] public async Task WithHttpCommand_CallsPrepareRequestCallback_BeforeSendingRequest() { // Arrange @@ -308,7 +306,6 @@ public async Task WithHttpCommand_CallsPrepareRequestCallback_BeforeSendingReque } [Fact] - [QuarantinedTest("https://github.com/dotnet/aspire/issues/8200")] public async Task WithHttpCommand_CallsGetResponseCallback_AfterSendingRequest() { // Arrange