Don't skip event hub health checks

[Sébastien Ros (sebastienros)]

Description

Event Hub singleton health check would be skipped if health checks were not supported. We don't want to not support them so removing this skipping logic.

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]

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

tests/Aspire.Azure.Messaging.EventHubs.Tests/ConformanceTestsBase.cs:71

• Ensure that the behavior of running health checks regardless of support is covered by tests.

public void HealthChecksClientsAreReused()

[Eric Erhardt (eerhardt)]

Thanks!