tests for resourceready

[danmoseley]

Description

Closes #7364

Add tests for this which seems to have already been fixed by David in https://github.com/dotnet/aspire/pull/7163/files#diff-277b90393b291fcd3931cae92e7467ad513f793a3e5033bb85b1028b70c30a90

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?
  • Yes
    • Is this introducing a breaking change?
      • Yes
        • Link to aspire-docs issue (please use this breaking-change template):
      • No
        • Link to aspire-docs issue (please use this doc-idea template):
  • No

[Copilot]

Pull Request Overview

This PR adds tests verifying the resource health checks behave correctly under different ResourceReadyEvent conditions.

• Adds test to verify resources report an unhealthy status while a ResourceReadyEvent is running.
• Adds test to verify resources become healthy once the ResourceReadyEvent completes or when it is not present.
• Adds test to validate resources remain unhealthy when the ResourceReadyEvent fails.

[danmoseley]

https://github.com/dotnet/aspire/pull/7163/files#diff-8615d91b88608bf1e83f12f2ca649821b8594770f6408c126b824edc38096ad7R222 is a test of some of this but I wanted to unfold the various scenarios here with other similar tests.

[danmoseley]

2025-03-18T22:32:37.9303125Z   Failed Aspire.Hosting.Testing.Tests.TestingBuilderTests.CanOverrideLaunchProfileViaArgs(launchProfileName: "https", directArgs: True) [1 m 5 s]
2025-03-18T22:32:37.9303248Z   Error Message:
2025-03-18T22:32:37.9303963Z    Polly.Timeout.TimeoutRejectedException : The operation didn't complete within the allowed timeout of '00:00:10'.
2025-03-18T22:32:37.9304349Z ---- System.Threading.Tasks.TaskCanceledException : The operation was canceled.
2025-03-18T22:32:37.9304769Z -------- System.IO.IOException : Unable to read data from the transport connection: Operation canceled.
2025-03-18T22:32:37.9305046Z ------------ System.Net.Sockets.SocketException : Operation canceled

[danmoseley]

2025-03-18T22:33:06.4156736Z   Failed Aspire.Hosting.Tests.WithHttpCommandTests.WithHttpCommand_EnablesCommandOnceResourceIsRunning [1 m 2 s]
2025-03-18T22:33:06.4157247Z   Error Message:
2025-03-18T22:33:06.4157784Z    System.TimeoutException : The operation at /_/tests/Aspire.Hosting.Tests/WithHttpCommandTests.cs:370 timed out after reaching the limit of 60000ms.
2025-03-18T22:33:06.4158354Z   Stack Trace:
2025-03-18T22:33:06.4159021Z      at Microsoft.AspNetCore.InternalTesting.AsyncTestHelpers.TimeoutAfter(Task task, TimeSpan timeout, String filePath, Int32 lineNumber) in /_/tests/Shared/AsyncTestHelpers.cs:line 149
2025-03-18T22:33:06.4160308Z    at Aspire.Hosting.Tests.WithHttpCommandTests.WithHttpCommand_EnablesCommandOnceResourceIsRunning() in /_/tests/Aspire.Hosting.Tests/WithHttpCommandTests.cs:line 370
2025-03-18T22:33:06.4161033Z --- End of stack trace from previous location ---

2025-03-18T22:35:42.1540090Z   Failed Aspire.Hosting.Tests.Health.ResourceHealthCheckServiceTests.ResourcesWithoutHealthCheckAnnotationsGetReadyEventFired [5 s]
2025-03-18T22:35:42.1541288Z   Error Message:
2025-03-18T22:35:42.1542383Z    System.TimeoutException : The operation at /_/tests/Aspire.Hosting.Tests/Health/ResourceHealthCheckServiceTests.cs:360 timed out after reaching the limit of 5000ms.
2025-03-18T22:35:42.1543649Z   Stack Trace:
2025-03-18T22:35:42.1544898Z      at Microsoft.AspNetCore.InternalTesting.AsyncTestHelpers.TimeoutAfter(Task task, TimeSpan timeout, String filePath, Int32 lineNumber) in /_/tests/Shared/AsyncTestHelpers.cs:line 149
2025-03-18T22:35:42.1547670Z    at Aspire.Hosting.Tests.Health.ResourceHealthCheckServiceTests.ResourcesWithoutHealthCheckAnnotationsGetReadyEventFired() in /_/tests/Aspire.Hosting.Tests/Health/ResourceHealthCheckServiceTests.cs:line 360
2025-03-18T22:35:42.1549535Z --- End of stack trace from previous location ---

[joperezr]

LGTM

[danmoseley]

#8101