Skip to content

[automated] Unquarantine stable tests#14862

Merged
radical merged 2 commits intomainfrom
copilot/unquarantine-stable-tests-please-work
Mar 3, 2026
Merged

[automated] Unquarantine stable tests#14862
radical merged 2 commits intomainfrom
copilot/unquarantine-stable-tests-please-work

Conversation

Copy link
Contributor

Copilot AI commented Mar 2, 2026

Removes [QuarantinedTest] from three tests that have had 25+ days of quarantine run data with zero failures, making them eligible to rejoin the standard test suite.

Changes

Fixes #14099
Fixes #12472
Fixes #12710

Checklist

  • Is this feature complete?
    • Yes. Ready to ship.
    • No. Follow-up changes expected.
  • Are you including unit tests for the changes and scenario tests if relevant?
    • Yes
    • No
  • Did you add public API?
    • Yes
      • If yes, did you have an API Review for it?
        • Yes
        • No
      • Did you add <remarks /> and <code /> elements on your triple slash comments?
        • Yes
        • No
    • No
  • Does the change make any security assumptions or guarantees?
    • Yes
      • If yes, have you done a threat model and had a security review?
        • Yes
        • No
    • No
  • Does the change require an update in our Aspire docs?
Original prompt

This section details on the original issue you should resolve

<issue_title>[automated] Unquarantine stable tests</issue_title>
<issue_description>

Unquarantine Candidates

The following quarantined tests have had 25+ days of quarantined run data with zero failures in the last 25 days, and are candidates for unquarantining.

Last updated: 2026-03-02 23:08:32 UTC

Instructions for Copilot

Use the test-disabler agent to unquarantine the tests listed below. For each test:

  1. Remove the [Quarantine] attribute from the test method
  2. Include a Fixes <issue-url> line in the PR description for each tracking issue so they are automatically closed on merge
  3. If a test does not have a tracking issue (marked as "None" below), note this in the PR description
  4. Validate that all tests listed below have been unquarantined. If any test could not be found or was skipped, call this out in the PR description

PR requirements:

  • Prefix the PR title with [automated]
  • Apply the automated label to the PR

Candidates

Tracking Issue Oldest Run Test
#14099 2026-02-03 Aspire.Hosting.Azure.Tests.AzureCosmosDBEmulatorFunctionalTests.VerifyWaitForOnCosmosDBEmulatorBlocksDependentResources
#12472 2026-02-03 Aspire.Hosting.Tests.CliOrphanDetectorTests.CliOrphanDetectorAfterTheProcessWasRunningForAWhileThenStops
#12710 2026-02-03 Aspire.Hosting.Tests.CliOrphanDetectorTests.CliOrphanDetectorContinuesRunningWhenProcessAliveWithCorrectStartTime

</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: radical <1472+radical@users.noreply.github.com>
Copilot AI changed the title [WIP] Unquarantine stable tests with zero failures [automated] Unquarantine stable tests Mar 2, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 2, 2026

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 14862

Or

  • Run remotely in PowerShell:
iex "& { $(irm https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 14862"

@radical radical marked this pull request as ready for review March 3, 2026 00:39
@radical radical requested a review from mitchdenny as a code owner March 3, 2026 00:39
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes quarantine markers from previously flaky tests that have been stable long enough to re-enter the normal test suite.

Changes:

  • Removed [QuarantinedTest] from two CliOrphanDetectorTests test methods.
  • Removed [QuarantinedTest] from the Cosmos DB emulator functional test.
  • Cleaned up an unused using Aspire.TestUtilities; in CliOrphanDetectorTests.cs (no longer needed after attribute removal).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
tests/Aspire.Hosting.Tests/Cli/CliOrphanDetectorTests.cs Unquarantines two CLI orphan detector tests by removing [QuarantinedTest] attributes.
tests/Aspire.Hosting.Azure.Tests/AzureCosmosDBEmulatorFunctionalTests.cs Unquarantines Cosmos emulator functional test by removing [QuarantinedTest] attribute.

@radical radical enabled auto-merge (squash) March 3, 2026 00:59
@radical radical merged commit 8cc03e4 into main Mar 3, 2026
349 of 350 checks passed
@radical radical deleted the copilot/unquarantine-stable-tests-please-work branch March 3, 2026 01:02
@dotnet-policy-service dotnet-policy-service bot added this to the 13.3 milestone Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment